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

Встраивание переменной.

Проблема

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

Решение

Замените обращения к переменной этим выражением.

##Пример

До
boolean hasDiscount(Order order) {
  double basePrice = order.basePrice();
  return basePrice > 1000;
}
После
boolean hasDiscount(Order order) {
  return order.basePrice() > 1000;
}

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

Встраивание локальной переменной почти всегда используется как часть замены переменной вызовом метода или для облегчения извлечения метода.

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

  • Сам по себе данный рефакторинг не несёт почти никакой пользы. Тем не менее, если переменной присваивается результат выполнения какого-то метода, у вас есть возможность немного улучшить читабельность программы, избавившись от лишней переменной.

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

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

  2. Удалите объявление переменной и строку присваивания ей значения.

← Предыдущая статья
Встраивание метода
Следующая статья →
Удаление присваиваний параметрам