CConverter\Cache\CacheFile

Complexity

Total Complexity

10

Size/Duplication

Total Lines	98
Duplicated Lines	0 %

Importance

Changes	3
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	getCache()	0	5	1
A	isCacheDir()	0	5	2
A	isCurrent()	0	13	2
A	setNewCacheFile()	0	11	2
A	isFile()	0	11	2
A	getFileName()	0	8	1

<?php

namespace CConverter\Cache;

class CacheFile
{
    /**
     * check if is cache dir
     *
     * @return bool
     */
    public static function isCacheDir()
    {
        if(!is_dir(__DIR__ .'/cache/')) mkdir(__DIR__ ."/cache/", 0700);

        return true;
    }

    /**
     * check if cache file exists
     *
     * @return bool
     */
    public static function isFile()
    {
        $files = glob(__DIR__ .'/cache/*.txt');
        $count = count($files);

It seems like $files can also be of type false; however, parameter $var of count() does only seem to accept Countable|array, maybe add an additional type check?

        if($count>0){
            $output = true;
        }else{
            $output = false;
        }

        return $output;
    }

    /**
     *  get name of cache file
     *
     * @return string
     */
    public static function getFileName()
    {
        $files = glob(__DIR__ .'/cache/*.txt');
        $file = $files[0];
        $fileName = explode("/cache/", $file);
        $fileName = $fileName[1];

        return $fileName;
    }

    /**
     *  check if cache file is needed to update
     *
     * @return bool
     */
    public static function isCurrent($file)
    {
        $getSourceName = explode(".", $file);
        $getSourceName = $getSourceName[0];

        $current_time = time();
        if($current_time>$getSourceName){
                $output = false;
        }else{
                $output = true;
        }

        return $output;
    }

    /**
     *  create new cache file
     *
     * @param  string|bool   $file
     * @param  array|string  $data
     * @param  int           $time
     * @return bool
     */
    public static function setNewCacheFile($file, $data, $time)
    {
        if($file) unlink(__DIR__ .'/cache/'.$file);

Are you sure $file of type string|true can be used in concatenation?

        $current_time = time();
        $new_time = $current_time + $time*60;

        $file_name = $new_time.".txt";
        file_put_contents(__DIR__ .'/cache/'.$file_name, $data);

        return true;
    }

    /**
     *  get cache data
     *
     * @return string
     */
    public static function getCache($file)
    {
        $data = file_get_contents(__DIR__ .'/cache/'.$file);

        return $data;
    }
}