代码评审--流程、工具和最佳实践

为什么执行代码评审

1.      代码评审的最大好处是纯社会性的，当你知道你的每一行代码都有另外一个人看，自然而然会更加卖力的表现，拿出最好的状态编码，因为每个人都有虚荣心嘛；

2.      代码评审可以及时发现一些容易发现的BUG，而不必将发现BUG的时间点推迟到测试阶段；

3.      代码评审可以保证至少有两个人都理解任何一份代码。当出现员工休假，离职等情况的时候，至少保证团队的代码不会陷入无人理解或者无人处理的状况。

 

流程篇

代码评审的流程有以下两种：

1.      提交前评审(pre-push code review)

2.      提交后评审(post-push code review)

 

提交前评审的流程如下：

1.      程序员在试图提交代码变更到代码库之前，先提交变更申请，变更申请包含了这次变更的内容，评审人；

2.      评审人查看变更内容，评估变更，与变更申请人沟通，评估是否通过变更；

3.      如果评审人通过变更，则变更申请人才可以提交代码到代码库；

4.      如果评审人不通过变更，则变更申请人需要根据讨论结果或评审建议做出修改，直到与评审人达成一致，通过评审，才可以提交代码到代码库；

 

提交后评审的流程如下：

1.      程序员提交变更代码到代码库；

2.      评审人审查这次变更的内容，如果评审通过ÿ