Psi\Bundle\ObjectRender\Template\Locator

Complexity

Total Complexity

8

Size/Duplication

Total Lines	49
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	locate()	0	18	3
A	resolvePath()	0	18	4

<?php

declare(strict_types=1);

namespace Psi\Bundle\ObjectRender\Template;

/**
 * Determine a file path from a given class reflection class.
 *
 * An *ordered* map of namespaces => paths must be provided. It should be
 * ordered by namespace length (longest to shortest).
 */
class Locator
{
    private $extension;
    private $map;

    public function __construct(array $map, string $extension)
    {
        $this->extension = $extension;
        $this->map = $map;
    }

    public function locate(\ReflectionClass $class): string
    {
        list($resolvedNs, $basePath) = $this->resolvePath($class->getName());

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

        $fname = str_replace('\\', '.', substr($class->getName(), strlen($resolvedNs)));

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

        if (0 === strpos($fname, '.')) {
            $fname = substr($fname, 1);
        }

        $fpath = sprintf(
            '%s/%s.%s',
            $basePath,
            $fname,
            ($this->extension ? $this->extension : '')
        );

        return $fpath;
    }

    private function resolvePath($classFqn): array
    {
        $resolved = null;

$resolved is not used, you could remove the assignment.

        foreach ($this->map as $namespace => $path) {
            if ($namespace === $classFqn) {
                continue;
            }

            if (0 === strpos($classFqn, $namespace)) {
                return [$namespace, $path];
            }
        }

        throw new \InvalidArgumentException(sprintf(
            'Could not resolve path for class "%s" in namespaces: "%s"',
            $classFqn, implode('", "', array_keys($this->map))
        ));
    }
}