Артему дали задачу на полдня. Менеджер сказал: "Там маленькая правка. Просто добавь еще одно условие". Артем открыл код и увидел метод на 280 строк. Внутри: скидки, статусы, проверки клиента, ручная работа с датами и один комментарий трехлетней давности: temporary fix. Если добавить условие быстро, релиз успевает. Если начать разбираться, задача перестает быть "маленькой". Тимлид занят. QA уже ждет билд. Менеджер пишет: "ну что там, долго?" Артем понимает: технически он может вставить еще один if. Но он не уверен, что не сломает старую бизнес-логику. Что бы вы ему посоветовали? Быстро сделать минимальный фикс, остановить задачу до анализа или предложить третий вариант? #Java #SoftwareEngineering #CodeReview #JuniorDeveloper
Артему дали задачу на полдня.
Менеджер сказал:
"Там маленькая правка. Просто добавь еще одно условие".
Артем открыл код и увидел метод на 280 строк. Внутри: скидки, статусы, проверки клиента, ручная работа с датами и один комментарий трехлетней давности: temporary fix.
Если добавить условие быстро, релиз успевает.
Если начать разбираться, задача перестает быть "маленькой".
Тимлид занят. QA уже ждет билд. Менеджер пишет: "ну что там, долго?"
Артем понимает: технически он может вставить еще один if. Но он не уверен, что не сломает старую бизнес-логику.
Что бы вы ему посоветовали?
Быстро сделать минимальный фикс, остановить задачу до анализа или предложить третий вариант?
#Java #SoftwareEngineering #CodeReview #JuniorDeveloper
