Комментарии 17
Какая-то чушь, начиная от того "кто это писал?"(вы что, VCS не пользуетесь?) и до выпиливания "ненужных базовых классов", после которого остались какие-то ошмётки, всё ещё зависящие от этих классов. Сначала думал, что это какой-то пхп, где такое возможно, но нет, это джава, развитые инструменты разработки, рефакторинга. Как вообще можно было довести проект до такого состояния?
вы что, VCS не пользуетесь?
Важный момент, о котором стоило упомянуть: историю коммитов несколько лет назад схлопнули — она была большой, глючил поиск по коммитам, предстоял глобальный рефакторинг всего, и коллеги решили, что так надо. Везде осталось только имя разработчика, который это сделал. Весь код, о котором говорится в статье, был написан до этого момента, и с тех пор практически не затрагивался.
ошмётки, всё ещё зависящие от этих классов
Модуль очень долго был отключён, не запускался и не тестировался — ни на ci, ни где-либо ещё. Остальной проект уже какое-то время счастливо жил на новых базовых классах, а рефакторинг модуля, который неизвестно когда понадобится и неизвестно, будет ли жить, тянул дополнительные издержки, и мы его откладывали.
Но вы правы в одном: у пхп-разрабочиков и правда бывает куда жестче в плане легаси)
А так некоторые проекты переживали три перехода в разные VCS :-)
У вас разве не все модули заведены в один проект? И не все разом собираются на ci?
У вас разве не все модули заведены в один проект? И не все разом собираются на ci?
Да, всё так, но конкретный модуль был отключён.
Блин, вы этим постом и комментариями в корпблоге капец антирекламу делаете:) с такими процессами я бы 100 раз подумал прежде чем к вам идти.
Процессы не рождаются сами по себе, они строятся людьми. Сейчас ребятам удалось построить процессы так, что они могут развивать линейку приложений, пилить фичи и при этом в сроки попадать (даже при учете таких ситуаций). Они молодцы в этом плане — просто вы, возможно, не видите этого и не захотите об этом читать.
А как у вас в команде процессы построены, кстати?
Единственным видом комментариев был копирайт. И благодаря ему я знала, кого хочу убить раньше, чем до меня доберутся те, кто вынужден поддерживать мой код на прошлых местах работы)
Никогда не ставлю копирайт в своих исходниках по работе. Ибо точно знаю, что с некой вероятностью через неделю за код будет просто стыдно. А через год мучительно стыдно.
А иногда стыдно сразу и приходится оставлять комментарий типа:
// залипуха жуткая. Но сроки жмут… надеюсь никогда не придется этот кусок развивать. Извините.
Увы… жизнь…
Но, пальцем иногда хочется потыкать в копираторов с гордостью публикующих свое ФИО в коде.
Например крипто библиотека одобренная ФСБ(!) где данные в TCP/IP летают открыто, а пароль(!!!) для "аутентификации" (!) запроса на подпись/шифрование отправляется вместе с данными "зашифрованный" XOR по константному байту (!!!).
Ой… надеюсь никогда до такого позора не докачусь. Но на всякий случай свой копирайт не вставляю :)
Это не я! История одного рефакторинга