TusPhp\Cache\ApcuStore

Complexity

Total Complexity

12

Size/Duplication

Total Lines	68
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	getActualCacheKey()	0	9	2
A	keys()	0	4	1
A	get()	0	11	5
A	delete()	0	3	1
A	set()	0	11	3

<?php

namespace TusPhp\Cache;

use Carbon\Carbon;
use APCUIterator;

class ApcuStore extends AbstractCache
{

    /**
     * {@inheritDoc}
     */
    public function get(string $key, bool $withExpired = false)
    {
        $contents = apcu_fetch($this->getActualCacheKey($key)) ?: null;

        if ($withExpired) {
            return $contents ?: null;
        }

        $isExpired = Carbon::parse($contents['expires_at'])->lt(Carbon::now());

        return $isExpired ? null : $contents;
    }

    /**
     * {@inheritDoc}
     */
    public function set(string $key, $value)
    {
        $contents = $this->get($key) ?: [];

        if (\is_array($value)) {
            $contents = $value + $contents;
        } else {
            array_push($contents, $value);
        }

        return apcu_store($this->getActualCacheKey($key), $contents, $this->getTtl());
    }

    /**
     * {@inheritDoc}
     */
    public function delete(string $key): bool
    {
        return apcu_delete($this->getActualCacheKey($key));

The expression return apcu_delete($this->getActualCacheKey($key)) could return the type string[] which is incompatible with the type-hinted return boolean. Consider adding an additional type-check to rule them out.

    }

    /**
     * {@inheritDoc}
     */
    public function keys(): array
    {
        $iterator = new APCUIterator('/^' . preg_quote($this->getPrefix()) . '.*$/', APC_ITER_KEY);
        return array_column(iterator_to_array($iterator, false), 'key');
    }

    /**
     * Get actual cache key with prefix.
     *
     * @param string $key
     *
     * @return string
     */
    protected function getActualCacheKey(string $key): string
    {
        $prefix = $this->getPrefix();

        if (false === strpos($key, $prefix)) {
            $key = $prefix . $key;
        }

        return $key;
    }
}