Laganica\Di\Resolver\ClassResolver

Complexity

Total Complexity

2

Size/Duplication

Total Lines	18
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Test Coverage

Coverage

83.33%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	resolve()	0	12	2

<?php

namespace Laganica\Di\Resolver;

use Laganica\Di\Definition\DefinitionInterface;
use Laganica\Di\Exception\NotFoundException;

/**
 * Class AutowireResolver
 *
 * @package Laganica\Di\Resolver
 */
class ClassResolver extends ReflectionResolver
{
    /**
     * @inheritDoc
     */
    public function resolve(DefinitionInterface $definition)
    {
        $class = $definition->getValue();

        if (class_exists($class)) {
            $params = $this->getConstructorParams($class);

It seems like $class defined by $definition->getValue() on line 20 can also be of type object<Closure>; however, Laganica\Di\Resolver\ReflectionResolver::getConstructorParams() does only seem to accept string, maybe add an additional type check?

            return new $class(...$params);
        }

        throw NotFoundException::create($class);

It seems like $class defined by $definition->getValue() on line 20 can also be of type object<Closure>; however, Laganica\Di\Exception\NotFoundException::create() does only seem to accept string, maybe add an additional type check?

    }
}