FlintRedisCacheFactory::create() - 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::create() A

↳ Parent: FlintRedisCacheFactory

Complexity

Conditions	4
Paths	8

Size

Total Lines	18
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	8
CRAP Score	4.5923

Importance

Changes

Metric	Value
dl	0
loc	18
ccs	8
cts	12
cp	0.6667
rs	9.2
c	0
b	0
f	0
cc	4
eloc	9
nc	8
nop	3
crap	4.5923

<?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::create() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like