Попросили разместить
Привет, ебаное. Я хочу рассказать об одном явлении в нашей профессии, которое портит жизнь всем. Бизнесу - тем, что удорожает разработку. Программисту - тем, что делает профессию неинтересной. Обществу в целом - тем, что в мире становится больше токсичности. И как следствие, больше нытья на данном ресурсе.
Как вы уже наверное догадались из названия, я хочу рассказать о придирках на code review. Каждый из нас, кто работал в более-менее крупных командах, с этим сталкивался. Когда ты выкладываешь код, 20% замечаний - это действительно вскрытие важных багов. Для этого code review и придумали, потому что глаза одного человека всегда замыливаются, и взгляд со стороны нужен, не важно джун ты или сеньор. Поэтому не нужно отказываться от code review.
А 80% замечаний - это нечто в стиле: вот тут можно более красивые абстракции переписать, как учили в книгах по паттернам проектирования, вот тут можно более модную библиотеку, а вот тут давай все на свете переименуем (а потом ещё раз переименуем), а вот тут давай заложим побольше гибкости на случаи, про которые мы пока ничего не знаем (а потом всё равно всё переписывать, потому что будущее непредсказуемо), а вот тут можно более красивую конструкцию языка применить, а вот тут давай структуру классов полностью поменяем, потому что мне так больше нравится и прочее прочее.
Есть обобщить - это замечания, требующие переписать код на другой вариант, который в целом плюс-минус такой же, как уже написанный, с точки зрения функциональности, читаемости и расширяемости. Это возникает всякий раз, когда есть больше одного равноценного способа сделать это (т.е. почти всегда). Почти всегда у каждого из нас своё мнение о красоте кода, и каждый из нас пытается это мнение продвинуть. В ущерб общей цели команды - сделать продукт как можно быстрее. Особенно неприятно то, что часто такие замечания исходят от людей с погонами - ведущих, лидов и проч., т.е. таких, которым нельзя просто так взять и отказать.
Если вы пишете такие замечания, помечайте, что они опциональные, не вставляйте палки в колёса разработчикам вашей команды! Поймите же, что когда вы пишете такие замечания как блокирующие, вы ухудшаете производительность команды - каждое такое замечание это всегда лишнее время, даже если не будет переписывания кода, надо потратить время на обсуждения. Любое переписывание уже готового кода - это всегда риск посадить новые скрытые баги (кто работал на практике а не в теории знают, что юнит-тесты это не панацея, да и не всегда ими всё покрыто в проекте). Кроме всего прочего, вы этими замечаниями демотивируете разработчика - вместо того, чтобы делать код для продукта, он делает код для удовлетворения ваших хотелок. Такая команда вдолгую проигрывает конкуренцию тем, командам, в которых понимают что ценнее: чуть более идеальный код или время и ресурсы, которые будут потрачены на его переписывание туда-сюда.
Поймите, что теория из умных книг по паттернам-хуяттернам и реальная практика очень отличаются. Хотите писать самый идеальный код на свете - делайте это дома в качестве хобби. Может быть суровым технарям не понравится эта суровая правда, но она правда: в современном рыночном IT код - это всего лишь инструмент достижения целей бизнеса.
Анонимус
Очень хотел поддержать автора, у меня есть несколько скриншотов с моего текущего места работы, когда типичная задротская личность препарирует мой код как он это хотел бы видеть. Ебаный аутист пишет такую дичь, что бывшие коллеги, с которыми работал, просто охуели, когда я показал им скрины. Вот была какая реакция:
Когда мне с утра прислали эти замечания, я заорал в прямом смысле слова. Еле сдержался, что бы не позвонить руководству и не сказать, что я увольняюсь, но чувствую, я это сделаю очень скоро. Тогда и опубликую всё это дерьмо.
Вы доебываетесь до рабочего кода.
Вы пытаетесь навязать свои мысли, своё видение, считая это абсолютно правильным.
Я не понимаю, что у вас в головах, выблядки. Синдром вахтера? Желание поиметь лычку? Ебаные аутисты, убогая задротская срань, не дающие нормально работать и превращающие жизнь разработчика в Ад. Из каких могил, из каких щелей вы все повылазили и заполонили сферу своей токсичностью?
PS Не говнокожу, код вылизываю, пишу по стандартам.