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 | class Report implements Common |
||
21 | { |
||
22 | /** |
||
23 | * Группировка по дням |
||
24 | */ |
||
25 | const GROUP_BY_DAY = 'day'; |
||
26 | |||
27 | /** |
||
28 | * Группировка по месяцам |
||
29 | */ |
||
30 | const GROUP_BY_MONTH = 'month'; |
||
31 | |||
32 | /** |
||
33 | * Группировка по годам |
||
34 | */ |
||
35 | const GROUP_BY_YEAR = 'year'; |
||
36 | |||
37 | /** |
||
38 | * Инстанс клиента API |
||
39 | * |
||
40 | * @var Client |
||
41 | */ |
||
42 | private $client; |
||
43 | |||
44 | /** |
||
45 | * Конструктор общего отчета |
||
46 | * |
||
47 | * Экземпляр класса Report нужен для осуществления запросов к API для получения отчетных данных ЭБС Лань. |
||
48 | * |
||
49 | * @param Client $client Инстанс клиента |
||
50 | * |
||
51 | * Пример: |
||
52 | * ```php |
||
53 | * $token = '7c0c2193d27108a509abd8ea84a8750c82b3a520'; // токен для тестового подписчика |
||
54 | * |
||
55 | * $client = new Client($token); // инициализация клиента |
||
56 | * |
||
57 | * $report = new Report($client): |
||
58 | * ``` |
||
59 | * |
||
60 | * @throws Exception |
||
61 | */ |
||
62 | 5 | public function __construct(Client $client) |
|
70 | |||
71 | /** |
||
72 | * Общая статистика чтения книг |
||
73 | * |
||
74 | * @param string $groupBy Группировка ('day|month|year') |
||
75 | * @param string $periodFrom Период с (формат Y-m-d, например 2017-10-01) |
||
76 | * @param string $periodTo Период с (формат Y-m-d, например 2017-11-01) |
||
77 | * |
||
78 | * |
||
79 | * |
||
80 | * @return mixed |
||
81 | * |
||
82 | * @throws Exception |
||
83 | */ |
||
84 | 1 | View Code Duplication | public function getBooksViewsStatistics($groupBy, $periodFrom, $periodTo) |
|
|||
85 | { |
||
86 | 1 | return $this->client->getResponse( |
|
87 | 1 | $this->getUrl(__FUNCTION__), |
|
88 | array( |
||
89 | 1 | 'group_by' => $groupBy, |
|
90 | 1 | 'period_range_from' => $periodFrom, |
|
91 | 1 | 'period_range_to' => $periodTo, |
|
92 | ) |
||
93 | )['data']; |
||
94 | } |
||
95 | |||
96 | /** |
||
97 | * Получение данных для запроса через API |
||
98 | * |
||
99 | * @param string $method Http-метод запроса |
||
100 | * @param array $params Параметры для формирования урла |
||
101 | * |
||
102 | * @return array |
||
103 | * |
||
104 | * @throws Exception |
||
105 | */ |
||
106 | 5 | public function getUrl($method, array $params = array()) |
|
149 | |||
150 | /** |
||
151 | * Общая статистика чтения журналов |
||
152 | * |
||
153 | * @param string $groupBy Группировка ('day|month|year') |
||
154 | * @param string $periodFrom Период с (формат Y-m-d, например 2017-10-01) |
||
155 | * @param string $periodTo Период с (формат Y-m-d, например 2017-11-01) |
||
156 | * |
||
157 | * @return mixed |
||
158 | * |
||
159 | * @throws Exception |
||
160 | */ |
||
161 | 1 | View Code Duplication | public function getJournalsViewsStatistics($groupBy, $periodFrom, $periodTo) |
162 | { |
||
163 | 1 | return $this->client->getResponse( |
|
164 | 1 | $this->getUrl(__FUNCTION__), |
|
165 | array( |
||
166 | 1 | 'group_by' => $groupBy, |
|
167 | 1 | 'period_range_from' => $periodFrom, |
|
168 | 1 | 'period_range_to' => $periodTo, |
|
169 | ) |
||
170 | )['data']; |
||
171 | } |
||
172 | |||
173 | /** |
||
174 | * Статистика посещаемости |
||
175 | * |
||
176 | * @param string $groupBy Группировка ('day|month|year') |
||
177 | * @param string $periodFrom Период с (формат Y-m-d, например 2017-10-01) |
||
178 | * @param string $periodTo Период с (формат Y-m-d, например 2017-11-01) |
||
179 | * |
||
180 | * @return mixed |
||
181 | * |
||
182 | * @throws Exception |
||
183 | */ |
||
184 | 1 | View Code Duplication | public function getUsersVisitsStatistics($groupBy, $periodFrom, $periodTo) |
185 | { |
||
186 | 1 | return $this->client->getResponse( |
|
187 | 1 | $this->getUrl(__FUNCTION__), |
|
188 | array( |
||
189 | 1 | 'group_by' => $groupBy, |
|
190 | 1 | 'period_range_from' => $periodFrom, |
|
191 | 1 | 'period_range_to' => $periodTo, |
|
192 | ) |
||
193 | )['data']; |
||
194 | } |
||
195 | |||
196 | /** |
||
197 | * Доступные пакеты книг |
||
198 | * |
||
199 | * @return mixed |
||
200 | * |
||
201 | * @throws Exception |
||
202 | */ |
||
203 | 1 | public function getAvailablePackets() |
|
207 | |||
208 | /** |
||
209 | * Доступные книги в пакете |
||
210 | * |
||
211 | * @param int $pdKey Идентификатор пакета |
||
212 | * |
||
213 | * @return mixed |
||
214 | * |
||
215 | * @throws Exception |
||
216 | */ |
||
217 | 1 | public function getAvailableBooks($pdKey) |
|
221 | |||
222 | /** |
||
223 | * Доступные журналы |
||
224 | * |
||
225 | * @return mixed |
||
226 | * |
||
227 | * @throws Exception |
||
228 | */ |
||
229 | public function getAvailableJournals() |
||
233 | } |
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.