Логотип ConstructiCat
CodeBust.
Навигация по разделу ▾

Параметризация метода.

Проблема

Несколько методов выполняют похожие действия, которые отличаются только какими-то внутренними значениями, числами или операциями.

Решение

Объедините все эти методы в один с параметром, в который будет передаваться отличающееся значение.

##Пример

До
void tenPercentRaise() {
  salary *= 1.1;
}
void fivePercentRaise() {
  salary *= 1.05;
}
После
void raise(double factor) {
  salary *= 1 + factor;
}

##Зачем делать рефакторинг

Если у вас есть схожие методы, скорей всего, в них присутствует дублирующий код со всеми вытекающими недостатками.

Кроме того, если вам нужно будет добавить ещё одну вариацию функциональности, вам придётся создавать ещё один метод. Вместо этого можно бы было запустить существующий метод с другим параметром.

##Как делать рефакторинг

  1. Создайте новый метод с параметром и поместите в него общий для всех методов код, применяя извлечение метода. Обратите внимание, иногда общей оказывается только определённая часть методов. В этом случае рефакторинг сведётся к извлечению только этой общей части в новый метод.

  2. Отличающееся значение замените параметром в коде нового метода.

  3. Для каждого старого метода найдите места, где они вызываются, и поменяйте их вызовы на вызовы нового метода с параметром. После чего старый метод можно удалить.