Effective Code Review

在Google上用Effective Code Review作为关键字搜索，最顶上的文章就是这篇：Effective Code Reviews without the Pain ，写得挺实在的，那我们就借鉴一把：

目的
1.       提高代码质量（短期与长期）
2.       Developer之间的技术交流，提高个人能力

原则
1.       提问比过早的下对或错的结论要好。
2.       对代码，不对人。
3.       一定要有coding standard ，并且是不断维护的。
4.       切不可把Review 结果与个人绩效挂钩。
5.       需要明白解决方案有很多种，代码的写法也有很多种，公开讨论的目的是找到最佳的编码方式。

总结
Code Review不是单纯的代码检查，而是通过彼此借鉴，发现潜在的问题，寻找更好的架构和编码方式，不断的提高代码，并且作为技术交流，提高个人编码水平。

不小心看到了Firefox关于提交的Patch时的Review规则： http://www.mozilla.org/projects/firefox/review.html

包含三个部分：
1. Review Rules
2. Unit Test Rules
3. Indivisuals and Roles

简单清晰，包产到户，简单高效。

---

进行CODE REVIEW是有好处的，是一个将有经验的开发者传授知识给缺少经验开发者的好机会。不过这里需要注意几个事情：
1. REVIEW团队的大小；如果人太多，是很耗时的，一般以两个人为主，一个是作者，一个是REVIEWER，后者提出怎么样修改，然后两个人一起商议是否这样修改。
2.如果REVIEW的代码或系统太多或人太多，这个时候采用REVIEW代码是低效的，这个时候应该REVIEW的设计类图，例如：UML图。