Wandu\Router\Loader\ArrayAccessLoader

Complexity

Total Complexity

7

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Test Coverage

Coverage

92.86%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	middleware()	0	7	2
A	execute()	0	11	4

<?php
namespace Wandu\Router\Loader;

use Psr\Http\Message\ServerRequestInterface;
use Wandu\Router\Contracts\LoaderInterface;
use Wandu\Router\Contracts\MiddlewareInterface;
use Wandu\Router\Exception\HandlerNotFoundException;

class ArrayAccessLoader implements LoaderInterface
{
    /** @var \ArrayAccess */
    protected $container;

    /**
     * @param \ArrayAccess|array $container
     */
    public function __construct($container)
    {
        $this->container = $container;

It seems like $container can also be of type array. However, the property $container is declared as type object<ArrayAccess>. Maybe add an additional type check?

    }

    /**
     * {@inheritdoc}
     */
    public function middleware(string $className, ServerRequestInterface $request): MiddlewareInterface
    {
        if (!isset($this->container[$className])) {
            throw new HandlerNotFoundException($className);
        }
        return $this->container[$className];
    }

    /**
     * {@inheritdoc}
     */
    public function execute(string $className, string $methodName, ServerRequestInterface $request)
    {
        if (!isset($this->container[$className])) {
            throw new HandlerNotFoundException($className, $methodName);
        }
        $object = $this->container[$className];
        if (!method_exists($object, $methodName) && !method_exists($object, '__call')) {
            throw new HandlerNotFoundException($className, $methodName);
        }
        return call_user_func([$object, $methodName], $request);
    }
}