Опис проблеми
У коді розрахунку вартості прибирання використовуються жорстко прописані числові значення (наприклад, множники ціни 1.2, 1.5 або базові тарифи 500), які розкидані прямо всередині умовних операторів if/else. Це вважається запахом коду "Магічні числа" (Magic Numbers) і ускладнює зміну тарифів у майбутньому.
Доказ у коді
Конкретні числові коефіцієнти використовуються у файлі HouseCleaning.php всередині логіки розгалуження варіантів.
Пропозиція щодо вирішення
Винести всі числові тарифи та коефіцієнти в константи класу (наприклад, const BASE_TARIFF = 500;) або передавати їх через конструктор, щоб їх можна було легко змінити в одному місці.
Опис проблеми
У коді розрахунку вартості прибирання використовуються жорстко прописані числові значення (наприклад, множники ціни
1.2,1.5або базові тарифи500), які розкидані прямо всередині умовних операторівif/else. Це вважається запахом коду "Магічні числа" (Magic Numbers) і ускладнює зміну тарифів у майбутньому.Доказ у коді
Конкретні числові коефіцієнти використовуються у файлі HouseCleaning.php всередині логіки розгалуження варіантів.
Пропозиція щодо вирішення
Винести всі числові тарифи та коефіцієнти в константи класу (наприклад,
const BASE_TARIFF = 500;) або передавати їх через конструктор, щоб їх можна було легко змінити в одному місці.