Bindings::withArgs() - Code Metrics - Inspection of "Add implementation of withArgs optionn" - ntentan/panie - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#6)

by James Ekow Abaka

created 2018-12-03 02:46 UTC

Bindings::withArgs() A

↳ Parent: Bindings

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes

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

<?php

namespace ntentan\panie;

/**
 * Holds all the bindings for the dependency injection.
 */
class Bindings
{

    /**
     * An array of all the bindings.
     *
     * @var array
     */
    private $bindings = [];
    
    /**
     * Key of the binding that would be altered with subsequent calls to a bindings object.
     *
     * @var string 
     */
    private $activeKey;

    /**
     * Set the key of the binding to be modified or accessed on subsequent calls.
     *
     * @param $activeKey
     * @return self
     */
    public function setActiveKey(string $activeKey) : self
    {
        $this->activeKey = $activeKey;
        return $this;
    }

    /**
     * Registers a method to be called when an instance is created for the current active binding.
     * This method could be a setter or some other optional initialization method. Parameters for methods registered
     * with call that have type hints are also injected.
     *
     * @param string $method
     * @param array $parameters
     * @return self
     */
    public function call(string $method, array $parameters = []) : self
    {
        $this->bindings[$this->activeKey]['calls'][] = [$method, $parameters];
        return $this;
    }

    /**
     * Provides a concrete class or factory function to which the currently selected binding should be linked.
     *
     * @param mixed $value
     * @return self
     */
    public function to($value)
    {
        if(isset($this->bindings[$this->activeKey])) {
            $this->bindings[$this->activeKey]['binding'] = $value;
        } else {
            $this->bindings[$this->activeKey] = ['binding' => $value, 'calls' => [], 'properties' => []];
        }
        return $this;
    }

    public function withArgs(array $args)
    {
        $this->bindings[$this->activeKey]['args'] = $args;
    }

    /**
     * Get the configuration of a binding.
     *
     * @param string $key
     * @return mixed
     */
    public function get(string $key)
    {
        $binding = $this->bindings[$key];
        if(!isset($binding['binding'])) {
            $binding['binding'] = $key;
        }
        return $binding;
    }

    /**
     * Checks if a binding exists.
     *
     * @param string $key
     * @return bool
     */
    public function has($key)
    {
        return isset($this->bindings[$key]);
    }

    /**
     * Register the current active binding as a singleton.
     *
     * @param bool $singleton
     */
    public function asSingleton($singleton = true)
    {
        $this->bindings[$this->activeKey]['singleton'] = $singleton;
    }

    /**
     * Setup bindings from an array.
     *
     * @param array $binding
     */
    private function setArrayBinding(array $binding)
    {
        if(isset($binding[0])) {
            $this->to($binding[0]);
        }
        
        if(isset($binding['calls'])){
            foreach($binding['calls'] as $key => $call) {
                if(is_string($call)) {
                    $method = $call;
                    $parameters = [];
                } else if(is_array($call)) {
                    $method = $key;
                    $parameters = $call;
                }
                $this->call($method, $parameters);
function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}
            }
        } else {
            $this->bindings[$this->activeKey]['calls'] = [];
        }

        $this->asSingleton($binding['singleton'] ?? false);
    }

    /**
     * Merge current bindings with another configuration overwriting conflicting values with the new bindings.
     *
     * @param array $bindings
     */
    public function merge(array $bindings)
    {
        foreach($bindings as $key => $binding) {
            $this->activeKey = $key;
            if(is_array($binding)) {
                $this->setArrayBinding($binding);
            } else {
                $this->bindings[$key] = ['binding' => $binding, 'calls' => [], 'properties' => []];
            }
        }
    }

}


