BaseManager::has() - Code Metrics - jaredchu/Simple-Cache - Measure and Improve Code Quality continuously with Scrutinizer

BaseManager::has() A
last analyzed 2017-10-11 09:43 UTC

↳ Parent: BaseManager

Complexity

Conditions	2
Paths	2

Size

Total Lines	9
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	9
rs	9.6666
c	0
b	0
f	0
cc	2
eloc	5
nc	2
nop	1

<?php
/**
 * Created by PhpStorm.
 * User: jaredchu
 * Date: 04/08/2017
 * Time: 11:36
 */

namespace JC\Cache;

/**
 * Class CacheManager
 * @package JC
 *
 * Manage the list of caching file and object
 * Save in $cFilename
 */
abstract class BaseManager
{
    /**
     * @var string
     */
    protected static $cFileName;

    /**
     * @param $cFileName
     */
    public static function setCFileName($cFileName)
    {
        static::$cFileName = $cFileName;
    }

    /**
     * @return string
     */
    public static function getCFileName()
    {
        return static::$cFileName;
    }

    /**
     * @return string
     *
     * Return absolute file path which store cache list
     */
    public static function getCFilePath()
    {
        return sys_get_temp_dir() . '/' . static::getCFileName();
    }

    /**
     * Unique string for each server
     * @param string $salt
     * @return string
     */
    public static function getUniqueString($salt = '')
// Bad
class Router
{
    public function generate($path)
    {
        return $_SERVER['HOST'].$path;
    }
}

// Better
class Router
{
    private $host;

    public function __construct($host)
    {
        $this->host = $host;
    }

    public function generate($path)
    {
        return $this->host.$path;
    }
}

class Controller
{
    public function myAction(Request $request)
    {
        // Instead of
        $page = isset($_GET['page']) ? intval($_GET['page']) : 1;

        // Better (assuming you use the Symfony2 request)
        $page = $request->query->get('page', 1);
    }
}
    {
        $uniqueString = isset($_SERVER['SERVER_SIGNATURE']) ? $_SERVER['SERVER_SIGNATURE'] . __DIR__ : __DIR__;
        return md5($uniqueString . $salt);
    }

    /**
     * @param $key
     * @return bool|string
     *
     * Return absolute file path of caching object
     */
    public static function get($key)
    {
        $cacheList = static::getCacheList();
        if (isset($cacheList[$key])) {
            if (($cacheList[$key][0] == 0 || $cacheList[$key][0] >= time())) {
                return $cacheList[$key][1];
            } else {
                unlink($cacheList[$key][1]);
                static::remove($key);
            }
        }

        return false;
    }

    /**
     * @param $key
     * @param $filePath
     * @return bool
     *
     * Save $key and $filePath in cacheList
     */
    public static function set($key, $filePath, $ttl)
    {
        $cacheList = static::getCacheList();

        if ($ttl != 0) {
            $ttl += time();
        }
        $cacheList[$key] = [$ttl, $filePath];

        return static::setCacheList($cacheList);
    }

    /**
     * @param $key
     * @return bool
     *
     * Remove caching object file and remove from cache list
     */
    public static function remove($key)
    {
        $cacheList = static::getCacheList();
        unset($cacheList[$key]);

        return static::setCacheList($cacheList);
    }

    /**
     * @param $key
     * @return bool
     */
    public static function has($key)
    {
        $cacheList = static::getCacheList();
        if (is_array($cacheList)) {
            return array_key_exists($key, $cacheList);
        }

        return false;
    }

    /**
     * @return array
     *
     * Return cache list in array
     */
    protected static function getCacheList()
    {
        if (file_exists(static::getCFilePath())) {
            return static::decode(file_get_contents(static::getCFilePath()));
        }

        return array();
    }

    /**
     * @param $cacheList
     * @return bool
     */
    protected static function setCacheList($cacheList)
    {
        return (bool)file_put_contents(static::getCFilePath(), static::encode($cacheList));
    }

    /**
     * @param $array
     * @return string
     */
    protected static function encode($array)
    {
        return json_encode($array);
    }

    /**
     * @param $string
     * @return array
     */
    protected static function decode($string)
    {
        return json_decode($string, true);
    }
}

