Duplicate code is one of the most pungent code smells. A rule that is often used is to re-structure code once it is duplicated in three or more places.
Common duplication problems, and corresponding solutions are:
1 | <?php |
||
20 | View Code Duplication | class NumberInterval implements ComparableIntervalInterface |
|
|
|||
21 | { |
||
22 | /** |
||
23 | * @var string |
||
24 | */ |
||
25 | const REGEXP = '/^ |
||
26 | (?:\(|\[) # start type char |
||
27 | \s* |
||
28 | (?<start>\-?\d+) # start point |
||
29 | \s*,\s* # separator |
||
30 | (?<end>\-?\d+) # end point |
||
31 | \s* |
||
32 | (?:\)|\]) # end type char |
||
33 | $/x'; |
||
34 | |||
35 | /** |
||
36 | * @var IntervalType |
||
37 | */ |
||
38 | private $type; |
||
39 | |||
40 | /** |
||
41 | * @var IntervalComparator |
||
42 | */ |
||
43 | private $comparator; |
||
44 | |||
45 | /** |
||
46 | * @var NumberIntervalPoint |
||
47 | */ |
||
48 | private $start; |
||
49 | |||
50 | /** |
||
51 | * @var NumberIntervalPoint |
||
52 | */ |
||
53 | private $end; |
||
54 | |||
55 | /** |
||
56 | * @param NumberIntervalPoint $start |
||
57 | * @param NumberIntervalPoint $end |
||
58 | * @param IntervalType $type |
||
59 | */ |
||
60 | private function __construct(NumberIntervalPoint $start, NumberIntervalPoint $end, IntervalType $type) |
||
71 | |||
72 | /** |
||
73 | * @param int|float $start |
||
74 | * @param int|float $end |
||
75 | * @param IntervalType $type |
||
76 | * |
||
77 | * @return self |
||
78 | */ |
||
79 | public static function create($start, $end, IntervalType $type) |
||
83 | |||
84 | /** |
||
85 | * @param int|float $start |
||
86 | * @param int|float $end |
||
87 | * |
||
88 | * @return self |
||
89 | */ |
||
90 | public static function closed($start, $end) |
||
94 | |||
95 | /** |
||
96 | * @param int|float $start |
||
97 | * @param int|float $end |
||
98 | * |
||
99 | * @return self |
||
100 | */ |
||
101 | public static function halfClosed($start, $end) |
||
105 | |||
106 | /** |
||
107 | * @param int|float $start |
||
108 | * @param int|float $end |
||
109 | * |
||
110 | * @return self |
||
111 | */ |
||
112 | public static function halfOpen($start, $end) |
||
116 | |||
117 | /** |
||
118 | * @param int|float $start |
||
119 | * @param int|float $end |
||
120 | * |
||
121 | * @return self |
||
122 | */ |
||
123 | public static function open($start, $end) |
||
127 | |||
128 | /** |
||
129 | * Create interval from string. |
||
130 | * |
||
131 | * Example formats: |
||
132 | * [0, 5] |
||
133 | * (-3, 2] |
||
134 | * [-3, -1) |
||
135 | * (3, 9) |
||
136 | * |
||
137 | * Spaces are ignored in format. |
||
138 | * |
||
139 | * @param string $string |
||
140 | * |
||
141 | * @return self |
||
142 | */ |
||
143 | public static function fromString($string) |
||
151 | |||
152 | /** |
||
153 | * @param NumberInterval $interval |
||
154 | * |
||
155 | * @return bool |
||
156 | */ |
||
157 | public function equal(NumberInterval $interval) |
||
161 | |||
162 | /** |
||
163 | * @param int|float $point |
||
164 | * |
||
165 | * @return bool |
||
166 | */ |
||
167 | public function contains($point) |
||
171 | |||
172 | /** |
||
173 | * @param NumberInterval $interval |
||
174 | * @param bool $check_interval_type |
||
175 | * |
||
176 | * @return bool |
||
177 | */ |
||
178 | public function intersects(NumberInterval $interval, $check_interval_type = true) |
||
182 | |||
183 | /** |
||
184 | * @param NumberInterval $interval |
||
185 | * |
||
186 | * @return NumberInterval|null |
||
187 | */ |
||
188 | public function intersection(NumberInterval $interval) |
||
192 | |||
193 | /** |
||
194 | * The point is before the interval. |
||
195 | * |
||
196 | * @param int|float $point |
||
197 | * |
||
198 | * @return bool |
||
199 | */ |
||
200 | public function before($point) |
||
204 | |||
205 | /** |
||
206 | * The point is after the interval. |
||
207 | * |
||
208 | * @param int|float $point |
||
209 | * |
||
210 | * @return bool |
||
211 | */ |
||
212 | public function after($point) |
||
216 | |||
217 | /** |
||
218 | * @return IntervalType |
||
219 | */ |
||
220 | public function type() |
||
224 | |||
225 | /** |
||
226 | * @return int|float |
||
227 | */ |
||
228 | public function start() |
||
232 | |||
233 | /** |
||
234 | * @return int|float |
||
235 | */ |
||
236 | public function end() |
||
240 | |||
241 | /** |
||
242 | * @return NumberIntervalPoint |
||
243 | */ |
||
244 | public function startPoint() |
||
248 | |||
249 | /** |
||
250 | * @return NumberIntervalPoint |
||
251 | */ |
||
252 | public function endPoint() |
||
256 | |||
257 | /** |
||
258 | * Returns a copy of this Interval with the start point altered. |
||
259 | * |
||
260 | * @param IntervalPointInterface|NumberIntervalPoint $start |
||
261 | * |
||
262 | * @return self |
||
263 | */ |
||
264 | public function withStart(IntervalPointInterface $start) |
||
268 | |||
269 | /** |
||
270 | * Returns a copy of this Interval with the end point altered. |
||
271 | * |
||
272 | * @param IntervalPointInterface|NumberIntervalPoint $end |
||
273 | * |
||
274 | * @return self |
||
275 | */ |
||
276 | public function withEnd(IntervalPointInterface $end) |
||
280 | |||
281 | /** |
||
282 | * Returns a copy of this Interval with the interval type altered. |
||
283 | * |
||
284 | * @param IntervalType $type |
||
285 | * |
||
286 | * @return self |
||
287 | */ |
||
288 | public function withType(IntervalType $type) |
||
292 | |||
293 | /** |
||
294 | * @return string |
||
295 | */ |
||
296 | public function __toString() |
||
300 | } |
||
301 |
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.