Survey - Code Metrics - Inspection of "[WIP] Alpha1" - geekhub-php/serve-seattle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#30)

unknown

created 2017-03-29 09:32 UTC

Survey A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	175
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	4

Test Coverage

Coverage

79.41%

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
wmc	14
c	1
b	0
f	0
lcom	2
cbo	4
dl	0
loc	175
ccs	27
cts	34
cp	0.7941
rs	10

11 Methods

Rating	Name	Size	Complexity
A	addSurveyAnswer()	9	2
A	getQuestions()	11	3
A	__construct()	5	1
A	getId()	4	1
A	setType()	6	1
A	getType()	4	1
A	setStatus()	6	1
A	getStatus()	4	1
A	setUser()	6	1
A	getUser()	4	1
A	getAnswers()	4	1

<?php

namespace AppBundle\Entity\Survey;

use AppBundle\Entity\User;
use Doctrine\ORM\Mapping as ORM;
use Knp\DoctrineBehaviors\Model as ORMBehaviors;
use Doctrine\Common\Collections\ArrayCollection;
use Symfony\Component\Validator\Constraints as Assert;

/**
 * Survey.
 *
 * @ORM\Entity(repositoryClass="AppBundle\Repository\SurveyRepository")
 */
class Survey
{
    use ORMBehaviors\Timestampable\Timestampable;

    /**
     * @var int
     *
     * @ORM\Column(type="integer")
     * @ORM\Id
     * @ORM\GeneratedValue(strategy="AUTO")
     */
    private $id;

    /**
     * @var SurveyType
     * @Assert\Type("object")
     * @Assert\Valid
     * @ORM\ManyToOne(targetEntity="AppBundle\Entity\Survey\SurveyType", inversedBy="surveys")
     * @ORM\JoinColumn(onDelete="SET NULL")
     */
    private $type;

    /**
     * @var string
     * @Assert\NotBlank()
     * @Assert\Type("string")
     * @ORM\Column(type="string")
     */
    private $status;

    /**
     * @var User
     * @Assert\Type("object")
     * @Assert\Valid
     * @ORM\ManyToOne(targetEntity="AppBundle\Entity\User", inversedBy="surveys")
     * @ORM\JoinColumn(onDelete="CASCADE")
     */
    private $user;

    /**
     * @var ArrayCollection[SurveyAnswer]
     * @ORM\OneToMany(targetEntity="SurveyAnswer", mappedBy="survey", cascade={"persist", "remove"})
     */
    private $answers;

    public function __construct()
    {
        $this->answers = new ArrayCollection();
        $this->status = 'current';
    }

    /**
     * Get id.
     *
     * @return int
     */
    public function getId()
    {
        return $this->id;
    }

    /**
     * Set type.
     *
     * @param SurveyType $type
     *
     * @return Survey
     */
    public function setType(SurveyType $type)
    {
        $this->type = $type;

        return $this;
    }

    /**
     * Get type.
     *
     * @return SurveyType
     */
    public function getType()
    {
        return $this->type;
    }

    /**
     * Set status.
     *
     * @param string $status
     *
     * @return Survey
     */
    public function setStatus($status)
    {
        $this->status = $status;

        return $this;
    }

    /**
     * Get status.
     *
     * @return string
     */
    public function getStatus()
    {
        return $this->status;
    }

    /**
     * Set user.
     *
     * @param User $user
     *
     * @return Survey
     */
    public function setUser(User $user)
    {
        $this->user = $user;

        return $this;
    }

    /**
     * Get user.
     *
     * @return User
     */
    public function getUser()
    {
        return $this->user;
    }

    /**
     * @param SurveyAnswer $answer
     *
     * @return Survey
     */
    public function addSurveyAnswer(SurveyAnswer $answer)
    {
        if (!$this->answers->contains($answer)) {
            $this->answers->add($answer);
            $answer->setSurvey($this);
        }

        return $this;
    }

    /**
     * Get answers.
     *
     * @return ArrayCollection
     */
    public function getAnswers()
    {
        return $this->answers;
    }

    /**
     * Get questions.
     *
     * @return array
     */
    public function getQuestions()
    {
        $sections = $this->getType()->getSections();
        foreach ($sections as $section) {
            foreach ($section->getQuestions() as $quest) {
                $questions[] = $quest;
foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}
            }
        }

        return $questions;
function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}
    }
}


