🦋Рефакторинг

Разумные причины для выполнения рефакторинга

1. Код повторяется

2. Метод слишком велик

3. Цикл слишком велик или слишком глубоко вложен в другие циклы

4. Класс имеет плохую связность

5. Интерфейс класса не формирует согласованную абстракцию

6. Метод принимает слишком много параметров

7. Отдельные части класса изменяются независимо от других частей

8. При изменении программы требуется параллельно изменять несколько классов

9. Родственные элементы данных, используемые вместе, не организованы в классы или структуры

10. Метод использует больше элементов другого класса, чем своего собственного

11. Поле в классе используется другим классом чаще, чем своим собственным

12. Объект-посредник ничего не делает

13. Подкласс использует только малую долю методов своих предков

14. Перед вызовом метода выполняется подготовительный код (после вызова метода выполняется код "уборки")

15. Программа содержит код, который может когда-нибудь понадобиться

16. Класс имеет больше одной ответственности. Полезно задать себе вопрос о том, что будет, если удалить часть данных или метод. Какие другие данные или методы при этом потеряют смысл?

17. Если у каждого подкласса на определённом уровне имена начинаются с одних и тех же прилагательных, то, вероятно, стоит разбить одну иерархию на две.

Некоторые запахи

1. Расходящиеся изменения. Если глядя на класс, вы замечаете про себя, что вот эти три метода придётся изменять для каждой новой базы данных, а вот эти четыре метода - при каждом появлении нового финансового требования, то это может означать, что вместо одного класса лучше иметь два.

2. "Стрельба дробью". В этой ситуации следует свести все изменения в один класс, использовав рефакторинги "Перенос метода" и "Перенос поля". Если подходящего кандидата среди имеющихся классов нет, создайте новый класс. Часто можно применить рефакторинг "Встраивание класса" и поместить целый пакет методов в один класс.

3. Завистливые функции. Если вы столкнулись с методом, вызывающим с полдюжины методов доступа к данным другого объекта, чтобы вычислить некоторое значение, то метод следует перенести в другое место с помощью рефакторинга "Перенос метода". Иногда завистью страдает только часть метода; в таком случае сначала примените к этой части рефакторинг "Извлечение метода", а уже затем рефакторинг "Перенос метода".

4. Группы данных. Данные, встречающиеся совместно, имеет смысл превращать в отдельный класс или структуру (например, StartDate и EndDate). Хорошая проверка заключается в том, чтобы удалить одно из значений данных и посмотреть, сохранят ли при этом смысл остальные данные. Если нет, то это верный признак того, что данные лучше объединить в один объект.

5. Временное поле. Если какое-то поле корректно только во время работы алгоритма; в других контекстах оно лишь вводит в заблуждение. В таком случае можно применить рефакторинг "Извлечение класса" к переменным и методам, в которых требуется это поле. Новый объект является объектом метода.

Рефакторинг на уровне данных

1. Встраивание выражения в код

2. Замена выражения на вызов метода

3. Введение промежуточной переменной

4. Использование локальной переменной вместо параметра

5. Преобразование кода в класс

6. Преобразование массива в класс

Рефакторинг на уровне отдельных операторов

1. Декомпозиция логического выражения путём ввода переменных с понятным именем

2. Вынесение сложного логического выражения в грамотно названную булеву функцию

3. Возврат из метода сразу после получения ответа вместо установки возвращаемого значения внутри вложенных операторов if-else

Рефакторинг на уровне отдельных методов

1. Извлечение метода из другого метода

2. Встраивание кода метода

3. Преобразование объёмного метода в класс

4. Отделение операций запроса данных от операций изменения данных

5. Разделение метода, поведение которого зависит от полученных параметров

Рефакторинг интерфейсов классов

1. Перемещение метода в другой класс

2. Разделение одного класса на несколько

3. Сокрытие делегата (или, наоборот, удаление посредника)

4. Замена наследования на композицию (или, наоборот, замена композиции на наследование)

5. Объединение суперкласса и подкласса, имеющих очень похожую реализацию