League\Container\SingleInstanceContainer

Complexity

Total Complexity

4

Size/Duplication

Total Lines	41
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	get()	0	8	2
A	has()	0	4	1

<?php

namespace League\Container;

use Interop\Container\ContainerInterface as InteropContainerInterface;

/**
 * Wraps a container intercepting requests for dependencies and caching the return.
 *
 * This ensures that a single instance of any service ID is only ever returned. This
 * be used with a reflection container to provide zero-configuration, single instance
 * DI.
 */
class SingleInstanceContainer implements InteropContainerInterface
{
    /**
     * @var ContainerInterface
     */
    private $wrapped;
    
    /**
     * @var mixed[]
     */
    private $instances = [];
    
    
    /**
     * @param InteropContainerInterface $container
     */
    public function __construct(InteropContainerInterface $container)
    {
        $this->wrapped = $container;

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

    }
    
    /**
     * @inheritdoc
     */
    public function get($id)
    {
        if (!isset($this->instances[$id])) {
            $this->instances[$id] = $this->wrapped->get($id);
        }
        
        return $this->instances[$id];
    }
    
    /**
     * @inheritdoc
     */
    public function has($id)
    {
        return $this->wrapped->has($id);
    }
}