Ревью кода: правила для обеих сторон
Проверяющему. Разделяйте обязательное и вкусовое: «здесь теряются данные» и «я бы назвал иначе» — разные вещи, и помечать это стоит явно. Объясняйте причину, а не только требование. Не переписывайте чужой код целиком в комментариях: автор ничего не поймёт, просто скопирует.
Автору. Не спорьте сразу — сначала переспросите, возможно, вы поняли замечание не так. Не приносите на ревью двадцать файлов: большие изменения смотрят формальнее и пропускают чаще. И благодарите за найденную ошибку — её нашли до пользователей, это удача.
Общее правило: ревью не про то, кто умнее. Оно про то, чтобы через год этот код мог менять кто-то третий.
Если ревью в вашей команде нет вовсе, просите хотя бы разовые разборы у опытного коллеги. Иначе рост идёт вслепую, и выясняется это на первом же собеседовании.
IT Омск — t.me/itnews_omsk
IT Омск
@itnews_omsk
Ревью кода: правила для обеих сторон
Этот пост опубликован в Telegram-канале IT Омск. Подписаться можно по ссылке: @itnews_omsk.