Evaluator::data() - Code Metrics - Inspection of "Use our own data_get alternative in order to avoid..." - anomalylabs/streams-platform - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#210)

by Ryan

created 2016-05-27 20:09 UTC

Evaluator::data() C

↳ Parent: Evaluator

Complexity

Conditions	11
Paths	17

Size

Total Lines	43
Code Lines	20

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	43
rs	5.2653
cc	11
eloc	20
nc	17
nop	2

How to fix Complexity

<?php namespace Anomaly\Streams\Platform\Support;

use ArrayAccess;
use Illuminate\Contracts\Container\Container;

/**
 * Class Evaluator
 *
 * @link    http://anomaly.is/streams-platform
 * @author  AnomalyLabs, Inc. <[email protected]>
 * @author  Ryan Thompson <[email protected]>
 * @package Anomaly\Streams\Platform\Support
 */
class Evaluator
{

    /**
     * The IoC container.
     *
     * @var \Illuminate\Contracts\Container\Container
     */
    protected $container;

    /**
     * Create a new Evaluator instance.
     *
     * @param Container $container
     */
    public function __construct(Container $container)
    {
        $this->container = $container;
    }

    /**
     * Evaluate a target entity with arguments.
     *
     * @param        $target
     * @param  array $arguments
     * @return mixed
     */
    public function evaluate($target, array $arguments = [])
    {
        /**
         * If the target is an instance of Closure then
         * call through the IoC it with the arguments.
         */
        if ($target instanceof \Closure) {
            return $this->container->call($target, $arguments);
        }

        /**
         * If the target is an array then evaluate
         * each of it's values.
         */
        if (is_array($target)) {
            foreach ($target as &$value) {
                $value = $this->evaluate($value, $arguments);
            }
        }

        /**
         * if the target is a string and is in a traversable
         * format then traverse the target using the arguments.
         */
        if (is_string($target) && !isset($arguments[$target]) && $this->isTraversable($target)) {
            $target = $this->data($arguments, $target, $target);

        }

        return $target;
    }

    /**
     * Check if a string is in a traversable format.
     *
     * @param  $target
     * @return bool
     */
    protected function isTraversable($target)
    {
        return (!preg_match('/[^a-z._]/', $target));
    }

    /**
     * Return the data from the key.
     *
     * @param $arguments
     * @param $target
     * @return mixed
     */
    protected function data($arguments, $target)
    {
        if (is_null($arguments)) {
            return $target;
        }

        $arguments = is_array($arguments) ? $arguments : explode('.', $arguments);

        foreach ($arguments as $segment) {
            if (is_array($target)) {
                if (!array_key_exists($segment, $target)) {
                    return value($target);
                }

                $target = $target[$segment];
            } elseif ($target instanceof ArrayAccess) {
                if (!isset($target[$segment])) {
                    return value($target);
                }

                $target = $target[$segment];
            } elseif (is_object($target)) {

                /**
                 * Check if the method exists first
                 * otherwise we might end up trying to
                 * access relational methods.
                 *
                 * Otherwise this is identical from
                 * data_get helper.
                 */
                if (method_exists($target, $segment) || !isset($target->{$segment})) {
                    return value($target);
                }

                $target = $target->{$segment};
            } else {
                return value($target);
            }
        }

        return $target;
    }
}


1			<?php namespace Anomaly\Streams\Platform\Support;
2
3			use ArrayAccess;
4			use Illuminate\Contracts\Container\Container;
5
6			/**
7			* Class Evaluator
8			*
9			* @link http://anomaly.is/streams-platform
10			* @author AnomalyLabs, Inc. <[email protected]>
11			* @author Ryan Thompson <[email protected]>
12			* @package Anomaly\Streams\Platform\Support
13			*/
14			class Evaluator
15			{
16
17			/**
18			* The IoC container.
19			*
20			* @var \Illuminate\Contracts\Container\Container
21			*/
22			protected $container;
23
24			/**
25			* Create a new Evaluator instance.
26			*
27			* @param Container $container
28			*/
29			public function __construct(Container $container)
30			{
31			$this->container = $container;
32			}
33
34			/**
35			* Evaluate a target entity with arguments.
36			*
37			* @param $target
38			* @param array $arguments
39			* @return mixed
40			*/
41			public function evaluate($target, array $arguments = [])
42			{
43			/**
44			* If the target is an instance of Closure then
45			* call through the IoC it with the arguments.
46			*/
47			if ($target instanceof \Closure) {
48			return $this->container->call($target, $arguments);
49			}
50
51			/**
52			* If the target is an array then evaluate
53			* each of it's values.
54			*/
55			if (is_array($target)) {
56			foreach ($target as &$value) {
57			$value = $this->evaluate($value, $arguments);
58			}
59			}
60
61			/**
62			* if the target is a string and is in a traversable
63			* format then traverse the target using the arguments.
64			*/
65			if (is_string($target) && !isset($arguments[$target]) && $this->isTraversable($target)) {
66			$target = $this->data($arguments, $target, $target);
			0 ignored issues – show Unused Code introduced 2016-05-27 20:17 UTC by Report Bug Copy Issue Report The call to `Evaluator::data()` has too many arguments starting with `$target`. This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue. If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress. In this case you can add the `@ignore` PhpDoc annotation to the duplicate definition and it will be ignored. Loading history...
67			}
68
69			return $target;
70			}
71
72			/**
73			* Check if a string is in a traversable format.
74			*
75			* @param $target
76			* @return bool
77			*/
78			protected function isTraversable($target)
79			{
80			return (!preg_match('/[^a-z._]/', $target));
81			}
82
83			/**
84			* Return the data from the key.
85			*
86			* @param $arguments
87			* @param $target
88			* @return mixed
89			*/
90			protected function data($arguments, $target)
91			{
92			if (is_null($arguments)) {
93			return $target;
94			}
95
96			$arguments = is_array($arguments) ? $arguments : explode('.', $arguments);
97
98			foreach ($arguments as $segment) {
99			if (is_array($target)) {
100			if (!array_key_exists($segment, $target)) {
101			return value($target);
102			}
103
104			$target = $target[$segment];
105			} elseif ($target instanceof ArrayAccess) {
106			if (!isset($target[$segment])) {
107			return value($target);
108			}
109
110			$target = $target[$segment];
111			} elseif (is_object($target)) {
112
113			/**
114			* Check if the method exists first
115			* otherwise we might end up trying to
116			* access relational methods.
117			*
118			* Otherwise this is identical from
119			* data_get helper.
120			*/
121			if (method_exists($target, $segment) \|\| !isset($target->{$segment})) {
122			return value($target);
123			}
124
125			$target = $target->{$segment};
126			} else {
127			return value($target);
128			}
129			}
130
131			return $target;
132			}
133			}
134

anomalylabs / streams-platform

Pull Request — master (#210)

Evaluator::data() C

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like