Router\Router

Complexity

Total Complexity

22

Size/Duplication

Total Lines	119
Duplicated Lines	0 %

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
B	checkPath()	0	17	6
B	parsePath()	0	28	6
A	checkVerb()	0	7	2
A	handle()	0	9	4
A	add()	0	3	1
A	setArgs()	0	4	2

<?php

namespace Router;

/**
 * Class Router
 * @package Router
 */
class Router
{
    /**
     * @var array
     */
    private $server;

    /**
     * @var array
     */
    private static $routes = [];

    /**
     * Router constructor.
     */
    public function __construct()
    {
        $this->server = new Server();

It seems like new Router\Server() of type Router\Server is incompatible with the declared type array of property $server.

    }

    /**
     * @param array $route
     */
    public function add(array $route): void
    {
        self::$routes[] = new Client($route);
    }

    /**
     * @return null|Client
     */
    public function handle(): ?Client
    {
        foreach (self::$routes as $route) {
            if ($this->checkVerb($route) && $this->checkPath($route)) {
                $this->setArgs($route);
                return $route;
            }
        }
        return null;
    }

    /**
     * @param Client $route
     * @return bool
     */
    private function checkVerb(Client $route): bool
    {
        if ($this->server->getMethod() != $route->getMethod()) {
            return false;
        }

        return true;
    }

    /**
     * @param Client $route
     * @return bool
     */
    private function checkPath(Client $route): bool
    {
        if (count($route->getUri()) != count($this->server->getUri())) {
            return false;
        }

        if (!$this->parsePath($route)) {
            return false;
        }

        for ($i = 0; $i < count($route->getUri()) && count($this->server->getUri()); $i++) {
            if ($route->getUri()[$i] != $this->server->getUri()[$i]) {
                return false;
            }
        }

        return true;
    }

    /**
     * @param Client $route
     * @return bool
     */
    private function parsePath(Client $route): bool
    {
        $serverUri = $this->server->getUri();
        $routeUri = $route->getUri();

        for ($i = 0; $i < count($routeUri); $i++) {

It seems like you are calling the size function count() as part of the test condition. You might want to compute the size beforehand, and not on each iteration.

            $serverUri[$i] = preg_replace('/\@.*/', $this->server->getUri()[$i], $routeUri[$i]);
            if ($serverUri[$i] != $routeUri[$i]) {
                // Regular expression named parameter matching
                if (preg_match('/\@([\w]+):(.*)/', $routeUri[$i], $match)) {
                    if (preg_match('/'.$match[2].'/', $serverUri[$i], $match[2])) {
                        if ($serverUri[$i] != $match[2][0]) {
                            return false;
                        }
                        $route->setArg($match[1], $match[2][0]);
                    } else {
                        return false;
                    }
                } else {
                    $route->setArg(str_replace(':', '', $routeUri[$i]), $serverUri[$i]);
                }
            }
        }
            
        $serverUri = implode('/', $serverUri);
        $route->setUri($serverUri);

        return true;
    }

    /**
     * @param Client $route
     */
    private function setArgs(Client $route): void
    {
        foreach ($_POST as $key => $value) {
            $route->setArg($key, $value);
        }
    }
}