SelamiApp\Factories\RouterFactory

Complexity

Total Complexity

4

Size/Duplication

Total Lines	42
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__invoke()	0	17	1
A	getRouter()	0	5	1
A	addRoutes()	0	6	2

<?php
declare(strict_types=1);

namespace SelamiApp\Factories;

use Zend\ServiceManager\Factory\FactoryInterface;
use Interop\Container\ContainerInterface;
use Psr\Http\Message\ServerRequestInterface;
use Selami\Router;

class RouterFactory implements FactoryInterface
{
    /**
     * @var array
     */
    private $routes;
    /**
     * @var Router
     */
    private $router;
    
    public function __invoke(ContainerInterface $container, $requestedName, array $options = null)
    {
        /**
         * @var array $config
         */
        $config = $container->get('config');
        $this->routes = $container->get('routes');

It seems like $container->get('routes') of type * is incompatible with the declared type array of property $routes.

        $request = $container->get(ServerRequestInterface::class);
        $this->router = new Router(
            $config['app']['default_return_type'] ?? Router::HTML,
            $request->getMethod(),
            $request->getUri()->getPath(),
            '',
            $config['app']['cache_file']
        );
        return $this->getRouter();
    }

    private function getRouter()
    {
        $this->addRoutes();
        return $this->router;
    }

    private function addRoutes()
    {
        foreach ($this->routes as $route) {
            $this->router->add($route[0], $route[1], $route[2], $route[3], $route[4] ?? '');
        }
    }
}