如何做好CodeReview

让Google程序员如此优秀的一个重要但非常简单的事情：Code Review


1、Code Review的好处

• 对代码保持敬畏
• 学习他人的优点
• 团队间传播知识

对于Coder来说，当你知道有同事会检查你的代码时，你的编程态度就会认真起来，写出的代码会更加整洁、符合规范，程序的结构也会更加合理。如果没有严肃的代码审查，缺乏他人检查带给你的紧迫感，你的代码提交就会逐渐随意起来。
对于Reviewer来说，检查同事的代码是一个很好的学习机会，对于一个问题或功能，一定是有多种修复或解决方案的，理解其他人的实现方式，避免局限在自我的思维方式，是非常有益的。
另外，在很多开发团队中，经常是一个人负责一个模块，每个人只关注自己的那个模块，假如这个人休假或离职，就会出现其他人无法维护的情况，这对整个团队来说是非常不利的。团队的其他成员也许做不到像代码作者一样对程序的熟悉程度，但熟悉程序的设计和架构是十分必要的。
2、营造健康的Code Review文化

• Code Review不是批评和指责
• Code Review不是展示技能的地方
• 互相尊重，换位思考
• 鼓励很重要

在做代码审查时，缺乏审查经验的Reviewer经常会犯一些错误，如批评多于鼓励、炫技多于查错，从而造成糟糕的代码审查体验，成为团队成员接受代码审查的障碍。
在团队中营造健康的Code Review文化，一是要互相尊重，懂得换位思考，Coder要主动向Reviewer解说代码的修改目的和内容，Reviewer则重点在于理解作者的实现思路，查看是否有明显错误；二是避免总是试图找出错误，要多一些鼓励的评论，如代码逻辑清晰、实现很巧妙。
3、Code Review三步走

• 第一遍：浏览代码，了解作者实现思路
• 第二遍：精读代码，理解作者实现方式，检查是否有明显错误
• 第三遍：和自己脑海的实现方式对比，提出建议或赞赏

4、