Mailcode\ClassCache

Complexity

Total Complexity

3

Size/Duplication

Total Lines	29
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	findClassesInFolder()	0	9	1
A	createCache()	0	7	2

<?php
/**
 * @package Mailcode
 * @subpackage ClassCache
 */

declare(strict_types=1);

namespace Mailcode;

use AppUtils\ClassHelper\Repository\ClassRepositoryManager;
use AppUtils\FileHelper\FolderInfo;
use AppUtils\FileHelper_Exception;

/**
 * Utility that handles the dynamic class loading and cache.
 *
 * @package Mailcode
 * @subpackage ClassCache
 */
class ClassCache
{
    /**
     * @param string|FolderInfo $folder
     * @param bool $recursive
     * @param string|null $instanceOf
     * @return class-string[]

The doc comment class-string[] at position 0 could not be parsed: Unknown type name 'class-string' at position 0 in class-string[].

     * @throws FileHelper_Exception
     */
    public static function findClassesInFolder($folder, bool $recursive=false, ?string $instanceOf=null) : array
    {
        return self::createCache()
            ->findClassesInFolder(
                FolderInfo::factory($folder),
                $recursive,
                $instanceOf
            )
            ->getClasses();
    }

    private static ?ClassRepositoryManager $cache = null;

    private static function createCache() : ClassRepositoryManager
    {
        if(!isset(self::$cache)) {
            self::$cache = ClassRepositoryManager::create(Mailcode::getCacheFolder());
        }

        return self::$cache;

The expression return self::cache could return the type null which is incompatible with the type-hinted return AppUtils\ClassHelper\Repository\ClassRepositoryManager. Consider adding an additional type-check to rule them out.

    }
}