WebStream\Cache\Driver\Apcu

Complexity

Total Complexity

17

Size/Duplication

Total Lines	111
Duplicated Lines	13.51 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	add()	0	12	3
A	get()	0	17	3
A	delete()	15	15	3
B	clear()	0	19	5
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 Container キャッシュ依存コンテナ
     */
    private $cacheContainer;

    /**
     * {@inheritdoc}
     */
    public function __construct(Container $cacheContainer)
    {
        $this->cacheContainer = $cacheContainer;
    }

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

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

        $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->cacheContainer->cachePrefix . $key;

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

        $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

This method seems to be duplicated in your project.

    {
        if (!$this->isAvailableCacheLibrary()) {
            return false;
        }
        $key = $this->cacheContainer->cachePrefix . $key;

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

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

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

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

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

            return false;
        }
    }

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

        if (class_exists('\APCUIterator')) {
            if (apcu_delete(new \APCUIterator('/^' . $this->cacheContainer->cachePrefix . '/', APC_ITER_KEY))) {

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

                $this->logger->info("Execute all cache cleared: " . $key . "*");

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

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

                return true;
            }
        } elseif (apcu_clear_cache()) {
            $this->logger->info("Execute all cache cleared: " . $key . "*");

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

            return true;
        }

        $this->logger->warn("Failed to clear all cache: " . $key . "*");

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

        return false;
    }

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

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

            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;
    }
}