Лекция №14. Рефакторинг (ч.3).

1. Решение задач обобщения.
1.1. Подъем поля (Pull up field).
1.2. Подъем метода (Pull up method).
1.3. Спуск метода (Push down method).
1.4. Спуск поля (Push down field).
1.5. Извлечение подкласса (Extract subclass).
1.6. Извлечение суперкласса (Extract superclass).
1.7. Извлечение интерфейса (Extract interface).
1.8. Свертывание иерархии (Collapse hierarchy).
1.9. Замена наследования делегированием (Replace Inheritance with delegation).
1.10. Замена делегирования наследованием (Replace delegation with inheritance).
2. Раздувальщики (сode smells).
2.1. Длинный метод (Long method).
2.2. Большой класс (Large class).
2.3. Одержимость элементарными типами (Primitive obsession).
2.4. Длинный список параметров (Long parameter list).
2.5. Группы данных (Data clumps).
3. Утяжелители изменений (code smells).
3.1. Расходящиеся модификации (Divergent change).
3.2. Стрельба дробью (Shotgun surgery).
4. Замусориватели (code smells).
4.1. Комментарии (Comments).
4.2. Дублирование кода (Duplicate code).
4.3. Мертвый код (Dead code).
4.4. Теоретическая общность (Speculate generality).
5. Опутыватели связями (code smells).
5.1. Завистливые функции (Feature envy).
5.2. Неуместная близость (Inappropriate intimacy).
5.3. Цепочка вызовов (Message chains).
5.4. Посредник (Middle man).

Transcript

1. ОБЪЕКТНО- ОРИЕНТИРОВАННОЕ ПРОГРАММИРОВАНИЕ Лекция № 2 / 14
 11.12.2018 г.
