UniversalContainerIdConfiguration::getData() - Code Metrics - Inspection of "Merge remote-tracking branch 'origin/hotfix/incons..." - sandrokeil/interop-config - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( f60e47...74fc93 )

by Sandro

created 2016-12-09 22:49 UTC

UniversalContainerIdConfiguration::getData() B

↳ Parent: UniversalContainerIdConfiguration

Complexity

Conditions	5
Paths	5

Size

Total Lines	18
Code Lines	15

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	18
rs	8.8571
c	0
b	0
f	0
cc	5
eloc	15
nc	5
nop	1

<?php
/**
 * Sandro Keil (https://sandro-keil.de)
 *
 * @link      http://github.com/sandrokeil/interop-config for the canonical source repository
 * @copyright Copyright (c) 2015-2016 Sandro Keil
 * @license   http://github.com/sandrokeil/interop-config/blob/master/LICENSE.md New BSD License
 */

namespace InteropTest\Config\TestAsset;

use Interop\Config\ConfigurationTrait;
use Interop\Config\ProvidesDefaultOptions;
use Interop\Config\RequiresConfigId;
use Interop\Config\RequiresMandatoryOptions;

class UniversalContainerIdConfiguration implements RequiresConfigId, ProvidesDefaultOptions, RequiresMandatoryOptions
{
    use ConfigurationTrait;

    const TYPE_ARRAY_ITERATOR = 0;
    const TYPE_ARRAY_OBJECT = 1;
    const TYPE_ARRAY_ARRAY = 2;
    const TYPE_ONLY_ITERATOR = 3;

    private static $dimensions = [
        'doctrine',
        'universal',
    ];

    private static $mandatoryOptions = [
        'params' => ['user', 'dbname'],
        'driverClass',
    ];

    private static $defaultOptions = [
        'params' => [
            'host' => 'awesomehost',
            'port' => '4444',
        ],
    ];

    private $type;

    public function __construct(int $type)
    {
        $this->type = $type;
    }

    public function dimensions(): iterable
    {
        return $this->getData('dimensions');
    }

    public function mandatoryOptions(): iterable
    {
        return $this->getData('mandatoryOptions');
    }

    public function defaultOptions(): iterable
    {
        return $this->getData('defaultOptions');
    }

    private function getData($name)
    {
        switch ($this->type) {
            case self::TYPE_ARRAY_ITERATOR:
                return new \ArrayIterator(self::$$name);
                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
            case self::TYPE_ARRAY_OBJECT:
                return new \ArrayObject(self::$$name);
                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
            case self::TYPE_ONLY_ITERATOR:
                return new OnlyIterator(self::$$name);
                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
            case self::TYPE_ARRAY_ARRAY:
            default:
                return self::$$name;
                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
        }
    }
}


1			<?php
2			/**
3			* Sandro Keil (https://sandro-keil.de)
4			*
5			* @link http://github.com/sandrokeil/interop-config for the canonical source repository
6			* @copyright Copyright (c) 2015-2016 Sandro Keil
7			* @license http://github.com/sandrokeil/interop-config/blob/master/LICENSE.md New BSD License
8			*/
9
10			namespace InteropTest\Config\TestAsset;
11
12			use Interop\Config\ConfigurationTrait;
13			use Interop\Config\ProvidesDefaultOptions;
14			use Interop\Config\RequiresConfigId;
15			use Interop\Config\RequiresMandatoryOptions;
16
17			class UniversalContainerIdConfiguration implements RequiresConfigId, ProvidesDefaultOptions, RequiresMandatoryOptions
18			{
19			use ConfigurationTrait;
20
21			const TYPE_ARRAY_ITERATOR = 0;
22			const TYPE_ARRAY_OBJECT = 1;
23			const TYPE_ARRAY_ARRAY = 2;
24			const TYPE_ONLY_ITERATOR = 3;
25
26			private static $dimensions = [
27			'doctrine',
28			'universal',
29			];
30
31			private static $mandatoryOptions = [
32			'params' => ['user', 'dbname'],
33			'driverClass',
34			];
35
36			private static $defaultOptions = [
37			'params' => [
38			'host' => 'awesomehost',
39			'port' => '4444',
40			],
41			];
42
43			private $type;
44
45			public function __construct(int $type)
46			{
47			$this->type = $type;
48			}
49
50			public function dimensions(): iterable
51			{
52			return $this->getData('dimensions');
53			}
54
55			public function mandatoryOptions(): iterable
56			{
57			return $this->getData('mandatoryOptions');
58			}
59
60			public function defaultOptions(): iterable
61			{
62			return $this->getData('defaultOptions');
63			}
64
65			private function getData($name)
66			{
67			switch ($this->type) {
68			case self::TYPE_ARRAY_ITERATOR:
69			return new \ArrayIterator(self::$$name);
70			break;
			0 ignored issues – show Unused Code introduced 2016-12-09 22:51 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
71			case self::TYPE_ARRAY_OBJECT:
72			return new \ArrayObject(self::$$name);
73			break;
			0 ignored issues – show Unused Code introduced 2016-12-09 22:51 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
74			case self::TYPE_ONLY_ITERATOR:
75			return new OnlyIterator(self::$$name);
76			break;
			0 ignored issues – show Unused Code introduced 2016-12-09 22:51 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
77			case self::TYPE_ARRAY_ARRAY:
78			default:
79			return self::$$name;
80			break;
			0 ignored issues – show Unused Code introduced 2016-12-09 22:51 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
81			}
82			}
83			}
84

sandrokeil / interop-config

Push — master ( f60e47...74fc93 )

UniversalContainerIdConfiguration::getData() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like