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.