Проблема: Стоит использовать в том случае, когда тело метода очевиднее самого метода.

Решение: Замените вызовы метода его содержимым и удалите сам метод.

Основаная причина — тело метода состоит из простого делегирования к другому методу. Само по себе такое делегирование — не проблема. Но если таких методов довольно много, становится очень легко в них запутаться.

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

Достоинства

(плюс) Минимизируя количество бесполезных методов, мы уменьшаем общую сложность кода.

Чек-лист проведения рефакторинга

  • Убедитесь, что метод не переопределяется в подклассах. Если он переопределяется, воздержитесь от рефакторинга.
  • Найдите все вызовы метода. Замените эти вызовы содержимым метода.
  • Удалите метод.
Tags

Нет комментариев

Добавить комментарий

Ваш адрес email не будет опубликован. Обязательные поля помечены *

Этот сайт использует Akismet для борьбы со спамом. Узнайте, как обрабатываются ваши данные комментариев.