Code Review по-Фейнмановски
Гениальный Фейнман в своей книге описывал занимательную историю о строительстве первых ядерных очистных установок. Ему, молодому специалисту, приносят на обзор свежий проект обогатительного завода - пачку синек со схемами накопителей, клапанов, труб и насосов. Офигевший от безумной сложности увиденного, Фейнман тыкает пальцем в один из таинственных крестиков и спрашивает "а что если этот клапан заклинит?". Инженеры смотрят на синьки, водят пальцами по линиям труб, и изумлённо произносят "всё ж накроется нафик! вы просто гений, мистер Фейнман!"
Мораль этой байки в том, что в достаточно сложной системе критически важных компонент больше чем безопасных.
Вот и в своей работе мне часто приходится так же делать code review. Приходит review request на что-то заумное, в чём фиг разберёшься за пятнадцать минут, а пол-дня тратить на чужую работу - жаба душит. Докапываться до стилевых несоответствий я себе не могу позволить - я ж не менеджер какой. Поэтому - выбираю самый непонятную строчку, и пишу комментарий что в этом месте что-то нечисто. И как правило, в этом месте действительно оказывается нечисто. Ибо, если всё непонятно - значит критических мест в коде больше чем надёжных.
Есть правда и небольшая тонкость - надо всё-таки чувствовать "куда надо бить молотком", за это нам и платят нашу зарплату...