97-things-every-programmer-should-know-14: 代码审查

你应当做代码审查，原因是：可以提高代码质量和降低缺陷率。

很多人之前可能在代码审查上体验不太好，导致不喜欢/排斥代码审查。一些开发单位要求所有代码部署到生产环境之前都要通过正式评审。通常是架构师或项目组长做，即执行架构师需要评审一切的工作流程。

有些公司在软件开发规范手册中有硬性规定，因此开发者必须遵守。也许极少数单位需要这些严格和正式的过程，但绝大多数应当不需要，在大多数单位中实践这种强制且正式的方式会适得其反。被审查者会觉得像是在接受审判，同时审查者既需要时间来阅读代码，也需要时间来理解系统的所有细节。审查者很快会成为审查过程中的瓶颈，这个会导致审查流程很快退化，变得难以实施。

代码审查的目的不是简单的纠正代码中的错误，而是分享知识和建立通用编码准则。 与其寻找错误，不如通过审查代码来学习和理解它。

审查代码时态度要温和，确保提的建议是具有建设性的而非刻薄的。 应当为评审会议介绍不同的评审角色，避免团队成员的资历影响代码评审。如一个审查者关注文档（注释），一个关注异常，另一个关注功能。这种方法有助于分散团队成员的负担。

每周有一个定期的代码审查日，需要花几个小时在一个审查会议上。在每次会议上以简单的循环模式轮换被审核人。注意每次评审会议上也要在团队成员之间切换角色。需要邀请新手参与代码审查，他们可能经验不足，但大学中学到的新知识可能能提供不同的视角；也需要邀请经验丰富的专家参与，他们能分享经验和知识，也能更快更准确的识别易出错的代码。

如果团队拥有统一的代码检查工具，那代码审查将会更容易，这样代码格式这类问题就永远不需要在代码审查会议上讨论了。

小结

使代码审查变得生动有趣可能是代码审查成功的最重要的因素。如果审查会议很痛苦或枯燥无味，就很难激励任何人。所以代码审查应当是非正式的，主要目的在团队成员之间共享知识。