Container::has() - Code Metrics - Inspection of "Provider Requirements" - mindplay-dk/unbox - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#18)

by Rasmus

created 2019-01-18 12:08 UTC

Container::has() A

↳ Parent: Container

Complexity

Conditions	2
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	2
CRAP Score	2

Importance

Changes

Metric	Value
dl	0
loc	4
ccs	2
cts	2
cp	1
rs	10
c	0
b	0
f	0
cc	2
nc	2
nop	1
crap	2

<?php

namespace mindplay\unbox;

use Interop\Container\ContainerInterface;
use Psr\Container\ContainerInterface as PsrContainerInterface;
use ReflectionClass;
use ReflectionFunction;
use ReflectionParameter;

/**
 * This class implements a simple dependency injection container.
 */
class Container extends Configuration implements ContainerInterface, FactoryInterface
{
    /**
     * @var bool[] map where component name => TRUE, if the component has been initialized
     */
    protected $active = [];

    /**
     * @param Configuration $config
     */
    public function __construct(Configuration $config)
    {
        $config->copyTo($this);

        $this->values = $this->values +

            [
                get_class($this)             => $this,
                __CLASS__                    => $this,
                PsrContainerInterface::class => $this,
                ContainerInterface::class    => $this,
                FactoryInterface::class      => $this,
            ];
    }

    /**
     * Resolve the registered component with the given name.
     *
     * @param string $name component name
     *
     * @return mixed
     *
     * @throws NotFoundException
     */
    public function get($name)
    {
        if (! isset($this->active[$name])) {
            if (isset($this->factory[$name])) {
                $factory = $this->factory[$name];

                $reflection = new ReflectionFunction($factory);

                $params = $this->resolve($reflection->getParameters(), $this->factory_map[$name]);

                $this->values[$name] = call_user_func_array($factory, $params);
            } elseif (! array_key_exists($name, $this->values)) {
                throw new NotFoundException($name);
            }

            $this->active[$name] = true;

            $this->initialize($name);
        }

        return $this->values[$name];
    }

    /**
     * Check if a component has been unboxed and is currently active.
     *
     * @param string $name component name
     *
     * @return bool
     */
    public function isActive($name)
    {
        return isset($this->active[$name]);
    }

    /**
     * Call any given callable, using dependency injection to satisfy it's arguments, and/or
     * manually specifying some of those arguments - then return the value from the call.
     *
     * This will work for any callable:
     *
     *     $container->call('foo');               // function foo()
     *     $container->call($foo, 'baz');         // instance method $foo->baz()
     *     $container->call([Foo::class, 'bar']); // static method Foo::bar()
     *     $container->call($foo);                // closure (or class implementing __invoke)
     *
     * In any of those examples, you can also supply custom arguments, either named or
     * positional, or mixed, as per the `$map` argument in `register()`, `configure()`, etc.
     *
     * See also {@see create()} which lets you invoke any constructor.
     *
     * @param callable|object $callback any arbitrary closure or callable, or object implementing __invoke()
     * @param mixed|mixed[]   $map      mixed list/map of parameter values (and/or boxed values)
     *
     * @return mixed return value from the given callable
     */
    public function call($callback, $map = [])
    {
        $params = Reflection::createFromCallable($callback)->getParameters();

        return call_user_func_array($callback, $this->resolve($params, $map));
    }

    /**
     * Create an instance of a given class.
     *
     * The container will internally resolve and inject any constructor arguments
     * not explicitly provided in the (optional) second parameter.
     *
     * @param string        $class_name fully-qualified class-name
     * @param mixed|mixed[] $map        mixed list/map of parameter values (and/or boxed values)
     *
     * @return mixed
     *
     * @throws InvalidArgumentException
     */
    public function create($class_name, $map = [])
    {
        if (! class_exists($class_name)) {
            throw new InvalidArgumentException("unable to create component: {$class_name} (autoloading failed)");
        }

        $reflection = new ReflectionClass($class_name);

        if (! $reflection->isInstantiable()) {
            throw new InvalidArgumentException("unable to create instance of abstract class: {$class_name}");
        }

        $constructor = $reflection->getConstructor();

        $params = $constructor
            ? $this->resolve($constructor->getParameters(), $map, false)
            : [];

        return $reflection->newInstanceArgs($params);
    }

