ClassDiscovery::instantiateClass() - Code Metrics - Inspection of "simplify discovery code (#118)" - php-http/discovery - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( e97560...d0b903 )

by Tobias

created 2018-12-29 09:30 UTC

ClassDiscovery::instantiateClass() A

↳ Parent: ClassDiscovery

Complexity

Conditions	4
Paths	7

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	3
CRAP Score	7.9062

Importance

Changes

Metric	Value
dl	0
loc	16
ccs	3
cts	8
cp	0.375
rs	9.7333
c	0
b	0
f	0
cc	4
nc	7
nop	1
crap	7.9062

<?php

namespace Http\Discovery;

use Http\Discovery\Exception\ClassInstantiationFailedException;
use Http\Discovery\Exception\DiscoveryFailedException;
use Http\Discovery\Exception\StrategyUnavailableException;

/**
 * Registry that based find results on class existence.
 *
 * @author David de Boer <[email protected]>
 * @author Márk Sági-Kazár <[email protected]>
 * @author Tobias Nyholm <[email protected]>
 */
abstract class ClassDiscovery
{
    /**
     * A list of strategies to find classes.
     *
     * @var array
     */
    private static $strategies = [
        Strategy\PuliBetaStrategy::class,
        Strategy\CommonClassesStrategy::class,
    ];

    /**
     * Discovery cache to make the second time we use discovery faster.
     *
     * @var array
     */
    private static $cache = [];

    /**
     * Finds a class.
     *
     * @param string $type
     *
     * @return string|\Closure
     *
     * @throws DiscoveryFailedException
     */
    protected static function findOneByType($type)
    {
        // Look in the cache
        if (null !== ($class = self::getFromCache($type))) {
            return $class;
        }

        $exceptions = [];
        foreach (self::$strategies as $strategy) {
            try {
                $candidates = call_user_func($strategy.'::getCandidates', $type);
            } catch (StrategyUnavailableException $e) {
                $exceptions[] = $e;

                continue;
            }

            foreach ($candidates as $candidate) {
                if (isset($candidate['condition'])) {
                    if (!self::evaluateCondition($candidate['condition'])) {
                        continue;
                    }
                }

                // save the result for later use
                self::storeInCache($type, $candidate);

                return $candidate['class'];
            }
        }

        throw DiscoveryFailedException::create($exceptions);
    }

    /**
     * Get a value from cache.
     *
     * @param string $type
     *
     * @return string|null
     */
    private static function getFromCache($type)
    {
        if (!isset(self::$cache[$type])) {
            return;
        }

        $candidate = self::$cache[$type];
        if (isset($candidate['condition'])) {
            if (!self::evaluateCondition($candidate['condition'])) {
                return;
            }
        }

        return $candidate['class'];
    }

    /**
     * Store a value in cache.
     *
     * @param string $type
     * @param string $class
     */
    private static function storeInCache($type, $class)
    {
        self::$cache[$type] = $class;
    }

    /**
     * Set new strategies and clear the cache.
     *
     * @param array $strategies string array of fully qualified class name to a DiscoveryStrategy
     */
    public static function setStrategies(array $strategies)
    {
        self::$strategies = $strategies;
        self::clearCache();
    }

    /**
     * Append a strategy at the end of the strategy queue.
     *
     * @param string $strategy Fully qualified class name to a DiscoveryStrategy
     */
    public static function appendStrategy($strategy)
    {
        self::$strategies[] = $strategy;
        self::clearCache();
    }

    /**
     * Prepend a strategy at the beginning of the strategy queue.
     *
     * @param string $strategy Fully qualified class name to a DiscoveryStrategy
     */
    public static function prependStrategy($strategy)
    {
        array_unshift(self::$strategies, $strategy);
        self::clearCache();
    }

    /**
     * Clear the cache.
     */
    public static function clearCache()
    {
        self::$cache = [];
    }