1			<?php
2			/**
3			* Created by PhpStorm.
4			* User: jaredchu
5			* Date: 04/08/2017
6			* Time: 11:36
7			*/
8
9			namespace JC\Cache;
10
11			/**
12			* Class CacheManager
13			* @package JC
14			*
15			* Manage the list of caching file and object
16			* Save in $cFilename
17			*/
18			abstract class BaseManager
19			{
20			/**
21			* @var string
22			*/
23			protected static $cFileName;
24
25			/**
26			* @param $cFileName
27			*/
28			public static function setCFileName($cFileName)
29			{
30			static::$cFileName = $cFileName;
31			}
32
33			/**
34			* @return string
35			*/
36			public static function getCFileName()
37			{
38			return static::$cFileName;
39			}
40
41			/**
42			* @return string
43			*
44			* Return absolute file path which store cache list
45			*/
46			public static function getCFilePath()
47			{
48			return sys_get_temp_dir() . '/' . static::getCFileName();
49			}
50
51			/**
52			* Unique string for each server
53			* @param string $salt
54			* @return string
55			*/
56			public static function getUniqueString($salt = '')
			0 ignored issues – show Coding Style introduced 2017-08-10 10:20 UTC by Report Bug Copy Issue Report `getUniqueString` uses the super-global variable `$_SERVER` which is generally not recommended. Instead of super-globals, we recommend to explicitly inject the dependencies of your class. This makes your code less dependent on global state and it becomes generally more testable: // Bad class Router { public function generate($path) { return $_SERVER['HOST'].$path; } } // Better class Router { private $host; public function __construct($host) { $this->host = $host; } public function generate($path) { return $this->host.$path; } } class Controller { public function myAction(Request $request) { // Instead of $page = isset($_GET['page']) ? intval($_GET['page']) : 1; // Better (assuming you use the Symfony2 request) $page = $request->query->get('page', 1); } } Loading history...
57			{
58			$uniqueString = isset($_SERVER['SERVER_SIGNATURE']) ? $_SERVER['SERVER_SIGNATURE'] . __DIR__ : __DIR__;
59			return md5($uniqueString . $salt);
60			}
61
62			/**
63			* @param $key
64			* @return bool\|string
65			*
66			* Return absolute file path of caching object
67			*/
68			public static function get($key)
69			{
70			$cacheList = static::getCacheList();
71			if (isset($cacheList[$key])) {
72			if (($cacheList[$key][0] == 0 \|\| $cacheList[$key][0] >= time())) {
73			return $cacheList[$key][1];
74			} else {
75			unlink($cacheList[$key][1]);
76			static::remove($key);
77			}
78			}
79
80			return false;
81			}
82
83			/**
84			* @param $key
85			* @param $filePath
86			* @return bool
87			*
88			* Save $key and $filePath in cacheList
89			*/
90			public static function set($key, $filePath, $ttl)
91			{
92			$cacheList = static::getCacheList();
93
94			if ($ttl != 0) {
95			$ttl += time();
96			}
97			$cacheList[$key] = [$ttl, $filePath];
98
99			return static::setCacheList($cacheList);
100			}
101
102			/**
103			* @param $key
104			* @return bool
105			*
106			* Remove caching object file and remove from cache list
107			*/
108			public static function remove($key)
109			{
110			$cacheList = static::getCacheList();
111			unset($cacheList[$key]);
112
113			return static::setCacheList($cacheList);
114			}
115
116			/**
117			* @param $key
118			* @return bool
119			*/
120			public static function has($key)
121			{
122			$cacheList = static::getCacheList();
123			if (is_array($cacheList)) {
124			return array_key_exists($key, $cacheList);
125			}
126
127			return false;
128			}
129
130			/**
131			* @return array
132			*
133			* Return cache list in array
134			*/
135			protected static function getCacheList()
136			{
137			if (file_exists(static::getCFilePath())) {
138			return static::decode(file_get_contents(static::getCFilePath()));
139			}
140
141			return array();
142			}
143
144			/**
145			* @param $cacheList
146			* @return bool
147			*/
148			protected static function setCacheList($cacheList)
149			{
150			return (bool)file_put_contents(static::getCFilePath(), static::encode($cacheList));
151			}
152
153			/**
154			* @param $array
155			* @return string
156			*/
157			protected static function encode($array)
158			{
159			return json_encode($array);
160			}
161
162			/**
163			* @param $string
164			* @return array
165			*/
166			protected static function decode($string)
167			{
168			return json_decode($string, true);
169			}
170			}

jaredchu / Simple-Cache

BaseManager::has() A last analyzed 2017-10-11 09:43 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

BaseManager::has() A
last analyzed 2017-10-11 09:43 UTC