App\Component\Http\Router

Complexity

Total Complexity

7

Size/Duplication

Total Lines	102
Duplicated Lines	0 %

Test Coverage

Coverage

87.1%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	compile()	0	20	3
A	__construct()	0	10	1
A	setRoutes()	0	4	1
A	getParams()	0	3	1
A	getMatchingRoute()	0	3	1

<?php

namespace App\Component\Http;

use App\Component\Http\Interfaces\IRoutes;
use App\Component\Http\Interfaces\IRequest;
use App\Component\Http\Interfaces\IRouter;

class Router implements IRouter
{
    /**
     * active route
     *
     * @var string
     */
    private $activeRoute;

    /**
     * routes collection
     *
     * @var IRoutes
     */
    private $routes;

    /**
     * request
     *
     * @var IRequest
     */
    private $request = null;

    /**
     * route params
     *
     * @var array
     */
    private $params;

    /**
     * route match expr
     *
     * @var string
     */
    private $matchingRoute;

    /**
     * instanciate
     *
     * @param IRoutes $routes
     * @param IRequest $request
     */
    public function __construct(IRoutes $routes, IRequest $request)
    {
        $this->routes = [];

It seems like array() of type array is incompatible with the declared type App\Component\Http\Interfaces\IRoutes of property $routes.

        $this->request = $request;
        $this->activeRoute = '';
        $this->params = [];
        $this->matchingRoute = '';
        $this->setRoutes($routes);
        $this->activeRoute = substr($this->request->getUri(), 1);
        return $this;
    }

    /**
     * set routes
     *
     * @param IRoutes $routes
     * @return Router
     */
    public function setRoutes(IRoutes $routes): Router
    {
        $this->routes = $routes->get();

It seems like $routes->get() of type array is incompatible with the declared type App\Component\Http\Interfaces\IRoutes of property $routes.

        return $this;
    }

    /**
     * compile
     *
     * @return array
     */
    public function compile(): array
    {
        
        $routes = $this->routes;
        $routesLength = count($routes);

$routes of type App\Component\Http\Interfaces\IRoutes is incompatible with the type Countable|array expected by parameter $var of count().

        for ($i = 0; $i < $routesLength; $i++) {
            $matches = [];
            $match = preg_match(
                $routes[$i]->getExpr(),
                $this->activeRoute,
                $matches
            );
            if ($match) {
                $this->params = $matches;
                $this->matchingRoute = $routes[$i]->getExpr();
                array_shift($matches);
                return $matches;
            }
        }
        return [];
    }

    public function getParams():array
    {
        return $this->params;
    }

    public function getMatchingRoute():string
    {
        return $this->matchingRoute;
    }
}