    /**
     * Evaluates conditions to boolean.
     *
     * @param mixed $condition
     *
     * @return bool
     */
    protected static function evaluateCondition($condition)
    {
        if (is_string($condition)) {
            // Should be extended for functions, extensions???
            return self::safeClassExists($condition);
        }
        if (is_callable($condition)) {
            return (bool) $condition();
        }
        if (is_bool($condition)) {
            return $condition;
        }
        if (is_array($condition)) {
            foreach ($condition as $c) {
                if (false === static::evaluateCondition($c)) {
                    // Immediately stop execution if the condition is false
                    return false;
                }
            }

            return true;
        }

        return false;
    }

    /**
     * Get an instance of the $class.
     *
     * @param string|\Closure $class A FQCN of a class or a closure that instantiate the class.
     *
     * @return object
     *
     * @throws ClassInstantiationFailedException
     */
    protected static function instantiateClass($class)
    {
        try {
            if (is_string($class)) {
                return new $class();
            }

            if (is_callable($class)) {
                return $class();
            }
        } catch (\Exception $e) {
            throw new ClassInstantiationFailedException('Unexpected exception when instantiating class.', 0, $e);
        }

        throw new ClassInstantiationFailedException('Could not instantiate class because parameter is neither a callable nor a string');
    }

    /**
     * We want to do a "safe" version of PHP's "class_exists" because Magento has a bug
     * (or they call it a "feature"). Magento is throwing an exception if you do class_exists()
     * on a class that ends with "Factory" and if that file does not exits.
     *
     * This function will catch all potential exceptions and make sure it returns a boolean.
     *
     * @param string $class
     * @param bool   $autoload
/**
 * @param array $germany
 * @param array $island
 * @param array $italy
 */
function finale($germany, $island) {
    return "2:1";
}
     *
     * @return bool
     */
    public static function safeClassExists($class)
    {
        try {
            return class_exists($class);
        } catch (\Exception $e) {
            return false;
        }
    }
}


