ChangeSenderAddress - Code Metrics - etrias-nl/paazl-connector - Measure and Improve Code Quality continuously with Scrutinizer

ChangeSenderAddress A
last analyzed 2017-08-08 11:14 UTC

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	261
Duplicated Lines	100 %

Importance

Changes

Metric	Value
dl	261
loc	261
rs	10
c	0
b	0
f	0
wmc	19

19 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	11	11	1
A	getCity()	3	3	1
A	setProvince()	5	5	1
A	setHousenumber()	5	5	1
A	getAdditionalAddressLine()	3	3	1
A	getZipcode()	3	3	1
A	setStreet()	5	5	1
A	getAddition()	3	3	1
A	getCountry()	3	3	1
A	setCountry()	5	5	1
A	setAddresseeLine()	5	5	1
A	setCity()	5	5	1
A	setAddition()	5	5	1
A	setAdditionalAddressLine()	5	5	1
A	getProvince()	3	3	1
A	setZipcode()	5	5	1
A	getAddresseeLine()	3	3	1
A	getHousenumber()	3	3	1
A	getStreet()	3	3	1

How to fix Duplicated Code

<?php

/*
 * This file is part of PHP CS Fixer.
 *
 * (c) Fabien Potencier <[email protected]>
 *     Dariusz Rumiński <[email protected]>
 *
 * This source file is subject to the MIT license that is bundled
 * with this source code in the file LICENSE.
 */

namespace Etrias\PaazlConnector\SoapTypes;

class ChangeSenderAddress

{
    /**
     * @var string
     */
    protected $addresseeLine = null;

    /**
     * @var string
     */
    protected $street = null;

    /**
     * @var string
     */
    protected $housenumber = null;

    /**
     * @var string
     */
    protected $addition = null;

    /**
     * @var string
     */
    protected $zipcode = null;

    /**
     * @var string
     */
    protected $city = null;

    /**
     * @var string
     */
    protected $province = null;

    /**
     * @var string
     */
    protected $country = null;

    /**
     * @var string
     */
    protected $additionalAddressLine = null;

    /**
     * Constructor.
     *
     * @var string
     * @var string $street
     * @var string $housenumber
     * @var string $addition
     * @var string $zipcode
     * @var string $city
     * @var string $province
     * @var string $country
     * @var string $additionalAddressLine
     *
     * @param mixed $addresseeLine
     * @param mixed $street
     * @param mixed $housenumber
     * @param mixed $addition
     * @param mixed $zipcode
     * @param mixed $city
     * @param mixed $province
     * @param mixed $country
     * @param mixed $additionalAddressLine
     */
    public function __construct($addresseeLine, $street, $housenumber, $addition, $zipcode, $city, $province, $country, $additionalAddressLine)
    {
        $this->addresseeLine = $addresseeLine;
class Alien {}

class Dalek extends Alien {}

class Plot
{
    /** @var  Dalek */
    public $villain;
}

$alien = new Alien();
$plot = new Plot();
if ($alien instanceof Dalek) {
    $plot->villain = $alien;
}
        $this->street = $street;
        $this->housenumber = $housenumber;
        $this->addition = $addition;
        $this->zipcode = $zipcode;
        $this->city = $city;
        $this->province = $province;
        $this->country = $country;
        $this->additionalAddressLine = $additionalAddressLine;
    }

    /**
     * @return string
     */
    public function getAddresseeLine()
    {
        return $this->addresseeLine;
    }

    /**
     * @param string $addresseeLine
     *
     * @return $this
     */
    public function setAddresseeLine($addresseeLine)
    {
        $this->addresseeLine = $addresseeLine;

        return $this;
    }

    /**
     * @return string
     */
    public function getStreet()
    {
        return $this->street;
    }

    /**
     * @param string $street
     *
     * @return $this
     */
    public function setStreet($street)
    {
        $this->street = $street;

        return $this;
    }

    /**
     * @return string
     */
    public function getHousenumber()
    {
        return $this->housenumber;
    }

    /**
     * @param string $housenumber
     *
     * @return $this
     */
    public function setHousenumber($housenumber)
    {
        $this->housenumber = $housenumber;

        return $this;
    }

