Що таке "функція заздрості", і чому це вважається запахом коду?

Це запитання в ТА говорить про те, щоб виправити те, що, на думку ОП, є кодом заздрості . Ще один приклад, коли я бачив, як цитується ця витончена фраза, - це нещодавно відповідь тут у програмістах. Хоча я крапля в коментарі до цієї відповіді з проханням про інформації , яку я думав , що це було б загальної допомогу програмістам наступних Q & A , щоб зрозуміти , що мається на увазі під терміном ознака заздрості . Будь ласка, не соромтесь редагувати додаткові теги, якщо вважаєте за потрібне.

Заздрість до функцій - це термін, який використовується для опису ситуації, в якій один об'єкт потрапляє на поля іншого об'єкта для того, щоб здійснити якесь обчислення або прийняти рішення, а не просити об’єкт зробити сам обчислення.

Як тривіальний приклад розглянемо клас, що представляє прямокутник. Користувачеві прямокутника, можливо, доведеться знати його площу. Програміст може викрити widthі heightполя, а потім зробити обчислення поза Rectangleкласом. З іншого боку , Rectangleможе тримати widthі heightполе приватними і забезпечує getAreaметод. Це, мабуть, кращий підхід.

Проблема з першою ситуацією, і причина, яку вона вважає кодовим запахом, полягає в тому, що вона порушує інкапсуляцію.

Як правило, кожного разу, коли ви широко використовуєте поля з іншого класу для виконання будь-якої логіки чи обчислень, розгляньте можливість переміщення цієї логіки до методу на самому класі.

Можлива ситуація, коли нормально широко використовувати інший метод класу / структура - коли ваш клас / структура є контейнером для даних. Зазвичай з цими даними можна обійтися без зовнішнього контексту.

Такі класи все ще можуть містити деяку внутрішню логіку, але частіше вони використовуються як контейнери:

class YourUid {
 public:
  YourUid(int id_in_workplace_, int id_in_living_place_, DB* FBI_database, int id_in_FBI_database);
  bool IsInvalidWorker() const { return id_in_workplace == consts::invalid_id_in_workplace; }
  bool CanMessWith() const { return !FBI_database_.is_cool(id_in_FBI_database_); }
  int id_in_workplace;
  int id_in_living_place;
 private:
  int id_in_FBI_database_;
  const DB* FBI_database_;
};

@jhewlett у своїй відповіді посилається на цю статтю, щоб довести, що вам не слід широко використовувати інших членів класу, але є інша ситуація з кодовим запахом, описана там із моїми прикладами:

Довгий список параметрів. Обмежте кількість потрібних параметрів у заданому методі або використовуйте об’єкт для об'єднання параметрів.