TomPHP\ContainerConfigurator\League\ServiceServiceProvider

Complexity

Total Complexity

13

Size/Duplication

Total Lines	124
Duplicated Lines	7.26 %

Coupling/Cohesion

Components	1
Dependencies	6

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	register()	0	6	2
B	registerService()	0	34	4
A	addMethodCalls()	0	6	2
A	createAliasFactory()	0	6	1
A	createFactoryFactory()	9	9	1
A	resolveArguments()	0	13	2

<?php

namespace TomPHP\ContainerConfigurator\League;

use League\Container\Definition\ClassDefinition;
use League\Container\ServiceProvider\AbstractServiceProvider;
use TomPHP\ContainerConfigurator\Exception\NotClassDefinitionException;
use TomPHP\ContainerConfigurator\ServiceConfig;
use TomPHP\ContainerConfigurator\ServiceDefinition;

final class ServiceServiceProvider extends AbstractServiceProvider
{
    /**
     * @var array
     */
    private $config;

    /**
     * @api
     *
     * @param ServiceConfig $config
     */
    public function __construct(ServiceConfig $config)
    {
        $this->config   = $config;

It seems like $config of type object<TomPHP\ContainerConfigurator\ServiceConfig> is incompatible with the declared type array of property $config.

        $this->provides = $config->getKeys();
    }

    public function register()
    {
        foreach ($this->config as $config) {
            $this->registerService($config);
        }
    }

    /**
     * @param ServiceDefinition $definition
     *
     * @throws NotClassDefinitionException
     *
     * @return void
     */
    private function registerService(ServiceDefinition $definition)
    {
        if ($definition->isFactory()) {
            $this->getContainer()->add(
                $definition->getName(),
                $this->createFactoryFactory($definition),
                $definition->isSingleton()
            );

            return;
        }

        if ($definition->isAlias()) {
            $this->getContainer()->add(
                $definition->getName(),
                $this->createAliasFactory($definition)
            );

            return;
        }

        $service = $this->getContainer()->add(
            $definition->getName(),
            $definition->getClass(),
            $definition->isSingleton()
        );

        if (!$service instanceof ClassDefinition) {
            throw NotClassDefinitionException::fromServiceName($definition->getName());
        }

        $service->withArguments($definition->getArguments());
        $this->addMethodCalls($service, $definition);
    }

    /**
     * @param ClassDefinition   $service
     * @param ServiceDefinition $definition
     */
    private function addMethodCalls(ClassDefinition $service, ServiceDefinition $definition)
    {
        foreach ($definition->getMethods() as $method => $args) {
            $service->withMethodCall($method, $args);
        }
    }

    /**
     * @param ServiceDefinition $definition
     *
     * @return \Closure
     */
    private function createAliasFactory(ServiceDefinition $definition)
    {
        return function () use ($definition) {
            return $this->getContainer()->get($definition->getClass());
        };
    }

    /**
     * @param ServiceDefinition $definition
     *
     * @return \Closure
     */
    private function createFactoryFactory(ServiceDefinition $definition)

This method seems to be duplicated in your project.

    {
        return function () use ($definition) {
            $className = $definition->getClass();
            $factory   = new $className();

            return $factory(...$this->resolveArguments($definition->getArguments()));
        };
    }

    /**
     * @param array $arguments
     *
     * @return array
     */
    private function resolveArguments(array $arguments)
    {
        return array_map(
            function ($argument) {
                if ($this->container->has($argument)) {
                    return $this->container->get($argument);
                }

                return $argument;
            },
            $arguments
        );
    }
}