    /**
     * @return string
     */
    public function getAddition()
    {
        return $this->addition;
    }

    /**
     * @param string $addition
     *
     * @return $this
     */
    public function setAddition($addition)
    {
        $this->addition = $addition;

        return $this;
    }

    /**
     * @return string
     */
    public function getZipcode()
    {
        return $this->zipcode;
    }

    /**
     * @param string $zipcode
     *
     * @return $this
     */
    public function setZipcode($zipcode)
    {
        $this->zipcode = $zipcode;

        return $this;
    }

    /**
     * @return string
     */
    public function getCity()
    {
        return $this->city;
    }

    /**
     * @param string $city
     *
     * @return $this
     */
    public function setCity($city)
    {
        $this->city = $city;

        return $this;
    }

    /**
     * @return string
     */
    public function getProvince()
    {
        return $this->province;
    }

    /**
     * @param string $province
     *
     * @return $this
     */
    public function setProvince($province)
    {
        $this->province = $province;

        return $this;
    }

    /**
     * @return string
     */
    public function getCountry()
    {
        return $this->country;
    }

    /**
     * @param string $country
     *
     * @return $this
     */
    public function setCountry($country)
    {
        $this->country = $country;

        return $this;
    }

    /**
     * @return string
     */
    public function getAdditionalAddressLine()
    {
        return $this->additionalAddressLine;
    }

    /**
     * @param string $additionalAddressLine
     *
     * @return $this
     */
    public function setAdditionalAddressLine($additionalAddressLine)
    {
        $this->additionalAddressLine = $additionalAddressLine;

        return $this;
    }
}


1		<?php
2
3		/*
4		* This file is part of PHP CS Fixer.
5		*
6		* (c) Fabien Potencier <[email protected]>
7		* Dariusz Rumiński <[email protected]>
8		*
9		* This source file is subject to the MIT license that is bundled
10		* with this source code in the file LICENSE.
11		*/
12
13		namespace Etrias\PaazlConnector\SoapTypes;
14
15	View Code Duplication	class ChangeSenderAddress
		0 ignored issues – show Duplication introduced 2017-08-01 13:29 UTC by Report Bug Copy Issue Report This class seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