    /**
     * Internally resolves parameters to functions or constructors.
     *
     * This is the heart of the beast.
     *
     * @param ReflectionParameter[] $params parameter reflections
     * @param array                 $map    mixed list/map of parameter values (and/or boxed values)
     * @param bool                  $safe   if TRUE, it's considered safe to resolve against parameter names
     *
     * @return array parameters
     *
     * @throws ContainerException
     */
    protected function resolve(array $params, $map, $safe = true)
    {
        $args = [];

        foreach ($params as $index => $param) {
            $param_name = $param->name;

            if (array_key_exists($param_name, $map)) {
                $value = $map[$param_name]; // // resolve as user-provided named argument
            } elseif (array_key_exists($index, $map)) {
                $value = $map[$index]; // resolve as user-provided positional argument
            } else {
                // as on optimization, obtain the argument type without triggering autoload:

                $type = Reflection::getParameterType($param);

                if ($type && isset($map[$type])) {
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
                    $value = $map[$type]; // resolve as user-provided type-hinted argument
                } elseif ($type && $this->has($type)) {
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
                    $value = $this->get($type); // resolve as component registered by class/interface name
                } elseif ($safe && $this->has($param_name)) {
                    $value = $this->get($param_name); // resolve as component with matching parameter name
                } elseif ($param->isOptional()) {
                    $value = $param->getDefaultValue(); // unresolved, optional: resolve using default value
                } elseif ($type && $param->allowsNull()) {
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
                    $value = null; // unresolved, type-hinted, nullable: resolve as NULL
                } else {
                    // unresolved - throw a container exception:

                    $reflection = $param->getDeclaringFunction();

                    throw new ContainerException(
                        "unable to resolve parameter: \${$param_name} " . ($type ? "({$type}) " : "") .
                        "in file: " . $reflection->getFileName() . ", line " . $reflection->getStartLine()
                    );
                }
            }

            if ($value instanceof BoxedValueInterface) {
                $value = $value->unbox($this); // unbox a boxed value
            }

            $args[] = $value; // argument resolved!
        }

        return $args;
    }

    /**
     * Dynamically inject a component into this Container.
     *
     * Enables classes that extend `Container` to dynamically inject components (to implement "auto-wiring")
     *
     * @param string $name
     * @param mixed  $value
     */
    protected function inject($name, $value)
    {
        $this->values[$name] = $value;
        $this->active[$name] = true;
    }

    /**
     * Internally initialize an active component.
     *
     * @param string $name component name
     *
     * @return void
     */
    private function initialize($name)
    {
        if (isset($this->config[$name])) {
            foreach ($this->config[$name] as $index => $config) {
                $map = $this->config_map[$name][$index];

                $reflection = Reflection::createFromCallable($config);

                $params = $this->resolve($reflection->getParameters(), $map);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);

                $value = call_user_func_array($config, $params);

                if ($value !== null) {
                    $this->values[$name] = $value;
                }
            }
        }
    }
}


