VSV\GVQ_API\Question\Serializers\QuestionJsonEnricher

Complexity

Total Complexity

3

Size/Duplication

Total Lines	30
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	enrich()	0	11	2
A	__construct()	0	3	1

<?php declare(strict_types=1);

namespace VSV\GVQ_API\Question\Serializers;

use Ramsey\Uuid\UuidFactoryInterface;
use VSV\GVQ_API\Common\Serializers\JsonEnricher;

class QuestionJsonEnricher implements JsonEnricher
{
    /**
     * @var UuidFactoryInterface
     */
    private $uuidFactory;

    /**
     * @param UuidFactoryInterface $uuidFactory
     */
    public function __construct(UuidFactoryInterface $uuidFactory)
    {
        $this->uuidFactory = $uuidFactory;
    }

    /**
     * @param string $json
     * @return string
     */
    public function enrich(string $json): string
    {
        $questionAsArray = json_decode($json, true);

        $questionAsArray['id'] = $this->uuidFactory->uuid4();

        for ($index = 0; $index < count($questionAsArray['answers']); $index++) {

It seems like you are calling the size function count() as part of the test condition. You might want to compute the size beforehand, and not on each iteration.

            $questionAsArray['answers'][$index]['id'] = $this->uuidFactory->uuid4();
        }

        return json_encode($questionAsArray);
    }
}