Цель встречи:- обмен опытом с командой,
- проверка качества кода.
Тайминг:10-30 минут
Одна из задач код-ревью — найти и исправить баги и проблемы в структуре кода. А вовсе не заставить автора написать код так, как это сделали бы вы.
Не забывайте, что в программном обеспечении многие вещи можно решить по-разному, в зависимости от вкусов разработчика, и у каждого подхода есть свои плюсы и минусы.
Собираясь предложить очередную правку, спросите себя: может быть, это просто моё мнение? Если автор выбрал другие способы, обосновав свой выбор, то лучше поддержите его. Уважайте право автора на собственные решения.
Важно следить за командным анализом разбора кода и модерировать встречу. Руководитель всегда должен следить за соблюдениями правил.
Что недопустимо во время код-ревью:- Переходить на личности.
- Указывать только на ошибки, без предложенных вариантов исправления.
Что точно можно делать во время код-ревью:- Хвалить то, что написано хорошо.
- Объяснять, что именно понравилось в написанном коде.
- Вместо критики задавать вопросы. (Почему выбран именно такой способ?)
- Использование личных примеров. Покажите автору кода, что не только он совершал такую ошибку. Это отличный способ смягчить критику: «Мне самому это тяжело далось…», «Я делал то же самое».
Вникая в чужой код, оставляйте в комментариях свой
«монолог на ходу». «Ага, понял, что это делает… Хорошо, подключается сюда и вызывает это, замечательно… а эта часть зависит от тех двух, отлично».
Ход мыслей другого программиста, который разбирается в твоём коде, — ещё одна форма проверки работы.