Comments 5
Самое эффективное ревью - это то которое делаешь самостоятельно, на утро следующего дня. За ночь глаз отвыкает от кода и видишь свои ошибки сам. Так что иногда код должен просто "настояться")
Раньше до ИИ было, что изменения в код добавлялись медленнее, и в найтли сборке было легче заметить изменения и найти причину.
Вдобавок людям лень писать повторяющийся код, и они начинают его как-то реорганизовывать и более лаконично писать, а ИИ это чуждо и он может однотипных изменений хоть на тысячу строк сделать. И, к сожалению, как будто теперь вместо простого и организованного кода, его теперь просто больше и он хуже качеством.
Получается что-то вроде трагедии общин - каждому разработчику проще нагенерить код ценой потери качества, но в итоге получается раздутый проект с техдолгом, в котором всё перманентно разломано и все страдают.
P.S. А про слои ревью - полностью согласен. В предыдущей команде можно было влить изменения в мастер и потом создать ревью - было очень удобно. Сейчас требуется обязательный аппрув ревью человеком + долгое и очень нестабильное CI, в сумме это ад. Сначала ждёшь ревью от человека, потом он просит что-нибудь поправить (один слой), потом CI перезапускает тесты и зачастую падает. И эти тесты - ещё один слой, причём там бывают и тесты нестабильные, и просто поломанные кем-то другим, когда только ребейз поможет. Всякие мелочи даже исправлять не хочется, код пишется за минуты и потом тратятся дни, чтобы его влить.
Еще можно посмотреть на свои творения недели через две, а то и через месяц.
Сразу становится понятно, как нужно было назвать переменные, по каким сервисам разбить код, какой алгоритм оптимальнее для решения поставленной задачи, да и по поводу постановки возникают интересные мысли (а постановщик тоже куда-то торопился)
Напомнило споры о IDE и освоении слепого метода печати. Оптимизации написания кода с 1 минуты на строчку до 10 секунд. Меня всегда смешили эти дебаты, т.к. что есть эти 50 секунд на фоне того, что на выполнение таска всё равно уходит 2 дня.
схоронил на случай важных переговоров о нейрослопе и десятикратно возросшей продуктивности. какое-то время пытаюсь в разговорах ссылаться на эти идеи - как об стену горохом. очевидно что сослаться в 1-2 предложениях нормально не выходит. а тут всё структурировано и по полочкам. лайк
Каждый слой ревью замедляет работу в 10 раз