97 Things Every Programmer Should Know Code Reviews

Code Reviews

所有的代码在投放到产品线之前，都需要经过正式的code review，这一条列在公司软件开发规范中。

代码审核人需要花时间阅读代码，花时间了解系统的全部细节。他们会迅速变成这个流程的瓶颈。

代码审核的目的是分享知识，建立编码的共识。

在代码审核是要礼貌一点。确保建议都是建设性的，而不是灾难性的。

• 让不同的角色来参review会议，避免组员的资历影响代码审核。例如，可以有一位关注文档，一位关注异常，第三位关注功能。
• 保持每周固定的几个小时代码审阅。
• 每次会议都轮换审核员，请专家以资深的经验和专业知识，或新同仁以新的知识，从不同角度来审核代码。

在审核前，先用静态检查工具将代码做检查，这样审核会更顺利。这样代码格式就不会是讨论的焦点。

一定要让代码审核变得有乐趣，其目的是在组员间分享知识。