evelikto\di\creator\InjectableCreator::fromClass()

Complexity

Conditions	4
Paths	5

Size

Total Lines	15
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    protected function fromClass(string $name) {
        if (class_exists($name, true) === false)
            return null;

        if (method_exists($name, '__construct') === false)
            $value = new $name;
        else {
            $method = new \ReflectionMethod($name, '__construct');
            $value = new $name(...$this->autowire($method));

It seems like autowire() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }

        if ($value instanceof Injectable)
            return new Bean($name, $value);

It seems like $name can also be of type object; however, parameter $name of evelikto\di\Bean::__construct() does only seem to accept string, maybe add an additional type check?

        throw new ClassNotInjectable($name);

It seems like $name can also be of type object; however, parameter $name of evelikto\di\creator\ClassNotInjectable::__construct() does only seem to accept string, maybe add an additional type check?

    }
}