Slick\Di\ContainerBuilder

Complexity

Total Complexity

12

Size/Duplication

Total Lines	99
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

100%

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	getContainer()	0	8	2
A	setContainer()	0	5	1
A	hydrateContainer()	0	11	3
A	hydrateFromFile()	0	10	2
A	hydrateFromDirectory()	0	17	3

<?php

/**
 * This file is part of slick/di package
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Slick\Di;

use Interop\Container\ContainerInterface as InteropContainer;
use Slick\Di\Exception\InvalidDefinitionsPathException;

/**
 * Container Builder
 *
 * @package Slick\Di
 * @author  Filipe Silva <silvam.filipe@gmail.com>
*/
final class ContainerBuilder implements ContainerAwareInterface
{

    /**
     * @var ContainerInterface
     */
    private $container;

    /**
     * @var array|string
     */
    private $definitions;

    public function __construct($definitions = [])
    {
        $this->definitions = $definitions;
    }

    /**
     * Get current container
     *
     * If no container was created a new, empty container will be created.
     *
     * @return ContainerInterface|Container
     */
    public function getContainer()
    {
        if (!$this->container) {
            $this->setContainer(new Container());
        }
        $this->hydrateContainer($this->definitions);
        return $this->container;
    }


    /**
     * Set the dependency container
     *
     * @param InteropContainer $container
     *
     * @return ContainerBuilder
     */
    public function setContainer(InteropContainer $container)
    {
        $this->container = $container;

$container is of type object<Interop\Container\ContainerInterface>, but the property $container was declared to be of type object<Slick\Di\ContainerInterface>. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

        return $this;
    }

    /**
     * Hydrates the container with provided definitions
     *
     * @param string|array $definitions
     */
    protected function hydrateContainer($definitions)
    {
        if (! is_array($definitions)) {
            $this->hydrateFromFile($definitions);
            return;
        }

        foreach ($definitions as $name => $definition) {
            $this->container->register($name, $definition);
        }
    }

    /**
     * Hydrate the container with definitions from provided file
     *
     * @param $definitions
     */
    protected function hydrateFromFile($definitions)
    {
        if (! is_file($definitions)) {
            $this->hydrateFromDirectory($definitions);
            return;
        }

        $services = require $definitions;
        $this->hydrateContainer($services);
    }

    protected function hydrateFromDirectory($definitions)
    {
        try {
            $directory = new \RecursiveDirectoryIterator($definitions);
        } catch (\Exception $caught) {
            throw new InvalidDefinitionsPathException(
                'Provided definitions path is not valid or is not found. ' .
                'Could not create container. Please check '.$definitions
            );
        }

        $iterator = new \RecursiveIteratorIterator($directory);
        $phpFiles = new \RegexIterator($iterator, '/.*\.php$/i');
        foreach ($phpFiles as $phpFile) {
            $this->hydrateFromFile($phpFile);
        }
    }
}