Config::set() - Code Metrics - Inspection of "Added setter for config" - Deployee/config - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 99f005...837aa3 )

by Phillip

created 2019-01-23 20:47 UTC

Config::set() A

↳ Parent: Config

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes

Metric	Value
eloc	1
dl	0
loc	3
ccs	0
cts	3
cp	0
rs	10
c	0
b	0
f	0
cc	1
nc	1
nop	2
crap	2

<?php

namespace Deployee\Components\Config;


class Config implements ConfigInterface
{
    /**
     * @var array|\ArrayAccess
     */
    private $params;

    /**
     * Config constructor.
     * @param array|\ArrayAccess|null $params
     */
    public function __construct($params = null)
    {
        if($params !== null){
            $this->setParams($params);
        }
    }

    /**
     * @param array|\ArrayAccess $params
     */
    public function setParams($params)
    {
        if(!is_array($params)
            && !$params instanceof \ArrayAccess){

            throw new \InvalidArgumentException("Params must be array or implement \\ArrayAcess interface");
        }

        $this->params = $params;
    }

    /**
     * @param string $name
     * @param mixed $default
     * @return mixed
     */
    public function get($name, $default = null)
    {
        return $this->params[$name] ?? $default;
    }

    /**
     * @param string $name
     * @param mixed $value
     */
    public function set(string $name, $value)
    {
        $this->params[$name] = $value;
    }


}

1			<?php
2
3			namespace Deployee\Components\Config;
4
5
6			class Config implements ConfigInterface
7			{
8			/**
9			* @var array\|\ArrayAccess
10			*/
11			private $params;
12
13			/**
14			* Config constructor.
15			* @param array\|\ArrayAccess\|null $params
16			*/
17			public function __construct($params = null)
18			{
19			if($params !== null){
20			$this->setParams($params);
21			}
22			}
23
24			/**
25			* @param array\|\ArrayAccess $params
26			*/
27			public function setParams($params)
28			{
29			if(!is_array($params)
30			&& !$params instanceof \ArrayAccess){
			0 ignored issues – show introduced 2019-01-13 22:06 UTC by Report Bug Copy Issue Report `$params` is always a sub-type of `ArrayAccess`. Loading history...
31			throw new \InvalidArgumentException("Params must be array or implement \\ArrayAcess interface");
32			}
33
34			$this->params = $params;
35			}
36
37			/**
38			* @param string $name
39			* @param mixed $default
40			* @return mixed
41			*/
42			public function get($name, $default = null)
43			{
44			return $this->params[$name] ?? $default;
45			}
46
47			/**
48			* @param string $name
49			* @param mixed $value
50			*/
51			public function set(string $name, $value)
52			{
53			$this->params[$name] = $value;
54			}
55
56
57			}

Deployee / config

Push — master ( 99f005...837aa3 )

Config::set() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like