Driver::driverConnect() - Code Metrics - Inspection of "Merge pull request #623 from Geolim4/master" - PHPSocialNetwork/phpfastcache - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 4e7427...4a94bb )

by Georges

created 2018-06-24 12:51 UTC

Driver::driverConnect() B

↳ Parent: Driver

Complexity

Conditions	11
Paths	14

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	11
nc	14
nop	0
dl	0
loc	44
rs	7.3166
c	0
b	0
f	0

How to fix Complexity

<?php
/**
 *
 * This file is part of phpFastCache.
 *
 * @license MIT License (MIT)
 *
 * For full copyright and license information, please see the docs/CREDITS.txt file.
 *
 * @author Khoa Bui (khoaofgod)  <[email protected]> https://www.phpfastcache.com
 * @author Georges.L (Geolim4)  <[email protected]>
 *
 */
declare(strict_types=1);

namespace Phpfastcache\Drivers\Redis;

use Phpfastcache\Core\Pool\{
    DriverBaseTrait, ExtendedCacheItemPoolInterface
};
use Phpfastcache\Entities\DriverStatistic;
use Phpfastcache\Exceptions\{
    PhpfastcacheInvalidArgumentException, PhpfastcacheLogicException
};
use Psr\Cache\CacheItemInterface;
use Redis as RedisClient;

/**
 * Class Driver
 * @package phpFastCache\Drivers
 * @property Config $config Config object
 * @method Config getConfig() Return the config object
 */
class Driver implements ExtendedCacheItemPoolInterface
{
    use DriverBaseTrait;

    /**
     * @return bool
     */
    public function driverCheck(): bool
    {
        return \extension_loaded('Redis');
    }

    /**
     * @return bool
     * @throws PhpfastcacheLogicException
     */
    protected function driverConnect(): bool
    {
        if ($this->instance instanceof RedisClient) {
            throw new PhpfastcacheLogicException('Already connected to Redis server');
        }

        /**
         * In case of an user-provided
         * Redis client just return here
         */
        if ($this->getConfig()->getRedisClient() instanceof RedisClient) {

            /**
             * Unlike Predis, we can't test if we're are connected
             * or not, so let's just assume that we are
             */
            $this->instance = $this->getConfig()->getRedisClient();
            return true;
        }

        $this->instance = $this->instance ?: new RedisClient();

        /**
         * If path is provided we consider it as an UNIX Socket
         */
        if ($this->getConfig()->getPath()) {
            $isConnected = $this->instance->connect($this->getConfig()->getPath());
        } else {
            $isConnected = $this->instance->connect($this->getConfig()->getHost(), $this->getConfig()->getPort(), $this->getConfig()->getTimeout());
        }

        if (!$isConnected && $this->getConfig()->getPath()) {
            return false;
        }

        if (!$this->getConfig()->getPath()) {
            if ($this->getConfig()->getPassword() && !$this->instance->auth($this->getConfig()->getPassword())) {
                return false;
            }
        }

        if ($this->getConfig()->getDatabase() !== null) {
            $this->instance->select($this->getConfig()->getDatabase());
        }
        return true;
    }

    /**
     * @param \Psr\Cache\CacheItemInterface $item
     * @return null|array
     */
    protected function driverRead(CacheItemInterface $item)
    {
        $val = $this->instance->get($item->getKey());
        if ($val == false) {
            return null;
        }

        return $this->decode($val);
    }

    /**
     * @param \Psr\Cache\CacheItemInterface $item
     * @return mixed
     * @throws PhpfastcacheInvalidArgumentException
     */
    protected function driverWrite(CacheItemInterface $item): bool
    {
        /**
         * Check for Cross-Driver type confusion
         */
        if ($item instanceof Item) {
            $ttl = $item->getExpirationDate()->getTimestamp() - \time();

            /**
             * @see https://redis.io/commands/setex
             * @see https://redis.io/commands/expire
             */
            if ($ttl <= 0) {
                return $this->instance->expire($item->getKey(), 0);
            }

            return $this->instance->setex($item->getKey(), $ttl, $this->encode($this->driverPreWrap($item)));
        }

        throw new PhpfastcacheInvalidArgumentException('Cross-Driver type confusion detected');
    }

    /**
     * @param \Psr\Cache\CacheItemInterface $item
     * @return bool
     * @throws PhpfastcacheInvalidArgumentException
     */
    protected function driverDelete(CacheItemInterface $item): bool
    {
        /**
         * Check for Cross-Driver type confusion
         */
        if ($item instanceof Item) {
            return (bool)$this->instance->del($item->getKey());
        }

        throw new PhpfastcacheInvalidArgumentException('Cross-Driver type confusion detected');
    }

    /**
     * @return bool
     */
    protected function driverClear(): bool
    {
        return $this->instance->flushDB();
    }

    /********************
     *
     * PSR-6 Extended Methods
     *
     *******************/

    /**
     * @return DriverStatistic
     */
    public function getStats(): DriverStatistic
    {
        // used_memory
        $info = $this->instance->info();
        $date = (new \DateTime())->setTimestamp(\time() - $info['uptime_in_seconds']);

        return (new DriverStatistic())
            ->setData(\implode(', ', \array_keys($this->itemInstances)))
            ->setRawData($info)
            ->setSize((int)$info['used_memory'])
            ->setInfo(\sprintf("The Redis daemon v%s is up since %s.\n For more information see RawData. \n Driver size includes the memory allocation size.",
                $info['redis_version'], $date->format(\DATE_RFC2822)));
    }
}

