Bankiru\Api\Rpc\Routing\Loader\AnnotationClassLoader

Complexity

Total Complexity

20

Size/Duplication

Total Lines	112
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Test Coverage

Coverage

74.58%

Importance

Changes	2
Bugs	0	Features	0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
C	load()	0	34	7
A	getParentAnnotations()	0	21	4
A	addRoute()	0	19	2
A	supports()	0	6	4
A	getResolver()	0	3	1
A	setResolver()	0	3	1

<?php
/**
 * Created by PhpStorm.
 * User: batanov.pavel
 * Date: 16.02.2016
 * Time: 13:01
 */

namespace Bankiru\Api\Rpc\Routing\Loader;

use Bankiru\Api\Rpc\Routing\Annotation\Method;
use Bankiru\Api\Rpc\Routing\LoaderInterface;
use Bankiru\Api\Rpc\Routing\MethodCollection;
use Bankiru\Api\Rpc\Routing\Route;
use Doctrine\Common\Annotations\Reader;
use Symfony\Component\Config\Resource\FileResource;

class AnnotationClassLoader implements LoaderInterface
{
    /** @var  Reader */
    private $reader;

    /**
     * Constructor.
     *
     * @param Reader $reader
     */
    public function __construct(Reader $reader)
    {
        $this->reader = $reader;
    }


    /** {@inheritdoc} */
    public function load($class, $type = null)
    {
        if (!class_exists($class)) {
            throw new \InvalidArgumentException(sprintf('Class "%s" does not exist.', $class));
        }

        $class = new \ReflectionClass($class);
        if ($class->isAbstract()) {
            throw new \InvalidArgumentException(
                sprintf('Annotations from class "%s" cannot be read as it is abstract.', $class->getName())

Consider using $class->name. There is an issue with getName() and APC-enabled PHP versions.

            );
        }

        $parents = $this->getParentAnnotations($class);

        $collection = new MethodCollection();
        $collection->addResource(new FileResource($class->getFileName()));

        foreach ($class->getMethods() as $method) {
            if (!$method->isPublic()) {
                continue;
            }
            
            foreach ($this->reader->getMethodAnnotations($method) as $annot) {
                if ($annot instanceof Method) {
                    $this->addRoute($collection, $annot, $parents, $class, $method);
                }
            }
        }

        $collection->addPrefix($parents['method']);

        return $collection;
    }

    protected function getParentAnnotations(\ReflectionClass $class)
    {
        $parents = [
            'method'          => '',
            'context'         => [],
            'default_context' => true,
        ];

        /** @var Method $annot */
        if ($annot = $this->reader->getClassAnnotation($class, Method::class)) {
            if (null !== $annot->getMethod()) {
                $parents['method'] = $annot->getMethod();
            }

            if (null !== $annot->getContext()) {
                $parents['context'] = $annot->getContext();
            }
        }

        return $parents;
    }

    protected function addRoute(
        MethodCollection $collection,
        Method $annot,
        array $parents,
        \ReflectionClass $class,
        \ReflectionMethod $method
    )
    {
        $collection->add(
            $annot->getMethod(),
            new Route(
                $annot->getMethod(),
                $class->getName() . '::' . $method->getName(),

Consider using $class->name. There is an issue with getName() and APC-enabled PHP versions.

                array_merge($parents['context'], $annot->getContext()),
                $parents['default_context'] && $annot->isDefaultContext(),
                $annot->isInherit()
            )
        );
    }

    /** {@inheritdoc} */
    public function supports($resource, $type = null)
    {
        return is_string($resource) &&
               preg_match('/^(?:\\\\?[a-zA-Z_\x7f-\xff][a-zA-Z0-9_\x7f-\xff]*)+$/', $resource) &&
               (!$type || 'annotation' === $type);

The expression $type of type string|null is loosely compared to false; this is ambiguous if the string can be empty. You might want to explicitly use === null instead.

    }

    /** {@inheritdoc} */
    public function getResolver()
    {
    }

    /** {@inheritdoc} */
    public function setResolver($resolver)
    {
    }
}