pjpawel\LightApi\Container\ContainerLoader

Complexity

Total Complexity

14

Size/Duplication

Total Lines	78
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	createDefinitionsFromConfig()	0	11	6
A	add()	0	5	1
A	getDefinitions()	0	11	3
A	prepareContainerLocator()	0	4	2
A	addDefinitions()	0	4	2

<?php

namespace pjpawel\LightApi\Container;

use pjpawel\LightApi\Container\Definition\AliasDefinition;
use pjpawel\LightApi\Container\Definition\ClassDefinition;
use pjpawel\LightApi\Container\Definition\Definition;
use pjpawel\LightApi\Container\Definition\InterfaceDefinition;
use pjpawel\LightApi\Exception\ProgrammerException;
use Psr\Container\ContainerInterface;

class ContainerLoader implements ContainerInterface
{

    use ContainerTrait;

The trait pjpawel\LightApi\Container\ContainerTrait requires some properties which are not provided by pjpawel\LightApi\Container\ContainerLoader: $name, $className, $arguments

    /**
     * @var array<string,Definition>
     */
    public array $definitions;

    /**
     * @param array<string, string|array> $config
     * @throws \Exception
     */
    public function createDefinitionsFromConfig(array $config): void
    {
        foreach ($config as $name => $value) {
            if (is_string($value) && str_starts_with($value, '@')) {
                $this->definitions[$name] = new AliasDefinition($name, substr($value, 1));
            } elseif (class_exists($name)) {
                $this->definitions[$name] = new ClassDefinition($name, $value);

It seems like $value can also be of type string; however, parameter $arguments of pjpawel\LightApi\Container\Definition\ClassDefinition::__construct() does only seem to accept array, maybe add an additional type check?

            } elseif (interface_exists($name)) {
                $this->definitions[$name] = new InterfaceDefinition($name, $value);

It seems like $value can also be of type array; however, parameter $className of pjpawel\LightApi\Container\Definition\InterfaceDefinition::__construct() does only seem to accept string, maybe add an additional type check?

            } else {
                throw new ProgrammerException('Invalid container definition for name ' . $name);
            }
        }
    }

    /**
     * You must use name and args. Optionally you can provide object
     *
     * @param array $definition
     */
    public function add(array $definition): void
    {
        $newDefinition = new ClassDefinition($definition['name'], $definition['args'] ?? []);
        $newDefinition->object = $definition['object'] ?? null;
        $this->definitions[$definition['name']] = $newDefinition;
    }

    /**
     * @param array<string,Definition> $definitions
     * @return void
     */
    public function addDefinitions(array $definitions): void
    {
        foreach ($definitions as $id => $definition) {
            $this->definitions[$id] = $definition;
        }
    }

    /**
     * @param string[] $ids
     * @return Definition[]
     * @throws \pjpawel\LightApi\Container\ContainerNotFoundException
     */
    public function getDefinitions(array $ids): array
    {
        /** @var Definition[] $definitions */
        $definitions = [];
        foreach ($ids as $id) {
            if (!isset($this->definitions[$id])) {
                throw new ContainerNotFoundException();
            }
            $definitions[] = $this->definitions[$id];
        }
        return $definitions;
    }

    /**
     * @param array<string,string> $services
     * @return void
     */
    public function prepareContainerLocator(array $services): void
    {
        foreach ($services as $internalId => $serviceId) {
            $this->definitions[$internalId] = new AliasDefinition($internalId, $serviceId);
        }
    }
}