16		{
17		/**
18		* @var string
19		*/
20		protected $addresseeLine = null;
21
22		/**
23		* @var string
24		*/
25		protected $street = null;
26
27		/**
28		* @var string
29		*/
30		protected $housenumber = null;
31
32		/**
33		* @var string
34		*/
35		protected $addition = null;
36
37		/**
38		* @var string
39		*/
40		protected $zipcode = null;
41
42		/**
43		* @var string
44		*/
45		protected $city = null;
46
47		/**
48		* @var string
49		*/
50		protected $province = null;
51
52		/**
53		* @var string
54		*/
55		protected $country = null;
56
57		/**
58		* @var string
59		*/
60		protected $additionalAddressLine = null;
61
62		/**
63		* Constructor.
64		*
65		* @var string
66		* @var string $street
67		* @var string $housenumber
68		* @var string $addition
69		* @var string $zipcode
70		* @var string $city
71		* @var string $province
72		* @var string $country
73		* @var string $additionalAddressLine
74		*
75		* @param mixed $addresseeLine
76		* @param mixed $street
77		* @param mixed $housenumber
78		* @param mixed $addition
79		* @param mixed $zipcode
80		* @param mixed $city
81		* @param mixed $province
82		* @param mixed $country
83		* @param mixed $additionalAddressLine
84		*/
85		public function __construct($addresseeLine, $street, $housenumber, $addition, $zipcode, $city, $province, $country, $additionalAddressLine)
86		{
87		$this->addresseeLine = $addresseeLine;
		0 ignored issues – show Documentation Bug introduced 2017-08-01 13:29 UTC by Report Bug Copy Issue Report `$addresseeLine` is of type `mixed`, but the property `$addresseeLine` was declared to be of type `string`. Are you sure that you always receive this specific sub-class here, or does it make sense to add an `instanceof` check? Our type inference engine has found a suspicous assignment of a value to a property. This check raises an issue when a value that can be of a given class or a super-class is assigned to a property that is type hinted more strictly. Either this assignment is in error or an instanceof check should be added for that assignment. class Alien {} class Dalek extends Alien {} class Plot { /** @var Dalek */ public $villain; } $alien = new Alien(); $plot = new Plot(); if ($alien instanceof Dalek) { $plot->villain = $alien; } Loading history...
88		$this->street = $street;
89		$this->housenumber = $housenumber;
90		$this->addition = $addition;
91		$this->zipcode = $zipcode;
92		$this->city = $city;
93		$this->province = $province;
94		$this->country = $country;
95		$this->additionalAddressLine = $additionalAddressLine;
96		}
97
98		/**
99		* @return string
100		*/
101		public function getAddresseeLine()
102		{
103		return $this->addresseeLine;
104		}
105
106		/**
107		* @param string $addresseeLine
108		*
109		* @return $this
110		*/
111		public function setAddresseeLine($addresseeLine)
112		{
113		$this->addresseeLine = $addresseeLine;
114
115		return $this;
116		}
117
118		/**
119		* @return string
120		*/
121		public function getStreet()
122		{
123		return $this->street;
124		}
125
126		/**
127		* @param string $street
128		*
129		* @return $this
130		*/
131		public function setStreet($street)
132		{
133		$this->street = $street;
134
135		return $this;
136		}
137
138		/**
139		* @return string
140		*/
141		public function getHousenumber()
142		{
143		return $this->housenumber;
144		}
145
146		/**
147		* @param string $housenumber
148		*
149		* @return $this
150		*/
151		public function setHousenumber($housenumber)
152		{
153		$this->housenumber = $housenumber;
154
155		return $this;
156		}
157
158		/**
159		* @return string
160		*/
161		public function getAddition()
162		{
163		return $this->addition;
164		}
165
166		/**
167		* @param string $addition
168		*
169		* @return $this
170		*/
171		public function setAddition($addition)
172		{
173		$this->addition = $addition;
174
175		return $this;
176		}
177
178		/**
179		* @return string
180		*/
181		public function getZipcode()
182		{
183		return $this->zipcode;
184		}
185
186		/**
187		* @param string $zipcode
188		*
189		* @return $this
190		*/
191		public function setZipcode($zipcode)
192		{
193		$this->zipcode = $zipcode;
194
195		return $this;
196		}
197
198		/**
199		* @return string
200		*/
201		public function getCity()
202		{
203		return $this->city;
204		}
205
206		/**
207		* @param string $city
208		*
209		* @return $this
210		*/
211		public function setCity($city)
212		{
213		$this->city = $city;
214
215		return $this;
216		}
217
218		/**
219		* @return string
220		*/
221		public function getProvince()
222		{
223		return $this->province;
224		}
225
226		/**
227		* @param string $province
228		*
229		* @return $this
230		*/
231		public function setProvince($province)
232		{
233		$this->province = $province;
234
235		return $this;
236		}
237
238		/**
239		* @return string
240		*/
241		public function getCountry()
242		{
243		return $this->country;
244		}
245
246		/**
247		* @param string $country
248		*
249		* @return $this
250		*/
251		public function setCountry($country)
252		{
253		$this->country = $country;
254
255		return $this;
256		}
257
258		/**
259		* @return string
260		*/
261		public function getAdditionalAddressLine()
262		{
263		return $this->additionalAddressLine;
264		}
265
266		/**
267		* @param string $additionalAddressLine
268		*
269		* @return $this
270		*/
271		public function setAdditionalAddressLine($additionalAddressLine)
272		{
273		$this->additionalAddressLine = $additionalAddressLine;
274
275		return $this;
276		}
277		}
278

etrias-nl / paazl-connector

ChangeSenderAddress A last analyzed 2017-08-08 11:14 UTC

Complexity

Size/Duplication

Importance

19 Methods

How to fix Duplicated Code

Duplicated Code

Duplication Side-by-Side

Filter issues like

ChangeSenderAddress A
last analyzed 2017-08-08 11:14 UTC