FlintRedisCacheFactory::newCacheInstance() - Code Metrics - Inspection of "renamed test" - Suven/flintRedis - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( c1c437...916c2d )

by Sven

created 2017-04-24 22:47 UTC

FlintRedisCacheFactory::newCacheInstance() A

↳ Parent: FlintRedisCacheFactory

Complexity

Conditions	2
Paths	2

Size

Total Lines	10
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	5
CRAP Score	2.0185

Importance

Changes

Metric	Value
dl	0
loc	10
ccs	5
cts	6
cp	0.8333
rs	9.4285
c	0
b	0
f	0
cc	2
eloc	6
nc	2
nop	1
crap	2.0185

<?php
namespace Suven\FlintRedis;

abstract class FlintRedisCacheFactory
{
    const STRATEGY_REDIS = 1;
    const STRATEGY_FLINTSTONE = 2;

    private static $cachedInstances = [];

    /**
     * @var number Either FlintRedisCacheFactory::STRATEGY_REDIS or FlintRedisCacheFactory:: STRATEGY_FLINTSTONE
     */
    public static $strategy = self::STRATEGY_REDIS;

    /**
     * @var array Options you want to pass to predis/flintstone
     */
    public static $options = [];

    /**
     * Retrieves a new or previously created cache-instance for the provided
     * parameters.
     *
     * @param  string  $realm    Your collections name
     * @return FlintRedisCache
     */
    public static function create($realm, $strategy = false, $options = false)
    {
        $instance = self::retrieveOldInstance($realm);

        if ($strategy) {
            self::$strategy = $strategy;

        }

        if ($options) {
            self::$options = $options;

        }

        if (!$instance) {
            $instance = self::newCacheInstance($realm);
        }

        return $instance;
    }

    private static function retrieveOldInstance($realm)
    {
        return isset(self::$cachedInstances[$realm]) ? self::$cachedInstances[$realm] : false;
    }

    private static function newCacheInstance($realm)
    {
        if (self::$strategy === self::STRATEGY_REDIS) {
            self::$cachedInstances[$realm] = new FlintRedisCacheRedis($realm, self::$options);
        } else {
            self::$cachedInstances[$realm] = new FlintRedisCacheFlintstone($realm, self::$options);
        }

        return self::$cachedInstances[$realm];
    }
}


1		<?php
2		namespace Suven\FlintRedis;
3
4		abstract class FlintRedisCacheFactory
5		{
6		const STRATEGY_REDIS = 1;
7		const STRATEGY_FLINTSTONE = 2;
8
9		private static $cachedInstances = [];
10
11		/**
12		* @var number Either FlintRedisCacheFactory::STRATEGY_REDIS or FlintRedisCacheFactory:: STRATEGY_FLINTSTONE
13		*/
14		public static $strategy = self::STRATEGY_REDIS;
15
16		/**
17		* @var array Options you want to pass to predis/flintstone
18		*/
19		public static $options = [];
20
21		/**
22		* Retrieves a new or previously created cache-instance for the provided
23		* parameters.
24		*
25		* @param string $realm Your collections name
26		* @return FlintRedisCache
27		*/
28	6	public static function create($realm, $strategy = false, $options = false)
29		{
30	6	$instance = self::retrieveOldInstance($realm);
31
32	6	if ($strategy) {
33		self::$strategy = $strategy;
		0 ignored issues – show Documentation Bug introduced 2017-04-24 23:10 UTC by Report Bug Copy Issue Report It seems like `$strategy` of type `boolean` is incompatible with the declared type `integer\|double` of property `$strategy`. 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...
34		}
35
36	6	if ($options) {
37		self::$options = $options;
		0 ignored issues – show Documentation Bug introduced 2017-04-24 23:10 UTC by Report Bug Copy Issue Report It seems like `$options` of type `boolean` is incompatible with the declared type `array` of property `$options`. 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...
38		}
39
40	6	if (!$instance) {
41	6	$instance = self::newCacheInstance($realm);
42	4	}
43
44	6	return $instance;
45		}
46
47	6	private static function retrieveOldInstance($realm)
48		{
49	6	return isset(self::$cachedInstances[$realm]) ? self::$cachedInstances[$realm] : false;
50		}
51
52	6	private static function newCacheInstance($realm)
53		{
54	6	if (self::$strategy === self::STRATEGY_REDIS) {
55	6	self::$cachedInstances[$realm] = new FlintRedisCacheRedis($realm, self::$options);
56	4	} else {
57		self::$cachedInstances[$realm] = new FlintRedisCacheFlintstone($realm, self::$options);
58		}
59
60	6	return self::$cachedInstances[$realm];
61		}
62		}
63

Suven / flintRedis

Push — master ( c1c437...916c2d )

FlintRedisCacheFactory::newCacheInstance() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like