1		<?php
2
3		namespace Http\Discovery;
4
5		use Http\Discovery\Exception\ClassInstantiationFailedException;
6		use Http\Discovery\Exception\DiscoveryFailedException;
7		use Http\Discovery\Exception\StrategyUnavailableException;
8
9		/**
10		* Registry that based find results on class existence.
11		*
12		* @author David de Boer <[email protected]>
13		* @author Márk Sági-Kazár <[email protected]>
14		* @author Tobias Nyholm <[email protected]>
15		*/
16		abstract class ClassDiscovery
17		{
18		/**
19		* A list of strategies to find classes.
20		*
21		* @var array
22		*/
23		private static $strategies = [
24		Strategy\PuliBetaStrategy::class,
25		Strategy\CommonClassesStrategy::class,
26		];
27
28		/**
29		* Discovery cache to make the second time we use discovery faster.
30		*
31		* @var array
32		*/
33		private static $cache = [];
34
35		/**
36		* Finds a class.
37		*
38		* @param string $type
39		*
40		* @return string\|\Closure
41		*
42		* @throws DiscoveryFailedException
43		*/
44	15	protected static function findOneByType($type)
45		{
46		// Look in the cache
47	15	if (null !== ($class = self::getFromCache($type))) {
48		return $class;
49		}
50
51	15	$exceptions = [];
52	15	foreach (self::$strategies as $strategy) {
53		try {
54	15	$candidates = call_user_func($strategy.'::getCandidates', $type);
55	15	} catch (StrategyUnavailableException $e) {
56		$exceptions[] = $e;
57
58		continue;
59		}
60
61	15	foreach ($candidates as $candidate) {
62	9	if (isset($candidate['condition'])) {
63	7	if (!self::evaluateCondition($candidate['condition'])) {
64	1	continue;
65		}
66	7	}
67
68		// save the result for later use
69	9	self::storeInCache($type, $candidate);
70
71	9	return $candidate['class'];
72	7	}
73	7	}
74
75	6	throw DiscoveryFailedException::create($exceptions);
76		}
77
78		/**
79		* Get a value from cache.
80		*
81		* @param string $type
82		*
83		* @return string\|null
84		*/
85	15	private static function getFromCache($type)
86		{
87	15	if (!isset(self::$cache[$type])) {
88	15	return;
89		}
90
91		$candidate = self::$cache[$type];
92		if (isset($candidate['condition'])) {
93		if (!self::evaluateCondition($candidate['condition'])) {
94		return;
95		}
96		}
97
98		return $candidate['class'];
99		}
100
101		/**
102		* Store a value in cache.
103		*
104		* @param string $type
105		* @param string $class
106		*/
107	9	private static function storeInCache($type, $class)
108		{
109	9	self::$cache[$type] = $class;
110	9	}
111
112		/**
113		* Set new strategies and clear the cache.
114		*
115		* @param array $strategies string array of fully qualified class name to a DiscoveryStrategy
116		*/
117	27	public static function setStrategies(array $strategies)
118		{
119	27	self::$strategies = $strategies;
120	27	self::clearCache();
121	27	}
122
123		/**
124		* Append a strategy at the end of the strategy queue.
125		*
126		* @param string $strategy Fully qualified class name to a DiscoveryStrategy
127		*/
128	1	public static function appendStrategy($strategy)
129		{
130	1	self::$strategies[] = $strategy;
131	1	self::clearCache();
132	1	}
133
134		/**
135		* Prepend a strategy at the beginning of the strategy queue.
136		*
137		* @param string $strategy Fully qualified class name to a DiscoveryStrategy
138		*/
139	1	public static function prependStrategy($strategy)
140		{
141	1	array_unshift(self::$strategies, $strategy);
142	1	self::clearCache();
143	1	}
144
145		/**
146		* Clear the cache.
147		*/
148	27	public static function clearCache()
149		{
150	27	self::$cache = [];
151	27	}
152
153		/**
154		* Evaluates conditions to boolean.
155		*
156		* @param mixed $condition
157		*
158		* @return bool
159		*/
160	7	protected static function evaluateCondition($condition)
161		{
162	7	if (is_string($condition)) {
163		// Should be extended for functions, extensions???
164		return self::safeClassExists($condition);
165		}
166	7	if (is_callable($condition)) {
167		return (bool) $condition();
168		}
169	7	if (is_bool($condition)) {
170	7	return $condition;
171		}
172	1	if (is_array($condition)) {
173	1	foreach ($condition as $c) {
174	1	if (false === static::evaluateCondition($c)) {
175		// Immediately stop execution if the condition is false
176	1	return false;
177		}
178	1	}
179
180		return true;
181		}
182
183		return false;
184		}
185
186		/**
187		* Get an instance of the $class.
188		*
189		* @param string\|\Closure $class A FQCN of a class or a closure that instantiate the class.
190		*
191		* @return object
192		*
193		* @throws ClassInstantiationFailedException
194		*/
195	5	protected static function instantiateClass($class)
196		{
197		try {
198	5	if (is_string($class)) {
199	5	return new $class();
200		}
201
202		if (is_callable($class)) {
203		return $class();
204		}
205		} catch (\Exception $e) {
206		throw new ClassInstantiationFailedException('Unexpected exception when instantiating class.', 0, $e);
207		}
208
209		throw new ClassInstantiationFailedException('Could not instantiate class because parameter is neither a callable nor a string');
210		}
211
212		/**
213		* We want to do a "safe" version of PHP's "class_exists" because Magento has a bug
214		* (or they call it a "feature"). Magento is throwing an exception if you do class_exists()
215		* on a class that ends with "Factory" and if that file does not exits.
216		*
217		* This function will catch all potential exceptions and make sure it returns a boolean.
218		*
219		* @param string $class
220		* @param bool $autoload
		0 ignored issues – show Bug introduced 2018-08-17 12:15 UTC by Report Bug Copy Issue Report There is no parameter named `$autoload`. Was it maybe removed? This check looks for PHPDoc comments describing methods or function parameters that do not exist on the corresponding method or function. Consider the following example. The parameter `$italy` is not defined by the method `finale(...)`. /** * @param array $germany * @param array $island * @param array $italy */ function finale($germany, $island) { return "2:1"; } The most likely cause is that the parameter was removed, but the annotation was not. Loading history...
221		*
222		* @return bool
223		*/
224		public static function safeClassExists($class)
225		{
226		try {
227		return class_exists($class);
228		} catch (\Exception $e) {
229		return false;
230		}
231		}
232		}
233

php-http / discovery

Push — master ( e97560...d0b903 )

ClassDiscovery::instantiateClass() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like