Interpreter::__construct() - Code Metrics - Inspection of "Merge pull request #4 from Jagepard/wip" - Jagepard/PhpDesignPatterns-Interpreter - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 11dcca...383fee )

by Korotkov

created 2018-06-08 12:37 UTC

Interpreter::__construct() A

↳ Parent: Interpreter

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	1
nc	1
nop	1
dl	0
loc	3
rs	10
c	0
b	0
f	0

<?php

declare(strict_types=1);

/**
 * @author    : Korotkov Danila <[email protected]>
 * @license   https://mit-license.org/ MIT
 */

namespace Behavioral\Interpreter;

/**
 * Class Interpreter
 * @package Behavioral\Interpreter
 */
class Interpreter
{

    /**
     * @var Item
     */
    protected $item;
    /**
     * @var Depository
     */
    protected $depository;

    /**
     * Interpreter constructor.
     * @param Depository $depository
     */
    public function __construct(Depository $depository)
    {
        $this->depository = $depository;
    }

    /**
     * @param string $input
     */
    public function interpret(string $input): void
    {
        if (count($this->depository->getItems())) {
            $values = explode(' ', $input);

            $this->setItem($values);
            $this->printValue($values);

            printf('%s', "\n");
        }
    }

    /**
     * @param array $values
     */
    protected function setItem(array $values): void
    {
        foreach ($values as $value) {
            if (is_numeric($value)) {
                $this->item = $this->depository->getItem($value - 1);
            }
        }
    }

    /**
     * @param array $values
     */
    protected function printValue(array $values): void
    {
        if ($this->item instanceof Item) {

            foreach ($values as $value) {
                switch ($value) {
                    case 'author':
                        printf('%s: %s', ' Author', $this->item->getAuthor());
                        break;
                    case 'album':
                        printf('%s: %s', ' Album', $this->item->getAlbum());
                        break;
                }
            }
        }
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			/**
6			* @author : Korotkov Danila <[email protected]>
7			* @license https://mit-license.org/ MIT
8			*/
9
10			namespace Behavioral\Interpreter;
11
12			/**
13			* Class Interpreter
14			* @package Behavioral\Interpreter
15			*/
16			class Interpreter
17			{
18
19			/**
20			* @var Item
21			*/
22			protected $item;
23			/**
24			* @var Depository
25			*/
26			protected $depository;
27
28			/**
29			* Interpreter constructor.
30			* @param Depository $depository
31			*/
32			public function __construct(Depository $depository)
33			{
34			$this->depository = $depository;
35			}
36
37			/**
38			* @param string $input
39			*/
40			public function interpret(string $input): void
41			{
42			if (count($this->depository->getItems())) {
43			$values = explode(' ', $input);
44
45			$this->setItem($values);
46			$this->printValue($values);
47
48			printf('%s', "\n");
49			}
50			}
51
52			/**
53			* @param array $values
54			*/
55			protected function setItem(array $values): void
56			{
57			foreach ($values as $value) {
58			if (is_numeric($value)) {
59			$this->item = $this->depository->getItem($value - 1);
60			}
61			}
62			}
63
64			/**
65			* @param array $values
66			*/
67			protected function printValue(array $values): void
68			{
69			if ($this->item instanceof Item) {
			0 ignored issues – show introduced 2018-06-08 12:39 UTC by Report Bug Copy Issue Report `$this->item` is always a sub-type of `Behavioral\Interpreter\Item`. If `$this->item` can have other possible types, add them to `src/Interpreter.php`:`20`. Loading history...
70			foreach ($values as $value) {
71			switch ($value) {
72			case 'author':
73			printf('%s: %s', ' Author', $this->item->getAuthor());
74			break;
75			case 'album':
76			printf('%s: %s', ' Album', $this->item->getAlbum());
77			break;
78			}
79			}
80			}
81			}
82			}
83

Jagepard / PhpDesignPatterns-Interpreter

Push — master ( 11dcca...383fee )

Interpreter::__construct() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like