HDNET\CacheCheck\Service\Statistics\FileBackend

Complexity

Total Complexity

4

Size/Duplication

Total Lines	36
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Importance

Changes	5
Bugs	0	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	countTags()	0	4	1
A	getExpires()	0	13	3

<?php
/**
 * Stats service for file backend
 *
 * @package CacheCheck\Service\Statistics
 * @author  Tim Lochmüller
 */

namespace HDNET\CacheCheck\Service\Statistics;

use HDNET\CacheCheck\Domain\Model\Cache;
use TYPO3\CMS\Core\Cache\Backend\FileBackend as CoreFileBackend;
use TYPO3\CMS\Core\Utility\GeneralUtility;

/**
 * Stats service for file backend
 *
 * @author Tim Lochmüller
 */
class FileBackend extends SimpleFileBackend
{

    /**
     * Get the number of tags
     *
     * @param Cache $cache
     *
     * @return int
     */
    public function countTags(Cache $cache)
    {
        return 'Not implement yet';
    }

    /**
     * Returns the average left lifetime of the cache entries
     *
     * @param Cache $cache
     *
     * @return int|null
     */
    public function getExpires(Cache $cache)
    {
        $cacheFileNames = glob(GeneralUtility::getFileAbsFileName($this->getCacheDirectory($cache) . '*'));
        if (!$cacheFileNames) {

The expression $cacheFileNames of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

            return null;
        }
        $expireTimes = [];
        foreach ($cacheFileNames as $cacheFileName) {
            $index = (int)file_get_contents($cacheFileName, null, null, (filesize($cacheFileName) - CoreFileBackend::DATASIZE_DIGITS), CoreFileBackend::DATASIZE_DIGITS);
            $expireTimes[] = (int)file_get_contents($cacheFileName, null, null, $index, CoreFileBackend::EXPIRYTIME_LENGTH);
        }
        return (int)array_sum($expireTimes) / count($expireTimes);
    }
}