Behavioral\Interpreter\Interpreter

Complexity

Total Complexity

8

Size/Duplication

Total Lines	47
Duplicated Lines	0 %

Importance

Changes	10
Bugs	0	Features	2

3 Methods

Rating	Name	Duplication	Size	Complexity
A	interpret()	0	8	3
A	getDataFromRegistry()	0	13	4
A	addAlbumToRegistry()	0	3	1

<?php

declare(strict_types=1);

/**
 * @author  : Jagepard <jagepard@yandex.ru>
 * @license https://mit-license.org/ MIT
 */

namespace Behavioral\Interpreter;

class Interpreter implements InterpreterInterface
{
    /**
     * @var array
     */
    private $registry;

    /**
     * @param  AlbumInterface  $item
     */
    public function addAlbumToRegistry(AlbumInterface $item): void
    {
        $this->registry[] = $item;
    }

    /**
     * @param  string  $input
     */
    public function interpret(string $input): void
    {
        $item = null;

The assignment to $item is dead and can be removed.

        $input = explode(" ", $input);

        foreach ($input as $value) {
            if (is_numeric($value)) {
                $this->getDataFromRegistry($input, $this->registry[$value - 1]);
            }
        }
    }

    /**
     * @param  array  $input
     * @param $item
     */
    private function getDataFromRegistry(array $input, AlbumInterface $item): void
    {
        foreach ($input as $value) {
            if ($value === "album") {
                printf("%s ", $item->getName());
            }

            if ($value === "author") {
                printf("%s", $item->getAuthor());
            }
        }

        printf("%s", PHP_EOL);
    }
}