Автор приводит пример о том, как он ревьюит джуна. Для меня это кажется несколько странным, когда сильный разработчик ревьюит заведомо менее скилованного разработчика, при этом испытывая какие негодования. По моему опыту я крайне редко испытываю разочарование/горечь от кода разработчика уровня слабее твоего. Банально потому ожидания такие. И делать ревью для таких ребят очень даже легко, так как они намного чаще прислушиваются к твоим рекомендациям. Очень часто я пишу псевдокод, чтобы джун уловил мою идея о том, как улучшить данный код и почему.
Другое дело, когда идет ревью senior-senior. Вот мне интересно, как автор расправлялся с этим? Для меня здесь вылазят куда более серьезные проблемы как личные, так и профессиональные. Зачастую (опять же из моего опыта) в таком случае каждый пытается пропихнуть именно свой стиль, как бы он это сделал — «А вот я абстракцию бы по-другому сделал», «Я бы заюзал то-то и то-то» и тд. Интересно то, что проблема решена и код рабочий, но появляются вопросы «А вот если через N лет бизнес скажет нам сделать это, данный код уже не будет гибким и нам надо переписывать» и тд. Преждевременная оптимизация, пушинг своих личных предпочтений и все остальное делает ревью иногда невыносимым.
Конечно, бывает так, что ты «совпадаешь» с видением другого сеньора и тогда ревью проходит идеально, строится на вопросах, а не на пушинге, например «А что если бы мы заюзали вот это, давай подумаем вместе, какие плюсы-минусы можем получить». Можно порисовать на доске и за 15-20 минут найти классное решение, которое удовлетворить обоих.
А как у вас проходят ревью между сеньорами? Часто ли сталкиваются лбами?
Другое дело, когда идет ревью senior-senior. Вот мне интересно, как автор расправлялся с этим? Для меня здесь вылазят куда более серьезные проблемы как личные, так и профессиональные. Зачастую (опять же из моего опыта) в таком случае каждый пытается пропихнуть именно свой стиль, как бы он это сделал — «А вот я абстракцию бы по-другому сделал», «Я бы заюзал то-то и то-то» и тд. Интересно то, что проблема решена и код рабочий, но появляются вопросы «А вот если через N лет бизнес скажет нам сделать это, данный код уже не будет гибким и нам надо переписывать» и тд. Преждевременная оптимизация, пушинг своих личных предпочтений и все остальное делает ревью иногда невыносимым.
Конечно, бывает так, что ты «совпадаешь» с видением другого сеньора и тогда ревью проходит идеально, строится на вопросах, а не на пушинге, например «А что если бы мы заюзали вот это, давай подумаем вместе, какие плюсы-минусы можем получить». Можно порисовать на доске и за 15-20 минут найти классное решение, которое удовлетворить обоих.
А как у вас проходят ревью между сеньорами? Часто ли сталкиваются лбами?