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

Спуск метода.

Проблема

Поведение, реализованное в суперклассе, используется только одним или несколькими подклассами.

Решение

Переместите это поведение в подклассы.

##Пример

До
class Employee {
  double salesQuota() { return quota; }
}
class Salesman extends Employee {}
class Engineer extends Employee {}
После
class Employee {}
class Salesman extends Employee {
  double salesQuota() { return quota; }
}
class Engineer extends Employee {}

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

Метод, который планировали сделать универсальным для всех классов, по факту используется только в одном подклассе. Такая ситуация может возникнуть, когда планируемые фичи так и не были реализованы.

Кроме того, такая ситуация может возникнуть после извлечения (или удаления) части функциональности из иерархии классов, после которого метод остался используемым только в одном подклассе.

Если вы видите, что метод необходим более чем одному подклассу (но не всем), возможно, стоит создать промежуточный подкласс и переместить метод в него. Это позволит избежать дублирования кода, которое возникло бы при спуске метода во все подклассы.

##Преимущества

  • Улучшает связность внутри классов. Метод находится там, где вы ожидаете его увидеть.

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

  1. Объявите метод в подклассе и скопируйте его код из суперкласса.

  2. Удалите метод из суперкласса.

  3. Найдите все места, где используется метод, и убедитесь, что он вызывается из нужного подкласса.