代码评审(Code Review)

代码评审的目的

不是为了去刻意批斗某个Coder，而是为了团队成员之间相互了解学习，加深成员对系统的理解，使团队成员的代码更加健壮，提早发现代码缺陷。

那么应该如何做代码评审呢？

流程说明：

1.代码送审者每次提交最好是一个完整的功能，而不是一个小功能分很多次提交。
2.代码送审时候需要填写代码说明/审核人/功能链接/bug链接
3.审查人员收到邮箱通知后，查看审查任务，进行代码评审。（需要定义一些审核规范，一些基本的规范可以通过工具在在控制，自动审核）
4.审查人员根据团队之前达成的共识（代码规范）去评审一些代码，然后给出通过或者不通过的决定
5.送审人员根据驳回的意见进行修改后，然后在次送审。
6.如果代码通过，则合并到分支库里面去。

代码评审的好处

• 提升系统的可维护性

• 及早发现潜在缺陷与BUG，降低事故成本。

• 促进团队内部知识共享，提高团队整体水平。

• 评审过程对于评审人员来说，也是一种思路重构的过程，可以帮助更多的人理解系统。

• 交叉审查代码，类似于结对编程，彼此都能熟悉对方模块业务，降低因人员流失的运营成本及风险。

后记：

1.代码审查建议每半月一次或一月一次，审查追求的是质量而不是数量。不要过分要求程序员做代码审查。如果你强迫他们每天做一小时的代码审查，他们很快就会痛恨它，把它当成一种无趣的任务。

2.代码审查是针对代码，不是针对人。代码审查是一种学习，是表扬，是获得反馈，是一种十分社交性的活动。代码审查应该是有趣的，不要让它变的无聊。

目前100000+人已关注加入我们