Comments 7
Код - это как рабочее место.
Правило бойскаута говорит: прибирай место, в котором ты был.
Вы же предлагаете, увидев грязь на рабочем месте, развести ещё больше грязи.
Ну и все примеры "low-hanging fruit" в коде заменены на карго-культовые изменения.
В последнее время, если вижу в статье длинные списки, то складывается ощущение, что не обошлось без ChatGPT.
Рефакторинг представляет собой преобразование кода, не влияющее на функциональность программы. Это вопрос, подобный соблюдению гигиены, например, чистке зубов после приема пищи: каждая отдельная процедура не решает какой-либо острой проблемы. Однако, регулярное выполнение этого ритуала в конечном итоге позволяет избежать лишних расходов на стоматологические услуги.
Решение архитектурных задач предполагает не рефакторинг, а улучшения, миграции, модернизации и другие подобные действия, к которым следует подходить совершенно иначе. Если у вас возник дискомфорт, связанный с зубами, пожалуйста, создайте отдельную задачу, чтобы выделить на это необходимое время и решить проблему с помощью специалистов. Не следует пытаться совмещать установку пломбы с походом на байдарках на выходных, но зубная щетка вам, безусловно, там пригодится.
Мне кажется, что ваш пример красивый, но не к месту. Код не обладает теми же свойствами, что и зубы. Видите ли, зубы портятся от проходящей вредной пищи, а код — от проходящих байт нет, как бы вредны они ни были (пусть даже от ставок и казино).
Ну, это я так понимаю тезис Дяди Боба из Clean Code - здесь он имел ввиду чисто гигиенические моменты. Задача бойскаута - убрать в кемпинге мусор, а не перестраивать там всё или ландшафты менять для борьбы с глобальным потеплением. Хотя последнее может быть тоже важно, но это задачи другого уровня.
На практике разница между мусором и архитектурными проблемами зависит от конкретного кода и проекта. Тут надо взвешивать риски и договариваться. Где-то можно безболезненно перекидать тонны кода из одного места в другое, а где-то даже отступы не получится поменять для улучшения читаемости без отдельного запроса на изменение и одобрения от начальства.
Low-hanging fruit