《C++编程规范：101条规则、准则与最佳实践》——1.5做代码审查

本节书摘来自异步社区出版社《c++编程规范：101条规则、准则与最佳实践》一书中的第1章，第1.5节，作者：【加】herb sutter , 【罗】andrei，更多章节内容可以访问云栖社区“异步社区”公众号查看。

摘要

审查代码：更多的关注有助于提高质量。亮出自己的代码，阅读别人的代码。互相学习，彼此都会受益。

讨论

好的代码审查过程对开发团队有许多好处。

通过来自同伴的良性压力提高代码质量。

找出错误、不可移植的代码（如果适用）和潜在的扩展问题。

通过思想交流获得更好的设计和实现。

快速培养新同事和入门者。

在团队中形成共同的价值观和集体主义。

增强整体实力，提升自信心、动力和职业荣誉感。

许多开发单位现在既不奖励高质量的代码和高质量的团队，也不投入时间和资金予以鼓励。我们估计几年之内这种情况仍然会存在，但是趋势已经在缓慢变化，这部分是因为软件安全性需求的不断增加。代码审查恰恰有助于提高软件的安全性，而且还是内部培训的一种极佳方法（而且没有成本）！

如果老板现在还不支持代码审查过程，那就先从提高管理层的认识做起（提示：一开始先给他们看看这本书），同时尽最大努力想各种办法腾出时间进行审查。这种时间是值得花的。

代码审查应该成为软件开发周期中的常规环节。如果能够与同事就奖惩制度达成一致，那就更好了。

代码审查无需太形式主义，最好通过书面形式进行——一封简单的电子邮件就足够了。这样能够更容易地跟踪你自己的过程，避免重复。

在审查别人的代码时，可能需要保存一份核对表以备参考。举贤不避亲，我们推荐本书的目录，它就是一个很好的核对表。愿你使用愉快！

小结：我们知道这是老调重弹，但还是不得不说。我们的天性都讨厌代码审查，但是内心又有一个小小的天才程序员乐此不疲，因为它富于成效，而且能够获得更好的代码和更可靠的应用程序。