code review

执行review的原因：

• 可以及时的发现一些容易发现的BUG，而不必将发现代码的时间点推迟到测试阶段
• 代码评审可以保证至少有两个人去理解一份代码，发生意味状况后，至少保证代码不会陷入无人处理的情况下
• 从心理上讲，你知道自己的代码还会有别人看，肯定会更认真的码

代码审批的流程：

分两种情况：1.提交前评审

                              程序猿在想要提交代码变更到代码库之前，先提交变更申请（包含变更内容，评审人）

                              评审人查看变更内容，   评估变更，与变更申请人进行沟通，评审是否通过变更

                              假设评审人不通过变更，则申请人需要根据讨论结果或评审建议做出修改，直到与评审人达到一致，通过评审，才能提交代码

                      2.提交后评审

                                 程序猿提交变更代码到代码库

                                 评审人进行审查，若通过，则标记此次的变更已经审查

                                 如果评审人不通过，则需要与申请人进行沟通，申请人需要根据讨论结果或评审建议做出修改，直到与评审人达到一致，通过评审，才能提交代码

提交前评审的好处：

1.程序猿会积极的将变更的代码组织的更好，更模块化，更容易阅读

2.评审人在代码提交之前发现问题，给出更好的建议，程序猿也容易接受

3.评审人会更加认真的对变更进行评审，发现问题后会积极的参与讨论

4.在提交之前就发现问题，并给予修改

 

提交后评审的好处：

1.post-push code review更加容易实施，过程对现有的组织架构和流程没有完全的颠覆，对团队成员的要求没有那么高

2.不需要对修改代码&提交变更这个过程中断，不需要等待评审的时间

3.可以作为组织向pre-push code review过程实施的过渡训练