Цель код-ревью
Не для преднамеренной критики определенного программиста, а для того, чтобы понимать и учиться друг у друга среди членов команды, углублять понимание системы участниками, делать код членов команды более надежным и обнаруживать дефекты кода на ранней стадии.
Итак, как следует проводить код-ревью?
Описание потока:
1.代码送审者每次提交最好是一个完整的功能,而不是一个小功能分很多次提交。
2.代码送审时候需要填写代码说明/审核人/功能链接/bug链接
3.审查人员收到邮箱通知后,查看审查任务,进行代码评审。(需要定义一些审核规范,一些基本的规范可以通过工具在在控制,自动审核)
4.审查人员根据团队之前达成的共识(代码规范)去评审一些代码,然后给出通过或者不通过的决定
5.送审人员根据驳回的意见进行修改后,然后在次送审。
6.如果代码通过,则合并到分支库里面去。
Преимущества код-ревью
Улучшить ремонтопригодность системы
Раннее обнаружение потенциальных дефектов и ошибок снижает стоимость аварий.
Способствовать обмену знаниями внутри команды и повышать общий уровень команды.
Процесс рецензирования — это также процесс рефакторинга мышления для рецензентов, который может помочь большему количеству людей понять систему.
Перекрестный просмотр кода, аналогичный парному программированию, позволяет друг другу ознакомиться с модульным бизнесом друг друга, снижая операционные расходы и риски, связанные с текучестью кадров.
постскриптум:
1. Ревью кода рекомендуется раз в полмесяца или раз в месяц, причем ревью преследует качество, а не количество. Не слишком просите программистов делать код-ревью. Если вы заставите их проводить проверку кода каждый день по часу, они быстро возненавидят эту скучную задачу.
2. Проверка кода предназначена для кода, а не для людей. Код-ревью — это обучение, это похвала, это получение обратной связи, и это очень социальная деятельность. Проверка кода должна быть веселой, а не скучной.