FakeUser::__construct() - Code Metrics - Inspection of "refactored XmlFormat internals" - thunderer/Serializard - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Branch — master (8bfcf4)

by Tomasz

created 2015-12-10 00:15 UTC

FakeUser::__construct() A

↳ Parent: FakeUser

Complexity

Conditions	1
Paths	1

Size

Total Lines	6
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	6
rs	9.4286
cc	1
eloc	4
nc	1
nop	3

<?php
namespace Thunder\Serializard\Tests\Fake;

final class FakeUser
{
    private $id;
    private $name;
    private $tag;
    /** @var FakeTag[] */
    private $tags = array();

    function __construct($id, $name, FakeTag $tag)

    {
        $this->id = $id;
        $this->name = $name;
        $this->tag = $tag;
    }

    public function setId($id) { $this->id = $id; return $this; }
    public function getId() { return $this->id; }

    public function setName($name) { $this->name = $name; return $this; }
    public function getName() { return $this->name; }

    public function getTag() { return $this->tag; }

    public function addTag(FakeTag $tag)
    {
        $tag->setUser($this);

        $this->tags[] = $tag;
    }

    public function removeTag(FakeTag $removed)
    {
        foreach($this->tags as $key => $tag) {
            if($removed->getName() === $tag->getName()) {
                unset($this->tags[$key]);
            }
        }
    }

    public function getTags() { return $this->tags; }
}


1			<?php
2			namespace Thunder\Serializard\Tests\Fake;
3
4			final class FakeUser
5			{
6			private $id;
7			private $name;
8			private $tag;
9			/** @var FakeTag[] */
10			private $tags = array();
11
12			function __construct($id, $name, FakeTag $tag)
			0 ignored issues – show Best Practice introduced 2015-12-10 00:18 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
13			{
14			$this->id = $id;
15			$this->name = $name;
16			$this->tag = $tag;
17			}
18
19			public function setId($id) { $this->id = $id; return $this; }
20			public function getId() { return $this->id; }
21
22			public function setName($name) { $this->name = $name; return $this; }
23			public function getName() { return $this->name; }
24
25			public function getTag() { return $this->tag; }
26
27			public function addTag(FakeTag $tag)
28			{
29			$tag->setUser($this);
30
31			$this->tags[] = $tag;
32			}
33
34			public function removeTag(FakeTag $removed)
35			{
36			foreach($this->tags as $key => $tag) {
37			if($removed->getName() === $tag->getName()) {
38			unset($this->tags[$key]);
39			}
40			}
41			}
42
43			public function getTags() { return $this->tags; }
44			}
45

thunderer / Serializard

Branch — master (8bfcf4)

FakeUser::__construct() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like