WebStream\Cache\Driver\Apcu

Complexity

Total Complexity

14

Size/Duplication

Total Lines	104
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	add()	0	12	3
A	get()	0	17	3
A	delete()	0	9	2
A	clear()	0	12	3
A	isAvailableCacheLibrary()	0	10	2

<?php
namespace WebStream\Cache\Driver;

use WebStream\DI\Injector;
use WebStream\Module\Container;

/**
 * Apcu
 * TTLがリクエストによりずれる問題はAPCuの仕様なので対応しない
 * @author Ryuichi Tanaka
 * @since 2015/07/05
 * @version 0.7
 */
class Apcu implements ICache
{
    use injector;

    /**
     * @var bool キャッシュ使用可能フラグ
     */
    private $isAvairable;

    /**
     * @var string キャッシュ接頭辞
     */
    private $cachePrefix;

    /**
     * constructor
     */
    public function __construct(Container $container)
    {
        $this->isAvairable = extension_loaded('apcu');
        $this->cachePrefix = $container->cachePrefix;

The property cachePrefix does not exist on object<WebStream\Module\Container>. Since you implemented __set, maybe consider adding a @property annotation.

    }

    /**
     * {@inheritdoc}
     */
    public function add($key, $value, $ttl = 0, $overrite = false): bool
    {
        if (!$this->isAvailableCacheLibrary()) {
            return false;
        }
        $key = $this->cachePrefix . $key;

        $result = $overrite ? apcu_store($key, $value, $ttl) : apcu_add($key, $value, $ttl);
        $this->logger->info("Execute cache save: " . $key);

The property logger does not exist on object<WebStream\Cache\Driver\Apcu>. Since you implemented __get, maybe consider adding a @property annotation.

        return $result;
    }

    /**
     * {@inheritdoc}
     */
    public function get($key)
    {
        if (!$this->isAvailableCacheLibrary()) {
            return null;
        }
        $key = $this->cachePrefix . $key;
        $value = apcu_fetch($key, $isSuccess);

The variable $isSuccess does not exist. Did you forget to declare it?

        if ($isSuccess) {
            $this->logger->info("Execute cache read: " . $key);

The property logger does not exist on object<WebStream\Cache\Driver\Apcu>. Since you implemented __get, maybe consider adding a @property annotation.

        } else {
            $this->logger->warn("Failed to read cache: " . $key);

The property logger does not exist on object<WebStream\Cache\Driver\Apcu>. Since you implemented __get, maybe consider adding a @property annotation.

            $value = null;
        }

        return $value;
    }

    /**
     * {@inheritdoc}
     */
    public function delete($key): bool
    {
        if (!$this->isAvailableCacheLibrary()) {
            return false;
        }
        $key = $this->cachePrefix . $key;

        return apcu_delete($key);
    }

    /**
     * {@inheritdoc}
     */
    public function clear(): bool
    {
        if (!$this->isAvailableCacheLibrary()) {
            return false;
        }

        if (class_exists('\APCUIterator')) {
            return apcu_delete(new \APCUIterator('/^' . $this->cachePrefix . '/', APC_ITER_KEY));
        } else {
            return apcu_clear_cache();
        }
    }

    /**
     * キャッシュライブラリが使用可能か検査する
     * @return bool 使用可能でtrue
     */
    private function isAvailableCacheLibrary(): bool
    {
        if ($this->isAvairable) {
            return true;
        }

        $this->logger->warn("APCu cache library is unavailable.");

The property logger does not exist on object<WebStream\Cache\Driver\Apcu>. Since you implemented __get, maybe consider adding a @property annotation.

        return false;
    }
}