Yiisoft\Composer\Config\Reader\ReaderFactory

Complexity

Total Complexity

6

Size/Duplication

Total Lines	44
Duplicated Lines	0 %

Test Coverage

Coverage

86.67%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	get()	0	12	2
A	findClass()	0	8	2
A	detectType()	0	7	2

<?php

declare(strict_types=1);

namespace Yiisoft\Composer\Config\Reader;

use Yiisoft\Composer\Config\Builder;
use Yiisoft\Composer\Config\Exception\UnsupportedFileTypeException;

/**
 * Reader - helper to load data from files of different types.
 */
class ReaderFactory
{
    /** @psalm-var array<string, ReaderInterface> */
    private static array $loaders = [];

    private static array $knownReaders = [
        'env' => EnvReader::class,
        'php' => PhpReader::class,
        'json' => JsonReader::class,
        'yaml' => YamlReader::class,
        'yml' => YamlReader::class,
    ];

    public static function get(Builder $builder, string $path): ReaderInterface
    {
        $class = static::findClass($path);

        $uniqid = $class . ':' . spl_object_hash($builder);
        if (empty(self::$loaders[$uniqid])) {
            /** @psalm-var ReaderInterface */
            self::$loaders[$uniqid] = new $class($builder);
        }

        /** @psalm-var ReaderInterface */
        return self::$loaders[$uniqid];
    }

    private static function detectType(string $path): string
    {
        if (strncmp(basename($path), '.env.', 5) === 0) {
            return 'env';
        }

        return pathinfo($path, PATHINFO_EXTENSION);

The expression return pathinfo($path, Yiisoft\Composer\Config\Reader\PATHINFO_EXTENSION) could return the type array which is incompatible with the type-hinted return string. Consider adding an additional type-check to rule them out.

    }

    private static function findClass(string $path): string
    {
        $type = static::detectType($path);
        if (!array_key_exists($type, static::$knownReaders)) {

Since $knownReaders is declared private, accessing it with static will lead to errors in possible sub-classes; you can either use self, or increase the visibility of $knownReaders to at least protected.

            throw new UnsupportedFileTypeException("Unsupported file type for \"$path\".");
        }

        return static::$knownReaders[$type];
    }
}