Creator::getOrFail() - Code Metrics - Inspection of "Refresh all classes" - PHPAlgorithms/GraphTools - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 84d6ef...f4a207 )

by Ventaquil

created 2016-03-03 15:51 UTC

Creator::getOrFail() A

↳ Parent: Creator

Complexity

Conditions	2
Paths	2

Size

Total Lines	10
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

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

<?php
namespace Algorithms\GraphTools;

class Creator
{
    protected $points = array(),

              $labels = array();

    public function __construct($points = null)
    {
        if (is_array($points)) {
            foreach ($points as $point) {
                $this->add($point);
            }
        } elseif(!is_null($points)) {
            throw new CreatorException('Creator contructor only accept points array');
        }
    }

    protected function pointNotExists($point)
    {
        Point::checkOrFail($point);

        foreach ($this->points as $p) {
            if ($p == $point) {
                return false;
            }
        }

        return true;
    }

    protected function addPoint($point)
    {
        if (!$this->pointNotExists($point)) {
            throw new CreatorException('Point is already stored in Creator object');
        }

        $this->points[] = $point;
    }

    public function add($point)
    {
        if (filter_var($point, FILTER_VALIDATE_INT) && is_numeric($point)) {
            $point = new Point($point);
        } elseif (!Point::check($point)) {
            throw new CreatorException('You can only add points to Creator object');
        }

        $this->addPoint($point);

        return $point;
    }

    public function get($id)
    {
        foreach ($this->points as $point)
        {
            if ($point->id == $id) {
                return $point;
            }
        }

        return null;
    }

    public function getOrFail($id)
    {
        $point = $this->get($id);

        if (!is_null($point)) {
            throw new CreatorException("Point with ID {$id} not found");
        }

        return $point;
    }
}


1			<?php
2			namespace Algorithms\GraphTools;
3
4			class Creator
5			{
6			protected $points = array(),
			0 ignored issues – show Coding Style introduced 2016-03-03 15:54 UTC by Report Bug Copy Issue Report It is generally advisable to only define one property per statement. Only declaring a single property per statement allows you to later on add doc comments more easily. It is also recommended by PSR2, so it is a common style that many people expect. Loading history...
7			$labels = array();
8
9			public function __construct($points = null)
10			{
11			if (is_array($points)) {
12			foreach ($points as $point) {
13			$this->add($point);
14			}
15			} elseif(!is_null($points)) {
16			throw new CreatorException('Creator contructor only accept points array');
17			}
18			}
19
20			protected function pointNotExists($point)
21			{
22			Point::checkOrFail($point);
23
24			foreach ($this->points as $p) {
25			if ($p == $point) {
26			return false;
27			}
28			}
29
30			return true;
31			}
32
33			protected function addPoint($point)
34			{
35			if (!$this->pointNotExists($point)) {
36			throw new CreatorException('Point is already stored in Creator object');
37			}
38
39			$this->points[] = $point;
40			}
41
42			public function add($point)
43			{
44			if (filter_var($point, FILTER_VALIDATE_INT) && is_numeric($point)) {
45			$point = new Point($point);
46			} elseif (!Point::check($point)) {
47			throw new CreatorException('You can only add points to Creator object');
48			}
49
50			$this->addPoint($point);
51
52			return $point;
53			}
54
55			public function get($id)
56			{
57			foreach ($this->points as $point)
58			{
59			if ($point->id == $id) {
60			return $point;
61			}
62			}
63
64			return null;
65			}
66
67			public function getOrFail($id)
68			{
69			$point = $this->get($id);
70
71			if (!is_null($point)) {
72			throw new CreatorException("Point with ID {$id} not found");
73			}
74
75			return $point;
76			}
77			}
78

PHPAlgorithms / GraphTools

Push — master ( 84d6ef...f4a207 )

Creator::getOrFail() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like