1			<?php
2			/**
3			*
4			* This file is part of phpFastCache.
5			*
6			* @license MIT License (MIT)
7			*
8			* For full copyright and license information, please see the docs/CREDITS.txt file.
9			*
10			* @author Khoa Bui (khoaofgod) <[email protected]> https://www.phpfastcache.com
11			* @author Georges.L (Geolim4) <[email protected]>
12			*
13			*/
14			declare(strict_types=1);
15
16			namespace Phpfastcache\Drivers\Redis;
17
18			use Phpfastcache\Core\Pool\{
19			DriverBaseTrait, ExtendedCacheItemPoolInterface
20			};
21			use Phpfastcache\Entities\DriverStatistic;
22			use Phpfastcache\Exceptions\{
23			PhpfastcacheInvalidArgumentException, PhpfastcacheLogicException
24			};
25			use Psr\Cache\CacheItemInterface;
26			use Redis as RedisClient;
27
28			/**
29			* Class Driver
30			* @package phpFastCache\Drivers
31			* @property Config $config Config object
32			* @method Config getConfig() Return the config object
33			*/
34			class Driver implements ExtendedCacheItemPoolInterface
35			{
36			use DriverBaseTrait;
37
38			/**
39			* @return bool
40			*/
41			public function driverCheck(): bool
42			{
43			return \extension_loaded('Redis');
44			}
45
46			/**
47			* @return bool
48			* @throws PhpfastcacheLogicException
49			*/
50			protected function driverConnect(): bool
51			{
52			if ($this->instance instanceof RedisClient) {
53			throw new PhpfastcacheLogicException('Already connected to Redis server');
54			}
55
56			/**
57			* In case of an user-provided
58			* Redis client just return here
59			*/
60			if ($this->getConfig()->getRedisClient() instanceof RedisClient) {
			0 ignored issues – show introduced 2018-05-12 01:48 UTC by Report Bug Copy Issue Report `$this->getConfig()->getRedisClient()` is always a sub-type of `Redis`. Loading history...
61			/**
62			* Unlike Predis, we can't test if we're are connected
63			* or not, so let's just assume that we are
64			*/
65			$this->instance = $this->getConfig()->getRedisClient();
66			return true;
67			}
68
69			$this->instance = $this->instance ?: new RedisClient();
70
71			/**
72			* If path is provided we consider it as an UNIX Socket
73			*/
74			if ($this->getConfig()->getPath()) {
75			$isConnected = $this->instance->connect($this->getConfig()->getPath());
76			} else {
77			$isConnected = $this->instance->connect($this->getConfig()->getHost(), $this->getConfig()->getPort(), $this->getConfig()->getTimeout());
78			}
79
80			if (!$isConnected && $this->getConfig()->getPath()) {
81			return false;
82			}
83
84			if (!$this->getConfig()->getPath()) {
85			if ($this->getConfig()->getPassword() && !$this->instance->auth($this->getConfig()->getPassword())) {
86			return false;
87			}
88			}
89
90			if ($this->getConfig()->getDatabase() !== null) {
91			$this->instance->select($this->getConfig()->getDatabase());
92			}
93			return true;
94			}
95
96			/**
97			* @param \Psr\Cache\CacheItemInterface $item
98			* @return null\|array
99			*/
100			protected function driverRead(CacheItemInterface $item)
101			{
102			$val = $this->instance->get($item->getKey());
103			if ($val == false) {
104			return null;
105			}
106
107			return $this->decode($val);
108			}
109
110			/**
111			* @param \Psr\Cache\CacheItemInterface $item
112			* @return mixed
113			* @throws PhpfastcacheInvalidArgumentException
114			*/
115			protected function driverWrite(CacheItemInterface $item): bool
116			{
117			/**
118			* Check for Cross-Driver type confusion
119			*/
120			if ($item instanceof Item) {
121			$ttl = $item->getExpirationDate()->getTimestamp() - \time();
122
123			/**
124			* @see https://redis.io/commands/setex
125			* @see https://redis.io/commands/expire
126			*/
127			if ($ttl <= 0) {
128			return $this->instance->expire($item->getKey(), 0);
129			}
130
131			return $this->instance->setex($item->getKey(), $ttl, $this->encode($this->driverPreWrap($item)));
132			}
133
134			throw new PhpfastcacheInvalidArgumentException('Cross-Driver type confusion detected');
135			}
136
137			/**
138			* @param \Psr\Cache\CacheItemInterface $item
139			* @return bool
140			* @throws PhpfastcacheInvalidArgumentException
141			*/
142			protected function driverDelete(CacheItemInterface $item): bool
143			{
144			/**
145			* Check for Cross-Driver type confusion
146			*/
147			if ($item instanceof Item) {
148			return (bool)$this->instance->del($item->getKey());
149			}
150
151			throw new PhpfastcacheInvalidArgumentException('Cross-Driver type confusion detected');
152			}
153
154			/**
155			* @return bool
156			*/
157			protected function driverClear(): bool
158			{
159			return $this->instance->flushDB();
160			}
161
162			/********************
163			*
164			* PSR-6 Extended Methods
165			*
166			*******************/
167
168			/**
169			* @return DriverStatistic
170			*/
171			public function getStats(): DriverStatistic
172			{
173			// used_memory
174			$info = $this->instance->info();
175			$date = (new \DateTime())->setTimestamp(\time() - $info['uptime_in_seconds']);
176
177			return (new DriverStatistic())
178			->setData(\implode(', ', \array_keys($this->itemInstances)))
179			->setRawData($info)
180			->setSize((int)$info['used_memory'])
181			->setInfo(\sprintf("The Redis daemon v%s is up since %s.\n For more information see RawData. \n Driver size includes the memory allocation size.",
182			$info['redis_version'], $date->format(\DATE_RFC2822)));
183			}
184			}

PHPSocialNetwork / phpfastcache

Push — master ( 4e7427...4a94bb )

Driver::driverConnect() B

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like