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 |
||
9 | class SimpleOffer extends BaseModel |
||
10 | { |
||
11 | /** |
||
12 | * @var int Максимальная длина тэга sales_notes |
||
13 | */ |
||
14 | public static $maxLengthSalesNotes = 50; |
||
15 | /** |
||
16 | * @var int Максимальная длина тэга vendor |
||
17 | */ |
||
18 | public static $maxLengthVendor = 120; |
||
19 | /** |
||
20 | * @var int Максимальная длина тэга name |
||
21 | */ |
||
22 | public static $maxLengthName = 120; |
||
23 | /** |
||
24 | * @inheritdoc |
||
25 | */ |
||
26 | public static $tag = 'offer'; |
||
27 | /** |
||
28 | * @inheritdoc |
||
29 | */ |
||
30 | public static $tagProperties = [ |
||
31 | 'id', |
||
32 | 'bid', |
||
33 | 'cbid', |
||
34 | 'available' |
||
35 | ]; |
||
36 | |||
37 | public $id; |
||
38 | public $bid; |
||
39 | public $cbid; |
||
40 | public $available; |
||
41 | |||
42 | public $url; |
||
43 | public $price; |
||
44 | public $oldprice; |
||
45 | public $currencyId; |
||
46 | public $categoryId; |
||
47 | public $market_category; |
||
48 | public $store; |
||
49 | public $pickup; |
||
50 | public $delivery; |
||
51 | public $local_delivery_cost; |
||
52 | public $name; |
||
53 | public $vendor; |
||
54 | public $vendorCode; |
||
55 | public $description; |
||
56 | public $sales_notes; |
||
57 | public $manufacturer_warranty; |
||
58 | public $country_of_origin; |
||
59 | public $adult; |
||
60 | public $age; |
||
61 | public $barcode; |
||
62 | 4 | public $cpa; |
|
63 | public $params = []; |
||
64 | public $pictures = []; |
||
65 | 4 | /** |
|
66 | 4 | * Опции доставки |
|
67 | 4 | * |
|
68 | 4 | * @var array |
|
69 | 4 | */ |
|
70 | 4 | public $deliveryOptions = []; |
|
71 | 4 | ||
72 | 4 | /** |
|
73 | 4 | * @inheritdoc |
|
74 | 4 | */ |
|
75 | 4 | public function getYmlAttributes() |
|
101 | 4 | ||
102 | 4 | /** |
|
103 | 4 | * @inheritdoc |
|
104 | */ |
||
105 | 4 | public function rules() |
|
188 | |||
189 | 5 | /** |
|
190 | 5 | * @param array $params |
|
191 | */ |
||
192 | public function setParams(array $params) |
||
196 | |||
197 | 5 | /** |
|
198 | * @param array $pictures |
||
199 | 5 | */ |
|
200 | public function setPictures(array $pictures) |
||
204 | |||
205 | /** |
||
206 | * @param array $options |
||
207 | * @throws Exception |
||
208 | 4 | */ |
|
209 | public function setDeliveryOptions(array $options) |
||
216 | 4 | ||
217 | /** |
||
218 | 4 | * @inheritdoc |
|
219 | */ |
||
220 | 4 | protected function getYmlBody() |
|
237 | 4 | ||
238 | 4 | /** |
|
239 | * Добавляет теги для опций доставки |
||
240 | 4 | * |
|
241 | 4 | * @param $string |
|
242 | * @throws Exception |
||
243 | 4 | */ |
|
244 | 4 | protected function appendDeliveryOptions(&$string) |
|
258 | |||
259 | /** |
||
260 | * @param string $attribute |
||
261 | * @param string $value |
||
262 | * @return string |
||
263 | */ |
||
264 | View Code Duplication | protected function getYmlParamTag($attribute, $value) |
|
274 | |||
275 | /** |
||
276 | * Добавляет теги для описания характеристик и параметров товара. |
||
277 | 4 | * |
|
278 | * @param string $string |
||
279 | 4 | * @throws Exception |
|
280 | */ |
||
281 | protected function appendParamTags(&$string) |
||
300 | |||
301 | /** |
||
302 | * @param string $value |
||
303 | * @return string |
||
304 | */ |
||
305 | View Code Duplication | protected function getYmlPictureTag($value) |
|
315 | } |
||
316 |
This check looks from parameters that have been defined for a function or method, but which are not used in the method body.