1		<?php
2
3		namespace mindplay\unbox;
4
5		use Interop\Container\ContainerInterface;
6		use Psr\Container\ContainerInterface as PsrContainerInterface;
7		use ReflectionClass;
8		use ReflectionFunction;
9		use ReflectionParameter;
10
11		/**
12		* This class implements a simple dependency injection container.
13		*/
14		class Container extends Configuration implements ContainerInterface, FactoryInterface
15		{
16		/**
17		* @var bool[] map where component name => TRUE, if the component has been initialized
18		*/
19		protected $active = [];
20
21		/**
22		* @param Configuration $config
23		*/
24	1	public function __construct(Configuration $config)
25		{
26	1	$config->copyTo($this);
27
28	1	$this->values = $this->values +
		0 ignored issues – show Documentation Bug introduced 2017-12-05 11:46 UTC by Report Bug Copy Issue Report It seems like `$this->values + array(ge...erface::class => $this)` of type `array<integer\|string,>` is incompatible with the declared type `array<integer,>` of property `$values`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
29		[
30	1	get_class($this) => $this,
31	1	__CLASS__ => $this,
32	1	PsrContainerInterface::class => $this,
33	1	ContainerInterface::class => $this,
34	1	FactoryInterface::class => $this,
35	1	];
36	1	}
37
38		/**
39		* Resolve the registered component with the given name.
40		*
41		* @param string $name component name
42		*
43		* @return mixed
44		*
45		* @throws NotFoundException
46		*/
47	1	public function get($name)
48		{
49	1	if (! isset($this->active[$name])) {
50	1	if (isset($this->factory[$name])) {
51	1	$factory = $this->factory[$name];
52
53	1	$reflection = new ReflectionFunction($factory);
54
55	1	$params = $this->resolve($reflection->getParameters(), $this->factory_map[$name]);
56
57	1	$this->values[$name] = call_user_func_array($factory, $params);
58	1	} elseif (! array_key_exists($name, $this->values)) {
59	1	throw new NotFoundException($name);
60		}
61
62	1	$this->active[$name] = true;
63
64	1	$this->initialize($name);
65	1	}
66
67	1	return $this->values[$name];
68		}
69
70		/**
71		* Check if a component has been unboxed and is currently active.
72		*
73		* @param string $name component name
74		*
75		* @return bool
76		*/
77	1	public function isActive($name)
78		{
79	1	return isset($this->active[$name]);
80		}
81
82		/**
83		* Call any given callable, using dependency injection to satisfy it's arguments, and/or
84		* manually specifying some of those arguments - then return the value from the call.
85		*
86		* This will work for any callable:
87		*
88		* $container->call('foo'); // function foo()
89		* $container->call($foo, 'baz'); // instance method $foo->baz()
90		* $container->call([Foo::class, 'bar']); // static method Foo::bar()
91		* $container->call($foo); // closure (or class implementing __invoke)
92		*
93		* In any of those examples, you can also supply custom arguments, either named or
94		* positional, or mixed, as per the `$map` argument in `register()`, `configure()`, etc.
95		*
96		* See also {@see create()} which lets you invoke any constructor.
97		*
98		* @param callable\|object $callback any arbitrary closure or callable, or object implementing __invoke()
99		* @param mixed\|mixed[] $map mixed list/map of parameter values (and/or boxed values)
100		*
101		* @return mixed return value from the given callable
102		*/
103	1	public function call($callback, $map = [])
104		{
105	1	$params = Reflection::createFromCallable($callback)->getParameters();
106
107	1	return call_user_func_array($callback, $this->resolve($params, $map));
108		}
109
110		/**
111		* Create an instance of a given class.
112		*
113		* The container will internally resolve and inject any constructor arguments
114		* not explicitly provided in the (optional) second parameter.
115		*
116		* @param string $class_name fully-qualified class-name
117		* @param mixed\|mixed[] $map mixed list/map of parameter values (and/or boxed values)
118		*
119		* @return mixed
120		*
121		* @throws InvalidArgumentException
122		*/
123	1	public function create($class_name, $map = [])
124		{
125	1	if (! class_exists($class_name)) {
126	1	throw new InvalidArgumentException("unable to create component: {$class_name} (autoloading failed)");
127		}
128
129	1	$reflection = new ReflectionClass($class_name);
130
131	1	if (! $reflection->isInstantiable()) {
132	1	throw new InvalidArgumentException("unable to create instance of abstract class: {$class_name}");
133		}
134
135	1	$constructor = $reflection->getConstructor();
136
137		$params = $constructor
138	1	? $this->resolve($constructor->getParameters(), $map, false)
139	1	: [];
140
141	1	return $reflection->newInstanceArgs($params);
142		}
143
144		/**
145		* Internally resolves parameters to functions or constructors.
146		*
147		* This is the heart of the beast.
148		*
149		* @param ReflectionParameter[] $params parameter reflections
150		* @param array $map mixed list/map of parameter values (and/or boxed values)
151		* @param bool $safe if TRUE, it's considered safe to resolve against parameter names
152		*
153		* @return array parameters
154		*
155		* @throws ContainerException
156		*/
157	1	protected function resolve(array $params, $map, $safe = true)
158		{
159	1	$args = [];
160
161	1	foreach ($params as $index => $param) {
162	1	$param_name = $param->name;
163
164	1	if (array_key_exists($param_name, $map)) {
165	1	$value = $map[$param_name]; // // resolve as user-provided named argument
166	1	} elseif (array_key_exists($index, $map)) {
167	1	$value = $map[$index]; // resolve as user-provided positional argument
168	1	} else {
169		// as on optimization, obtain the argument type without triggering autoload:
170
171	1	$type = Reflection::getParameterType($param);
172
173	1	if ($type && isset($map[$type])) {
		0 ignored issues – show Bug Best Practice introduced 2016-01-21 20:03 UTC by Report Bug Copy Issue Report The expression `$type` of type `string\|null` is loosely compared to `true`; this is ambiguous if the string can be empty. You might want to explicitly use `!== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `string` values, the empty string `''` is a special case, in particular the following results might be unexpected: '' == false // true '' == null // true 'ab' == false // false 'ab' == null // false // It is often better to use strict comparison '' === false // false '' === null // false Loading history...
174	1	$value = $map[$type]; // resolve as user-provided type-hinted argument
175	1	} elseif ($type && $this->has($type)) {
		0 ignored issues – show Bug Best Practice introduced 2016-01-21 20:03 UTC by Report Bug Copy Issue Report The expression `$type` of type `string\|null` is loosely compared to `true`; this is ambiguous if the string can be empty. You might want to explicitly use `!== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `string` values, the empty string `''` is a special case, in particular the following results might be unexpected: '' == false // true '' == null // true 'ab' == false // false 'ab' == null // false // It is often better to use strict comparison '' === false // false '' === null // false Loading history...
176	1	$value = $this->get($type); // resolve as component registered by class/interface name
177	1	} elseif ($safe && $this->has($param_name)) {
178	1	$value = $this->get($param_name); // resolve as component with matching parameter name
179	1	} elseif ($param->isOptional()) {
180	1	$value = $param->getDefaultValue(); // unresolved, optional: resolve using default value
181	1	} elseif ($type && $param->allowsNull()) {
		0 ignored issues – show Bug Best Practice introduced 2016-09-29 12:21 UTC by Report Bug Copy Issue Report The expression `$type` of type `string\|null` is loosely compared to `true`; this is ambiguous if the string can be empty. You might want to explicitly use `!== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `string` values, the empty string `''` is a special case, in particular the following results might be unexpected: '' == false // true '' == null // true 'ab' == false // false 'ab' == null // false // It is often better to use strict comparison '' === false // false '' === null // false Loading history...
182		$value = null; // unresolved, type-hinted, nullable: resolve as NULL
183		} else {
184		// unresolved - throw a container exception:
185
186	1	$reflection = $param->getDeclaringFunction();
187
188	1	throw new ContainerException(
189	1	"unable to resolve parameter: \${$param_name} " . ($type ? "({$type}) " : "") .
190	1	"in file: " . $reflection->getFileName() . ", line " . $reflection->getStartLine()
191	1	);
192		}
193		}
194
195	1	if ($value instanceof BoxedValueInterface) {
196	1	$value = $value->unbox($this); // unbox a boxed value
197	1	}
198
199	1	$args[] = $value; // argument resolved!
200	1	}
201
202	1	return $args;
203		}
204
205		/**
206		* Dynamically inject a component into this Container.
207		*
208		* Enables classes that extend `Container` to dynamically inject components (to implement "auto-wiring")
209		*
210		* @param string $name
211		* @param mixed $value
212		*/
213	1	protected function inject($name, $value)
214		{
215	1	$this->values[$name] = $value;
216	1	$this->active[$name] = true;
217	1	}
218
219		/**
220		* Internally initialize an active component.
221		*
222		* @param string $name component name
223		*
224		* @return void
225		*/
226	1	private function initialize($name)
227		{
228	1	if (isset($this->config[$name])) {
229	1	foreach ($this->config[$name] as $index => $config) {
230	1	$map = $this->config_map[$name][$index];
231
232	1	$reflection = Reflection::createFromCallable($config);
233
234	1	$params = $this->resolve($reflection->getParameters(), $map);
		0 ignored issues – show Documentation introduced 2016-09-29 12:21 UTC by Report Bug Copy Issue Report `$map` is of type `callable`, but the function expects a `array`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
235
236	1	$value = call_user_func_array($config, $params);
237
238	1	if ($value !== null) {
239	1	$this->values[$name] = $value;
240	1	}
241	1	}
242	1	}
243	1	}
244		}
245

mindplay-dk / unbox

Pull Request — master (#18)

Container::has() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like