Container::add() - Code Metrics - Inspection of "Add more tests" - devop-core/container - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 4e2afe...0af9fd )

by Zlatin

created 2017-10-28 13:33 UTC

Container::add() A

↳ Parent: Container

Complexity

Conditions	2
Paths	2

Size

Total Lines	8
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	5
CRAP Score	2

Importance

Changes

Metric	Value
dl	0
loc	8
ccs	5
cts	5
cp	1
rs	9.4285
c	0
b	0
f	0
cc	2
eloc	4
nc	2
nop	2
crap	2

<?php
namespace DevOp\Core\Container;

use Psr\Container\ContainerInterface;
use DevOp\Core\Container\Exception\NotFoundException;
use DevOp\Core\Container\Exception\ContainerException;

class Container implements ContainerInterface
{

    /**
     * @var \ArrayObject
     */
    private $services;

    /**
     * @param int|string $id
     * @return int|string|array|object
     * @throws NotFoundException
     */
    public function get($id)
    {
        if ($this->has($id)) {
            return $this->services[$id];
        }
        throw new NotFoundException();
    }

    /**
     * @param int|string $id
     * @return boolean
     */
    public function has($id)
    {
        if (isset($this->services[$id])) {
            return true;
        }
        return null;
    }

    /**
     * @param string $id
     * @param id|string|array|object $arguments
     * @throws ContainerException
     */
    public function add($id, $arguments)
    {
        if ($this->has($id)) {
            throw new ContainerException("Service with name {$id} allready registered.");
        }

        $this->services[$id] = $arguments;
    }
}


1		<?php
2		namespace DevOp\Core\Container;
3
4		use Psr\Container\ContainerInterface;
5		use DevOp\Core\Container\Exception\NotFoundException;
6		use DevOp\Core\Container\Exception\ContainerException;
7
8		class Container implements ContainerInterface
9		{
10
11		/**
12		* @var \ArrayObject
13		*/
14		private $services;
15
16		/**
17		* @param int\|string $id
18		* @return int\|string\|array\|object
19		* @throws NotFoundException
20		*/
21	9	public function get($id)
22		{
23	9	if ($this->has($id)) {
24	6	return $this->services[$id];
25		}
26	3	throw new NotFoundException();
27		}
28
29		/**
30		* @param int\|string $id
31		* @return boolean
32		*/
33	12	public function has($id)
34		{
35	12	if (isset($this->services[$id])) {
36	9	return true;
37		}
38	12	return null;
39		}
40
41		/**
42		* @param string $id
43		* @param id\|string\|array\|object $arguments
44		* @throws ContainerException
45		*/
46	9	public function add($id, $arguments)
47		{
48	9	if ($this->has($id)) {
49	3	throw new ContainerException("Service with name {$id} allready registered.");
50		}
51
52	9	$this->services[$id] = $arguments;
53	9	}
54		}
55

devop-core / container

Push — master ( 4e2afe...0af9fd )

Container::add() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like