BusBuilders::get() - Code Metrics - Inspection of "Refactor and add support for Handler Mapping Strat..." - thephpleague/tactician-bundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#67)

by Ross

created 2017-09-17 16:19 UTC

BusBuilders::get() A

↳ Parent: BusBuilders

Complexity

Conditions	2
Paths	2

Size

Total Lines	8
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	8
rs	9.4285
c	0
b	0
f	0
cc	2
eloc	4
nc	2
nop	1

<?php
declare(strict_types=1);

namespace League\Tactician\Bundle\DependencyInjection\Compiler\BusBuilder;

use League\Tactician\Bundle\DependencyInjection\InvalidCommandBusId;

final class BusBuilders implements \IteratorAggregate
{
    /**
     * @var BusBuilder[]
     */
    private $busBuilders = [];

    /**
     * @var string
     */
    private $defaultBusId;

    public function __construct(array $busBuilders, string $defaultBusId)
    {
        foreach ($busBuilders as $builder) {
            $this->busBuilders[$builder->id()] = $builder;
        }

        $this->defaultBusId = $defaultBusId;
    }

    public function getBusIds(): array
    {
        return array_keys($this->busBuilders);
    }

    public function defaultBus(): BusBuilder
    {
        return $this->get($this->defaultBusId);
    }

    private function get(string $busId): BusBuilder
    {
        if (!isset($this->busBuilders[$busId])) {
            throw InvalidCommandBusId::ofName($busId, array_keys($this->busBuilders));
        }

        return $this->busBuilders[$busId];
    }

    public function getIterator()
    {
        return new \ArrayIterator($this->busBuilders);
    }
}

1			<?php
2			declare(strict_types=1);
3
4			namespace League\Tactician\Bundle\DependencyInjection\Compiler\BusBuilder;
5
6			use League\Tactician\Bundle\DependencyInjection\InvalidCommandBusId;
7
8			final class BusBuilders implements \IteratorAggregate
9			{
10			/**
11			* @var BusBuilder[]
12			*/
13			private $busBuilders = [];
14
15			/**
16			* @var string
17			*/
18			private $defaultBusId;
19
20			public function __construct(array $busBuilders, string $defaultBusId)
21			{
22			foreach ($busBuilders as $builder) {
23			$this->busBuilders[$builder->id()] = $builder;
24			}
25
26			$this->defaultBusId = $defaultBusId;
27			}
28
29			public function getBusIds(): array
30			{
31			return array_keys($this->busBuilders);
32			}
33
34			public function defaultBus(): BusBuilder
35			{
36			return $this->get($this->defaultBusId);
37			}
38
39			private function get(string $busId): BusBuilder
40			{
41			if (!isset($this->busBuilders[$busId])) {
42			throw InvalidCommandBusId::ofName($busId, array_keys($this->busBuilders));
43			}
44
45			return $this->busBuilders[$busId];
46			}
47
48			public function getIterator()
49			{
50			return new \ArrayIterator($this->busBuilders);
51			}
52			}

thephpleague / tactician-bundle

Pull Request — master (#67)

BusBuilders::get() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like