ParameterBag::add() - Code Metrics - Inspection of "Rollback on env setting from webservice tests" - bricev/JobQueue - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — v2 ( 9bff46...d4d6f5 )

by Brice

created 2018-03-07 14:07 UTC

ParameterBag::add() A

↳ Parent: ParameterBag

Complexity

Conditions	4
Paths	3

Size

Total Lines	11
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	6
CRAP Score	4

Importance

Changes

Metric	Value
dl	0
loc	11
rs	9.2
c	0
b	0
f	0
ccs	6
cts	6
cp	1
cc	4
eloc	5
nc	3
nop	2
crap	4

<?php

namespace JobQueue\Domain\Task;

use JobQueue\Domain\Utils\Bag;

final class ParameterBag extends Bag
{
    /**
     *
     * @param array $data
     */
    public function __construct(array $data = [])
    {
        foreach ($data as $key => $value) {
            $this->add($key, $value);
        }
    }

    /**
     *
     * @param mixed $key
     * @return bool
     */
    public function has($key): bool
    {
        return isset($this->data[$key]);
    }

    /**
     *
     * @param mixed $key
     * @return mixed
     */
    public function get($key)
    {
        if (!$this->has($key)) {
            throw new \RuntimeException(sprintf('There is no value for "%s" key', $key));
        }

        return $this->data[$key];
    }

    /**
     *
     * @param mixed $key
     * @param mixed $value
     */
    public function add($key, $value)
    {
        if (!is_string($key)) {
            throw new \RuntimeException('The key must be a string');
        }

        if (!is_scalar($value) and !is_null($value)) {
            throw new \RuntimeException(sprintf('The "%s" value must be a scalar or null', $key));
        }

        $this->data[$key] = $value;
    }

    /**
     *
     * @param mixed $key
     */
    public function remove($key)
    {
        unset($this->data[$key]);
    }
}


1		<?php
2
3		namespace JobQueue\Domain\Task;
4
5		use JobQueue\Domain\Utils\Bag;
6
7		final class ParameterBag extends Bag
8		{
9		/**
10		*
11		* @param array $data
12		*/
13	29	public function __construct(array $data = [])
14		{
15	29	foreach ($data as $key => $value) {
16	20	$this->add($key, $value);
17		}
18	27	}
19
20		/**
21		*
22		* @param mixed $key
23		* @return bool
24		*/
25	2	public function has($key): bool
26		{
27	2	return isset($this->data[$key]);
28		}
29
30		/**
31		*
32		* @param mixed $key
33		* @return mixed
34		*/
35	1	public function get($key)
36		{
37	1	if (!$this->has($key)) {
38	1	throw new \RuntimeException(sprintf('There is no value for "%s" key', $key));
39		}
40
41		return $this->data[$key];
42		}
43
44		/**
45		*
46		* @param mixed $key
47		* @param mixed $value
48		*/
49	20	public function add($key, $value)
50		{
51	20	if (!is_string($key)) {
52	1	throw new \RuntimeException('The key must be a string');
53		}
54
55	20	if (!is_scalar($value) and !is_null($value)) {
56	1	throw new \RuntimeException(sprintf('The "%s" value must be a scalar or null', $key));
57		}
58
59	20	$this->data[$key] = $value;
60	20	}
61
62		/**
63		*
64		* @param mixed $key
65		*/
66		public function remove($key)
67		{
68		unset($this->data[$key]);
69		}
70		}
71

bricev / JobQueue

Push — v2 ( 9bff46...d4d6f5 )

ParameterBag::add() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like