Ревью кода: гигиена для обеих сторон

Ревью кода: гигиена для обеих сторон

Проверяющему. Разделяйте обязательное и вкусовое: «здесь теряются данные» и «я бы назвал иначе» — разные вещи, и пометить это стоит явно. Объясняйте причину, а не только требование. Не переписывайте чужой код целиком в комментариях: автор ничего не поймёт, просто скопирует.

Автору. Не спорьте сразу — сначала переспросите, возможно, вы поняли замечание не так. Не приносите на ревью двадцать файлов: большие изменения смотрят формальнее. И благодарите за найденную ошибку — её нашли до пользователей.

Общее правило: ревью не про то, кто умнее. Оно про то, чтобы через год этот код мог менять кто-то третий.

Если в вашей команде ревью нет вовсе, просите хотя бы разовые разборы у опытного коллеги — иначе рост идёт вслепую, и это выясняется на первом же собеседовании.

IT Волгоград — t.me/volgograd_it