Бывает Провожу сегодня ревью старого проекта, который нам передали на поддержку. Листаю один из модулей и у меня начина
Провожу сегодня ревью старого проекта, который нам передали на поддержку. Листаю один из модулей и у меня начинает натурально подгорать. Вслух комментирую на весь кабинет: «Господи, ну кто это писал?! Костыль на костыле, логика сломана, синяя изолента вместо архитектуры. Как это вообще в прод пропустили?!»
Решаю найти виновника, чтобы лично спросить, чем он думал. Делаю git blame (проверку авторства строк)...
Смотрю в гит-лог: мой ник, коммит 6 месяцев назад...
Посидел, подумал, внимательно перечитал ещё раз. Ну, в принципе, если посмотреть под другим углом, то архитектурно всё обосновано. Решение гибкое, компилятор не ругается, работает — значит, не трогаем. Контекст тогда сложный был. Я художник, я так видел.
Автор на Пикабу: GV67
Комментарии: pikabu.ru/link/7IB-_BTPzJ