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:
Complex classes like FirstNamesInflection often do a lot of different things. To break such a class down, we need to identify a cohesive component within that class. A common approach to find such a component is to look for fields/methods that share the same prefixes, or suffixes. You can also have a look at the cohesion graph to spot any un-connected, or weakly-connected components.
Once you have determined the fields that belong together, you can apply the Extract Class refactoring. If the component makes sense as a sub-class, Extract Subclass is also a candidate, and is often faster.
While breaking up the class, it is a good idea to analyze how other classes use FirstNamesInflection, and based on these observations, apply Extract Interface, too.
1 | <?php |
||
10 | class FirstNamesInflection extends \morphos\NamesInflection implements Cases |
||
11 | { |
||
12 | use RussianLanguage, CasesHelper; |
||
13 | |||
14 | /** |
||
15 | * @var string[][] |
||
16 | * @phpstan-var array<string, array<string, string>> |
||
17 | */ |
||
18 | protected static $exceptions = [ |
||
19 | 'лев' => [ |
||
20 | self::IMENIT => 'Лев', |
||
21 | self::RODIT => 'Льва', |
||
22 | self::DAT => 'Льву', |
||
23 | self::VINIT => 'Льва', |
||
24 | self::TVORIT => 'Львом', |
||
25 | self::PREDLOJ => 'Льве', |
||
26 | ], |
||
27 | 'павел' => [ |
||
28 | self::IMENIT => 'Павел', |
||
29 | self::RODIT => 'Павла', |
||
30 | self::DAT => 'Павлу', |
||
31 | self::VINIT => 'Павла', |
||
32 | self::TVORIT => 'Павлом', |
||
33 | self::PREDLOJ => 'Павле', |
||
34 | ] |
||
35 | ]; |
||
36 | |||
37 | /** @var string[] */ |
||
38 | protected static $menNames = [ |
||
39 | 'абрам', 'аверьян', 'авраам', 'агафон', 'адам', 'азар', 'акакий', 'аким', 'аксён', 'александр', 'алексей', |
||
40 | 'альберт', 'анатолий', 'андрей', 'андрон', 'антип', 'антон', 'аполлон', 'аристарх', 'аркадий', 'арнольд', |
||
41 | 'арсений', 'арсентий', 'артем', 'артём', 'артемий', 'артур', 'аскольд', 'афанасий', 'богдан', 'борис', |
||
42 | 'борислав', 'бронислав', 'вадим', 'валентин', 'валерий', 'варлам', 'василий', 'венедикт', 'вениамин', |
||
43 | 'веньямин', 'венцеслав', 'виктор', 'виген', 'вилен', 'виталий', 'владилен', 'владимир', 'владислав', 'владлен', |
||
44 | 'вова', 'всеволод', 'всеслав', 'вячеслав', 'гавриил', 'геннадий', 'георгий', 'герман', 'глеб', 'григорий', |
||
45 | 'давид', 'даниил', 'данил', 'данила', 'демьян', 'денис', 'димитрий', 'дмитрий', 'добрыня', 'евгений', 'евдоким', |
||
46 | 'евсей', 'егор', 'емельян', 'еремей', 'ермолай', 'ерофей', 'ефим', 'захар', 'иван', 'игнат', 'игорь', |
||
47 | 'илларион', 'иларион', 'илья', 'иосиф', 'казимир', 'касьян', 'кирилл', 'кондрат', 'константин', 'кузьма', |
||
48 | 'лавр', 'лаврентий', 'лазарь', 'ларион', 'лев', 'леонард', 'леонид', 'лука', 'максим', 'марат', 'мартын', |
||
49 | 'матвей', 'мефодий', 'мирон', 'михаил', 'моисей', 'назар', 'никита', 'николай', 'олег', 'осип', 'остап', |
||
50 | 'павел', 'панкрат', 'пантелей', 'парамон', 'пётр', 'петр', 'платон', 'потап', 'прохор', 'роберт', 'ростислав', |
||
51 | 'савва', 'савелий', 'семён', 'семен', 'сергей', 'сидор', 'спартак', 'тарас', 'терентий', 'тимофей', 'тимур', |
||
52 | 'тихон', 'ульян', 'фёдор', 'федор', 'федот', 'феликс', 'фирс', 'фома', 'харитон', 'харлам', 'эдуард', |
||
53 | 'эммануил', 'эраст', 'юлиан', 'юлий', 'юрий', 'яков', 'ян', 'ярослав', |
||
54 | ]; |
||
55 | |||
56 | /** @var string[] */ |
||
57 | protected static $womenNames = [ |
||
58 | 'авдотья', 'аврора', 'агата', 'агния', 'агриппина', 'ада', 'аксинья', 'алевтина', 'александра', 'алёна', |
||
59 | 'алена', 'алина', 'алиса', 'алла', 'альбина', 'амалия', 'анастасия', 'ангелина', 'анжела', 'анжелика', 'анна', |
||
60 | 'антонина', 'анфиса', 'арина', 'белла', 'божена', 'валентина', 'валерия', 'ванда', 'варвара', 'василина', |
||
61 | 'василиса', 'вера', 'вероника', 'виктория', 'виола', 'виолетта', 'вита', 'виталия', 'владислава', 'власта', |
||
62 | 'галина', 'глафира', 'дарья', 'диана', 'дина', 'ева', 'евгения', 'евдокия', 'евлампия', 'екатерина', 'елена', |
||
63 | 'елизавета', 'ефросиния', 'ефросинья', 'жанна', 'зиновия', 'злата', 'зоя', 'ивонна', 'изольда', 'илона', 'инга', |
||
64 | 'инесса', 'инна', 'ирина', 'ия', 'капитолина', 'карина', 'каролина', 'кира', 'клавдия', 'клара', 'клеопатра', |
||
65 | 'кристина', 'ксения', 'лада', 'лариса', 'лиана', 'лидия', 'лилия', 'лина', 'лия', 'лора', 'любава', 'любовь', |
||
66 | 'людмила', 'майя', 'маргарита', 'марианна', 'мариетта', 'марина', 'мария', 'марья', 'марта', 'марфа', 'марьяна', |
||
67 | 'матрёна', 'матрена', 'матрона', 'милена', 'милослава', 'мирослава', 'муза', 'надежда', 'настасия', 'настасья', |
||
68 | 'наталия', 'наталья', 'нелли', 'ника', 'нина', 'нинель', 'нонна', 'оксана', 'олимпиада', 'ольга', 'пелагея', |
||
69 | 'полина', 'прасковья', 'раиса', 'рената', 'римма', 'роза', 'роксана', 'руфь', 'сарра', 'светлана', 'серафима', |
||
70 | 'снежана', 'софья', 'софия', 'стелла', 'степанида', 'стефания', 'таисия', 'таисья', 'тамара', 'татьяна', |
||
71 | 'ульяна', 'устиния', 'устинья', 'фаина', 'фёкла', 'фекла', 'феодора', 'хаврония', 'христина', 'эвелина', |
||
72 | 'эдита', 'элеонора', 'элла', 'эльвира', 'эмилия', 'эмма', 'юдифь', 'юлиана', 'юлия', 'ядвига', 'яна', |
||
73 | 'ярослава', |
||
74 | ]; |
||
75 | |||
76 | /** @var string[] */ |
||
77 | protected static $immutableNames = [ |
||
78 | 'николя', |
||
79 | ]; |
||
80 | |||
81 | /** |
||
82 | * Checks if name is mutable |
||
83 | * @param string $name |
||
84 | * @param null|string $gender |
||
85 | * @return bool |
||
86 | */ |
||
87 | 716 | public static function isMutable($name, $gender = null) |
|
127 | |||
128 | /** |
||
129 | * @param string $name |
||
130 | * @return string |
||
131 | */ |
||
132 | 510 | public static function detectGender($name) |
|
185 | |||
186 | /** |
||
187 | * @param string $name |
||
188 | * @param null|string $gender |
||
189 | * @return string[] |
||
190 | * @phpstan-return array<string, string> |
||
191 | */ |
||
192 | 140 | public static function getCases($name, $gender = null) |
|
239 | |||
240 | /** |
||
241 | * @param string $name |
||
242 | * @return string[]|null |
||
243 | * @phpstan-return array<string, string>|null |
||
244 | */ |
||
245 | 84 | protected static function getCasesMan($name) |
|
318 | |||
319 | /** |
||
320 | * @param string $name |
||
321 | * @return string[]|null |
||
322 | * @phpstan-return array<string, string>|null |
||
323 | */ |
||
324 | 26 | protected static function getCasesWoman($name) |
|
371 | |||
372 | /** |
||
373 | * @param string $name |
||
374 | * @param string $case |
||
375 | * @param null|string $gender |
||
376 | * @return string |
||
377 | * @throws \Exception |
||
378 | */ |
||
379 | 52 | public static function getCase($name, $case, $gender = null) |
|
385 | } |
||
386 |
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.