Выбираем первый язык программирования
Да
т
У вас есть друзья?
i
Да
Т
Хотите много зарабатывать?
jL
Да
ш
Вы тупой?
т.
Т
Вы насмотрелись уроков ХАУДИ ХО?
/Г
Да
7
Python
Вам
г~ нравится 1
1 Windows?
Нет
Fortran
А они вам нужны?
Они тоже РНР
тупые?
Да
т
КОД ПЕРЕСТАЁТ РАБОТАТЬ
*убираешь комментарий*
КОД ПЕРЕСТАЁТ РАБОТАТЬ
Old shitty code was here
*/
ТЫ ПЕРЕСТАЁШЬ РАБОТАТЬ (в этой компании)
Ну у тебя же на то и ситема контроля версий, чтобы в случае чего посмотреть изменения и откатиться до нужной версии, а если еще по человечески вести гит а ни лить все в мастер с именами some commit, some fix, another fix, то и поиск изменения будет довольно быстрым
А не было бы там этого закомменченного куска - его бы при ремонте бага написали, оно бы сломало другую фичу, и потом бы всё равно нужно было чинить всё вместе, только была бы зря потрачена куча времеми.
А уж если это хотфикс, то оно бы ещё и могло в продакшен улететь со сломанной другой фичей и натворить крупных проблем.
Конечно, если обитать в каком-нибудь пет-прожекте, у которого полтора пользователя и три с половиной комита в гите, то можно и чистить закоменченный код и вообще любой обсессивно-компульсивной хуйнёй заниматься. Но если цена сломанной/отсутствующей функциональности в проде или задержки хотфикса - многие килобаксы, то извини, пока к тебе придёт понимание, компания может и уйти с рынка.
Но вычистить их в финальном варианте, в финальном коммите.
Конечно ревьювить каждый пулл-реквест времени банально не хватит, но помочь с решением как сделать надо если просят.
Поддерживать не мне, но замедление работы из-за тупого и ненужного технического долга это мой проеб. Да и не факт что Васян который накоммитил говно и помнит как оно работает все еще будет с нами, когда через три года это говно придется поправить.
А так написал тесты, провел, нашел ошибку, расписал план устранения и вперёд, всем спасибо
Потому что за легаси готовы платить больше, чем за новый код.
Что не заменяет, а дополняет систему контроля версий: не нужно вспоминать в каком из тысячи коммитов есть данный блок если что-то нужно срочно проверить.
Во-вторых поиск по истории гита в командной строке очень сильная штука, попробуйте. Можно искать и по кускам удаленных строчек кода.
> Во-первых нормальное именование коммитов.
"здравствуй дневник, я дописал I2C, железо проверено и работает, так что отладочная процедура сканирования всего диапазона адресов на шине больше не нужна. Тот кусочек кода который заполняет датафлеш паттерном 0x55AA - тоже. Если попадется устройство с проблемой на шине I2C или с подозрением на битую память - ищи дебажный код тут".
> Можно искать и по кускам удаленных строчек кода.
>> не нужно вспоминать
Ну и у нас на предприятии используется subversion.
legacy_func(shit)