Micro\Library\DTO\Preparation\CollectionPreparation

Complexity

Total Complexity

7

Size/Duplication

Total Lines	44
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	2
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	2	1
A	process()	0	16	4
A	createClassList()	0	8	2

<?php

declare(strict_types=1);

/*
 *  This file is part of the Micro framework package.
 *
 *  (c) Stanislau Komar <kost@micro-php.net>
 *
 *  For the full copyright and license information, please view the LICENSE
 *  file that was distributed with this source code.
 */

namespace Micro\Library\DTO\Preparation;

use Micro\Library\DTO\ClassDef\ClassDefinition;
use Micro\Library\DTO\Reader\ReaderInterface;

class CollectionPreparation implements CollectionPreparationInterface
{
    /**
     * @param iterable<PreparationProcessorInterface> $preparationProcessor
     */
    public function __construct(private iterable $preparationProcessor)
    {
    }

    /**
     * {@inheritDoc}
     */
    public function process(ReaderInterface $reader): iterable
    {
        $classCollection = $reader->read();
        if ($classCollection instanceof \Traversable) {
            $classCollection = iterator_to_array($classCollection);
        }

        $classList = $this->createClassList($classCollection);

It seems like $classCollection can also be of type iterable; however, parameter $classes of Micro\Library\DTO\Preparation\CollectionPreparation::createClassList() does only seem to accept array, maybe add an additional type check?

        foreach ($classCollection as $classDef) {
            $classDefObj = new ClassDefinition();
            foreach ($this->preparationProcessor as $processor) {
                $processor->process($classDef, $classDefObj, $classList);
            }

            yield $classDefObj;
        }
    }

    /**
     * @param array<string, mixed> $classes
     *
     * @return array<class-string>

The doc comment array<class-string> at position 2 could not be parsed: Unknown type name 'class-string' at position 2 in array<class-string>.

     */
    protected function createClassList(array $classes): array
    {
        $result = [];
        foreach ($classes as $classDef) {
            $result[] = $classDef[PreparationProcessorInterface::CLASS_NAME];
        }

        return $result;
    }
}