1		<?php
2
3		namespace AppBundle\Entity\Survey;
4
5		use AppBundle\Entity\User;
6		use Doctrine\ORM\Mapping as ORM;
7		use Knp\DoctrineBehaviors\Model as ORMBehaviors;
8		use Doctrine\Common\Collections\ArrayCollection;
9		use Symfony\Component\Validator\Constraints as Assert;
10
11		/**
12		* Survey.
13		*
14		* @ORM\Entity(repositoryClass="AppBundle\Repository\SurveyRepository")
15		*/
16		class Survey
17		{
18		use ORMBehaviors\Timestampable\Timestampable;
19
20		/**
21		* @var int
22		*
23		* @ORM\Column(type="integer")
24		* @ORM\Id
25		* @ORM\GeneratedValue(strategy="AUTO")
26		*/
27		private $id;
28
29		/**
30		* @var SurveyType
31		* @Assert\Type("object")
32		* @Assert\Valid
33		* @ORM\ManyToOne(targetEntity="AppBundle\Entity\Survey\SurveyType", inversedBy="surveys")
34		* @ORM\JoinColumn(onDelete="SET NULL")
35		*/
36		private $type;
37
38		/**
39		* @var string
40		* @Assert\NotBlank()
41		* @Assert\Type("string")
42		* @ORM\Column(type="string")
43		*/
44		private $status;
45
46		/**
47		* @var User
48		* @Assert\Type("object")
49		* @Assert\Valid
50		* @ORM\ManyToOne(targetEntity="AppBundle\Entity\User", inversedBy="surveys")
51		* @ORM\JoinColumn(onDelete="CASCADE")
52		*/
53		private $user;
54
55		/**
56		* @var ArrayCollection[SurveyAnswer]
57		* @ORM\OneToMany(targetEntity="SurveyAnswer", mappedBy="survey", cascade={"persist", "remove"})
58		*/
59		private $answers;
60
61	1	public function __construct()
62		{
63	1	$this->answers = new ArrayCollection();
64	1	$this->status = 'current';
65	1	}
66
67		/**
68		* Get id.
69		*
70		* @return int
71		*/
72	3	public function getId()
73		{
74	3	return $this->id;
75		}
76
77		/**
78		* Set type.
79		*
80		* @param SurveyType $type
81		*
82		* @return Survey
83		*/
84	1	public function setType(SurveyType $type)
85		{
86	1	$this->type = $type;
87
88	1	return $this;
89		}
90
91		/**
92		* Get type.
93		*
94		* @return SurveyType
95		*/
96	7	public function getType()
97		{
98	7	return $this->type;
99		}
100
101		/**
102		* Set status.
103		*
104		* @param string $status
105		*
106		* @return Survey
107		*/
108	1	public function setStatus($status)
109		{
110	1	$this->status = $status;
111
112	1	return $this;
113		}
114
115		/**
116		* Get status.
117		*
118		* @return string
119		*/
120	3	public function getStatus()
121		{
122	3	return $this->status;
123		}
124
125		/**
126		* Set user.
127		*
128		* @param User $user
129		*
130		* @return Survey
131		*/
132	1	public function setUser(User $user)
133		{
134	1	$this->user = $user;
135
136	1	return $this;
137		}
138
139		/**
140		* Get user.
141		*
142		* @return User
143		*/
144	5	public function getUser()
145		{
146	5	return $this->user;
147		}
148
149		/**
150		* @param SurveyAnswer $answer
151		*
152		* @return Survey
153		*/
154		public function addSurveyAnswer(SurveyAnswer $answer)
155		{
156		if (!$this->answers->contains($answer)) {
157		$this->answers->add($answer);
158		$answer->setSurvey($this);
159		}
160
161		return $this;
162		}
163
164		/**
165		* Get answers.
166		*
167		* @return ArrayCollection
168		*/
169		public function getAnswers()
170		{
171		return $this->answers;
172		}
173
174		/**
175		* Get questions.
176		*
177		* @return array
178		*/
179	1	public function getQuestions()
180		{
181	1	$sections = $this->getType()->getSections();
182	1	foreach ($sections as $section) {
183	1	foreach ($section->getQuestions() as $quest) {
184	1	$questions[] = $quest;
		0 ignored issues – show Coding Style Comprehensibility introduced 2017-03-25 18:16 UTC by Report Bug Copy Issue Report `$questions` was never initialized. Although not strictly required by PHP, it is generally a good practice to add `$questions = array();` before regardless. Adding an explicit array definition is generally preferable to implicit array definition as it guarantees a stable state of the code. Let’s take a look at an example: foreach ($collection as $item) { $myArray['foo'] = $item->getFoo(); if ($item->hasBar()) { $myArray['bar'] = $item->getBar(); } // do something with $myArray } As you can see in this example, the array `$myArray` is initialized the first time when the foreach loop is entered. You can also see that the value of the `bar` key is only written conditionally; thus, its value might result from a previous iteration. This might or might not be intended. To make your intention clear, your code more readible and to avoid accidental bugs, we recommend to add an explicit initialization `$myArray = array()` either outside or inside the foreach loop. Loading history...
185		}
186		}
187
188	1	return $questions;
		0 ignored issues – show Bug introduced 2017-03-25 18:16 UTC by Report Bug Copy Issue Report The variable `$questions` does not seem to be defined for all execution paths leading up to this point. If you define a variable conditionally, it can happen that it is not defined for all execution paths. Let’s take a look at an example: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } // $x is potentially undefined here. echo $x; } In the above example, the variable `$x` is defined if you pass “foo” or “bar” as argument for `$a`. However, since the `switch` statement has no default case statement, if you pass any other value, the variable `$x` would be undefined. Available Fixes Check for existence of the variable explicitly: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } if (isset($x)) { // Make sure it's always set. echo $x; } } Define a default value for the variable: function myFunction($a) { $x = ''; // Set a default which gets overridden for certain paths. switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } echo $x; } Add a value for the missing path: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; // We add support for the missing case. default: $x = ''; break; } echo $x; } Loading history...
189		}
190		}
191

geekhub-php / serve-seattle

Pull Request — master (#30)

Survey A

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

11 Methods

Available Fixes

Duplication Side-by-Side

Filter issues like