1		<?php
2
3		namespace ntentan\panie;
4
5		/**
6		* Holds all the bindings for the dependency injection.
7		*/
8		class Bindings
9		{
10
11		/**
12		* An array of all the bindings.
13		*
14		* @var array
15		*/
16		private $bindings = [];
17
18		/**
19		* Key of the binding that would be altered with subsequent calls to a bindings object.
20		*
21		* @var string
22		*/
23		private $activeKey;
24
25		/**
26		* Set the key of the binding to be modified or accessed on subsequent calls.
27		*
28		* @param $activeKey
29		* @return self
30		*/
31	10	public function setActiveKey(string $activeKey) : self
32		{
33	10	$this->activeKey = $activeKey;
34	10	return $this;
35		}
36
37		/**
38		* Registers a method to be called when an instance is created for the current active binding.
39		* This method could be a setter or some other optional initialization method. Parameters for methods registered
40		* with call that have type hints are also injected.
41		*
42		* @param string $method
43		* @param array $parameters
44		* @return self
45		*/
46	3	public function call(string $method, array $parameters = []) : self
47		{
48	3	$this->bindings[$this->activeKey]['calls'][] = [$method, $parameters];
49	3	return $this;
50		}
51
52		/**
53		* Provides a concrete class or factory function to which the currently selected binding should be linked.
54		*
55		* @param mixed $value
56		* @return self
57		*/
58	12	public function to($value)
59		{
60	12	if(isset($this->bindings[$this->activeKey])) {
61	1	$this->bindings[$this->activeKey]['binding'] = $value;
62		} else {
63	12	$this->bindings[$this->activeKey] = ['binding' => $value, 'calls' => [], 'properties' => []];
64		}
65	12	return $this;
66		}
67
68		public function withArgs(array $args)
69		{
70		$this->bindings[$this->activeKey]['args'] = $args;
71		}
72
73		/**
74		* Get the configuration of a binding.
75		*
76		* @param string $key
77		* @return mixed
78		*/
79	12	public function get(string $key)
80		{
81	12	$binding = $this->bindings[$key];
82	12	if(!isset($binding['binding'])) {
83		$binding['binding'] = $key;
84		}
85	12	return $binding;
86		}
87
88		/**
89		* Checks if a binding exists.
90		*
91		* @param string $key
92		* @return bool
93		*/
94	19	public function has($key)
95		{
96	19	return isset($this->bindings[$key]);
97		}
98
99		/**
100		* Register the current active binding as a singleton.
101		*
102		* @param bool $singleton
103		*/
104	3	public function asSingleton($singleton = true)
105		{
106	3	$this->bindings[$this->activeKey]['singleton'] = $singleton;
107	3	}
108
109		/**
110		* Setup bindings from an array.
111		*
112		* @param array $binding
113		*/
114	2	private function setArrayBinding(array $binding)
115		{
116	2	if(isset($binding[0])) {
117	2	$this->to($binding[0]);
118		}
119
120	2	if(isset($binding['calls'])){
121	1	foreach($binding['calls'] as $key => $call) {
122	1	if(is_string($call)) {
123	1	$method = $call;
124	1	$parameters = [];
125	1	} else if(is_array($call)) {
126	1	$method = $key;
127	1	$parameters = $call;
128		}
129	1	$this->call($method, $parameters);
		0 ignored issues – show Bug introduced 2017-12-20 07:29 UTC by Report Bug Copy Issue Report The variable `$method` does not seem to be defined for all execution paths leading up to this point. If you define a variable conditionally, it can happen that it is not defined for all execution paths. Let’s take a look at an example: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } // $x is potentially undefined here. echo $x; } In the above example, the variable `$x` is defined if you pass “foo” or “bar” as argument for `$a`. However, since the `switch` statement has no default case statement, if you pass any other value, the variable `$x` would be undefined. Available Fixes Check for existence of the variable explicitly: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } if (isset($x)) { // Make sure it's always set. echo $x; } } Define a default value for the variable: function myFunction($a) { $x = ''; // Set a default which gets overridden for certain paths. switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } echo $x; } Add a value for the missing path: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; // We add support for the missing case. default: $x = ''; break; } echo $x; } Loading history... Bug introduced 2017-12-20 07:29 UTC by Report Bug Copy Issue Report The variable `$parameters` does not seem to be defined for all execution paths leading up to this point. If you define a variable conditionally, it can happen that it is not defined for all execution paths. Let’s take a look at an example: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } // $x is potentially undefined here. echo $x; } In the above example, the variable `$x` is defined if you pass “foo” or “bar” as argument for `$a`. However, since the `switch` statement has no default case statement, if you pass any other value, the variable `$x` would be undefined. Available Fixes Check for existence of the variable explicitly: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } if (isset($x)) { // Make sure it's always set. echo $x; } } Define a default value for the variable: function myFunction($a) { $x = ''; // Set a default which gets overridden for certain paths. switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } echo $x; } Add a value for the missing path: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; // We add support for the missing case. default: $x = ''; break; } echo $x; } Loading history...
130		}
131		} else {
132	1	$this->bindings[$this->activeKey]['calls'] = [];
133		}
134
135	2	$this->asSingleton($binding['singleton'] ?? false);
136	2	}
137
138		/**
139		* Merge current bindings with another configuration overwriting conflicting values with the new bindings.
140		*
141		* @param array $bindings
142		*/
143	5	public function merge(array $bindings)
144		{
145	5	foreach($bindings as $key => $binding) {
146	5	$this->activeKey = $key;
147	5	if(is_array($binding)) {
148	2	$this->setArrayBinding($binding);
149		} else {
150	5	$this->bindings[$key] = ['binding' => $binding, 'calls' => [], 'properties' => []];
151		}
152		}
153	5	}
154
155		}
156

ntentan / panie

Pull Request — master (#6)

Bindings::withArgs() A

Complexity

Size

Duplication

Code Coverage

Importance

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like