Nip\Router\RouterAwareTrait

Complexity

Total Complexity

7

Size/Duplication

Total Lines	55
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

0%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	route()	0	7	2
A	getRouter()	0	8	2
A	setRouter()	0	6	1
A	initRouter()	0	4	1
A	newRouter()	0	4	1

<?php

namespace Nip\Router;

use Nip\Request;

/**
 * Class ConfigAwareTrait
 * @package Nip\Router
 */
trait RouterAwareTrait
{
    /**
     * @var Router|null
     */
    protected $router = null;

    /**
     * @param bool|Request $request
     * @return array
     */
    public function route($request = false)
    {
        $request = $request ? $request : $this->getRequest();

It seems like getRequest() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        $params = $this->getRouter()->route($request);

        return $params;
    }

    /**
     * @return Router
     */
    public function getRouter()
    {
        if (!$this->router) {
            $this->initRouter();
        }

        return $this->router;
    }

    /**
     * @param bool|Router $router
     * @return $this
     */
    public function setRouter($router = false)
    {
        $this->router = $router;

It seems like $router can also be of type boolean. However, the property $router is declared as type object<Nip\Router\Router>|null. Maybe add an additional type check?

        return $this;
    }

    protected function initRouter()
    {
        $this->setRouter($this->newRouter());
    }

    /**
     * @return Router
     */
    protected function newRouter()
    {
        return app()->get('router');
    }
}