2. ПРИЕМЫ РЕФАКТОРИНГА • Составление методов. • Перемещение функций между объектами. • Организация данных. • Упрощение условных выражений. • Упрощение вызовов методов. • Решение задач обобщения.
3. РЕШЕНИЕ ЗАДАЧ ОБОБЩЕНИЯ
4. ПОДЪЕМ ПОЛЯ (PULL UP FIELD) Два класса имеют одно и тоже поле. Переместите поле в базовый класс, убрав его из подклассов.
5. ПОДЪЕМ ПОЛЯ (PULL UP FIELD)
6. ПРИЧИНЫ РЕФАКТОРИНГА Подклассы развивались независимо друг от друга. Это привело к созданию одинаковых (или очень похожих) полей и методов.
7. ДОСТОИНСТВА • Убирает дублирование полей в подклассах. • Облегчает дальнейший перенос дублирующих методов из подклассов в базовый класс, если они есть.
8. ПОДЪЕМ МЕТОДА (PULL UP METHOD) Подклассы имеют методы, которые делают схожую работу. Сделайте методы идентичными, а затем переместите их в базовый класс.
9. ПОДЪЕМ МЕТОДА (PULL UP METHOD)
10. ПРИЧИНЫ РЕФАКТОРИНГА Подклассы развивались независимо друг от друга. Это привело к созданию одинаковых (или очень похожих) полей и методов.
11. ДОСТОИНСТВА Убирает дублирование полей в подклассах. Проще модифицировать метод.
12. СПУСК МЕТОДА (PUSH DOWN METHOD) Метод в базовом классе, используется только одним или несколькими подклассами. Переместите этот метод в подклассы.
13. СПУСК МЕТОДА (PUSH DOWN METHOD)
14. ПРИЧИНЫ РЕФАКТОРИНГА Метод перестал быть уникальным. Например, после извлечения (или удаления) части функциональности из иерархии классов.
15. ДОСТОИНСТВА Улучшает связность внутри классов. Метод находится там, где вы ожидаете его увидеть.
16. СПУСК ПОЛЯ (PUSH DOWN FIELD) Поле используется только в некоторых подклассах. Переместите поле в эти подклассы.
17. СПУСК ПОЛЯ (PUSH DOWN FIELD)
18. ПРИЧИНЫ РЕФАКТОРИНГА Поле перестало быть уникальным. Например, после извлечения (или удаления) части функциональности из иерархии классов.
19. ДОСТОИНСТВА • Улучшает связность внутри классов. Поле находится там, где оно используется. • Возможность развивать поля независимо друг от друга.
20. ИЗВЛЕЧЕНИЕ ПОДКЛАССА (EXTRACT SUBCLASS) Класс имеет фичи, которые используются только в определённых случаях. Создайте подкласс и используйте его в этих случаях.
21. ИЗВЛЕЧЕНИЕ ПОДКЛАССА (EXTRACT SUBCLASS)
22. ПРИЧИНЫ РЕФАКТОРИНГА В основном классе находятся методы и поля для реализации какого-то редкого случая использования класса.
23. ДОСТОИНСТВА • Создать подкласс довольно легко и быстро. • Можно выделить несколько разных подклассов, если основной класс реализует несколько подобных особых случаев.
24. НЕДОСТАТКИ Несмотря на всю очевидную простоту, Наследование может завести в тупик, если придётся выделить несколько различных иерархий классов.
25. ИЗВЛЕЧЕНИЕ СУПЕРКЛАССА (EXTRACT SUPERCLASS) Есть два класса с общими полями и методами. Создайте для них базовый класс и перенесите туда одинаковые поля и методы.
26. ИЗВЛЕЧЕНИЕ СУПЕРКЛАССА (EXTRACT SUPERCLASS)
27. ПРИЧИНЫ РЕФАКТОРИНГА Дублирование кода.
28. ДОСТОИНСТВА Убирает дублирование кода. Все находится в одном месте.
29. КОГДА НЕЛЬЗЯ ПРИМЕНИТЬ Нельзя применить к классам, которые уже имеют базовый класс.
30. ИЗВЛЕЧЕНИЕ ИНТЕРФЕЙСА (EXTRACT INTERFACE) В двух классах часть интерфейса оказалась общей. Выделите эту общую часть в интерфейс.
31. ИЗВЛЕЧЕНИЕ ИНТЕРФЕЙСА (EXTRACT INTERFACE)
32. ПРИЧИНЫ РЕФАКТОРИНГА Требуется описать операции, которые выполняют несколько классов.
33. СВЁРТЫВАНИЕ ИЕРАРХИИ (COLLAPSE HIERARCHY) Есть некая иерархия классов, в которой подкласс мало чем отличается от базового класса. Слейте подкласс и базовый класс воедино.
34. СВЁРТЫВАНИЕ ИЕРАРХИИ (COLLAPSE HIERARCHY)
35. ПРИЧИНЫ РЕФАКТОРИНГА Развитие программы привело к тому, что подкласс и базовый класс стали очень мало отличаться друг от друга.
36. ДОСТОИНСТВА • Уменьшает сложность программы. • Навигация по коду становится проще.
37. ЗАМЕНА НАСЛЕДОВАНИЯ ДЕЛЕГИРОВАНИЕМ (REPLACE INHERITANCE WITH DELEGATION) Есть подкласс, который использует только часть методов базового класса или не хочет наследовать его данные. Создайте поле и поместите в него объект суперкласса, делегируйте выполнение, уберите наследование.
38. ЗАМЕНА НАСЛЕДОВАНИЯ ДЕЛЕГИРОВАНИЕМ (REPLACE INHERITANCE WITH DELEGATION)
39. ПРИЧИНЫ РЕФАКТОРИНГА • Наследование возникло только ради объединения общего кода. • Подкласс использует только часть методов базового класса.
40. ДОСТОИНСТВА • Класс не содержит лишних методов, которые достались ему в наследство от суперкласса. • В поле-делегат можно подставлять разные объекты, имеющие различные реализации функциональности.
41. НЕДОСТАТКИ Приходится писать очень много простых делегирующих методов.
42. ЗАМЕНА ДЕЛЕГИРОВАНИЯ НАСЛЕДОВАНИЕМ (REPLACE DELEGATION WITH INHERITANCE) Класс содержит множество простых делегирующих методов ко всем методам другого класса. Сделайте класс наследником делегата, после чего делегирующие методы потеряют смысл.
43. ЗАМЕНА ДЕЛЕГИРОВАНИЯ НАСЛЕДОВАНИЕМ (REPLACE DELEGATION WITH INHERITANCE)
44. ПРИЧИНЫ РЕФАКТОРИНГА Делегирование является более гибким подходом, чем наследование. НО применение делегирования перестаёт быть выгодным, если вы делегируете действия только одному классу, причём всем его публичным методам.
45. ДОСТОИНСТВА Уменьшает количество кода.
46. КОГДА НЕЛЬЗЯ ПРИМЕНИТЬ • Если класс содержит делегирование только к части публичных методов класса-делегата. • Если класс имеет родителя.
47. CODE SMELLS
48. РАЗДУВАЛЬЩИКИ
49. ДЛИННЫЙ МЕТОД (LONG METHOD) Метод содержит слишком большое число строк кода. Длина метода более десяти строк должна начинать вас беспокоить.
50. БОЛЬШОЙ КЛАСС (LARGE CLASS) Класс содержит множество полей/методов/строк кода.
51. ОДЕРЖИМОСТЬ ЭЛЕМЕНТАРНЫМИ ТИПАМИ (PRIMITIVE OBSESSION) • Использование элементарных типов вместо маленьких объектов для небольших задач. • Использование строковых констант в качестве названий полей в массивах.
52. ДЛИННЫЙ СПИСОК ПАРАМЕТРОВ (LONG PARAMETER LIST) Количество параметров метода больше трёх-четырёх.
53. ГРУППЫ ДАННЫХ (DATA CLUMPS) В разных частях кода встречаются одинаковые группы переменных.
54. УТЯЖЕЛИТЕЛИ ИЗМЕНЕНИЙ
55. РАСХОДЯЩИЕСЯ МОДИФИКАЦИИ (DIVERGENT CHANGE) При внесении изменений в класс приходится изменять большое число различных методов.
56. СТРЕЛЬБА ДРОБЬЮ (SHOTGUN SURGERY) При выполнении любых модификаций приходится вносить множество мелких изменений в большое число классов.
57. ЗАМУСОРИВАТЕЛИ
58. КОММЕНТАРИИ (COMMENTS) Метод содержит множество поясняющих комментариев.
59. ДУБЛИРОВАНИЕ КОДА (DUPLICATE CODE) Два фрагмента кода выглядят почти одинаковыми.
60. МЁРТВЫЙ КОД (DEAD CODE) Переменная, параметр, поле, метод или класс больше не используются.
61. ТЕОРЕТИЧЕСКАЯ ОБЩНОСТЬ (SPECULATIVE GENERALITY) Переменная, параметр, поле, метод или класс больше не используются.
62. ОПУТЫВАТЕЛИ СВЯЗЯМИ
63. ЗАВИСТЛИВЫЕ ФУНКЦИИ (FEATURE ENVY) Метод обращается к данным другого объекта чаще, чем к собственным данным.
64. НЕУМЕСТНАЯ БЛИЗОСТЬ (INAPPROPRIATE INTIMACY) Один класс использует служебные поля и методы другого класса.
65. ЦЕПОЧКА ВЫЗОВОВ (MESSAGE CHAINS) В коде присутствуют большие цепочки вызовов.
66. ПОСРЕДНИК (MIDDLE MAN) Если класс выполняет одно действие – делегирует работу другому классу – стоит задуматься, зачем он вообще существует.
67. КОНЕЦ ТРЕТЬЕЙ ЧАСТИ