C代码评审最终指南——第2部分，有哪些关键点需要注意？

本文共计6363个文字，预计阅读时间需要26分钟。

在第1部分中，我们针对代码审核进行了广泛讨论。本部分将重点讨论C++，并提供代码审核清单和一些最佳实践。您可以选择任意顺序阅读，但建议您先阅读以下内容：

在第 1 部分中，我们对代码评审做了一个广泛讨论。这一部分将重点讨论 C++，提供一个代码评审清单和一些最佳实践。您可以按任何顺序阅读，但是我们建议您先返回去阅读一下我们之前的帖子。

C++ 评审最终清单

代码评审清单从来都不是全面的——因为要检查的问题、项目和潜在事情的数量几乎是无穷无尽的。因此，制定一份涵盖所有潜在可能性的清单是不可能的，也更难执行。相反，我们将重点关注在 C++ 代码评审期间应该涵盖的更为广泛的方面。

类别1——对底层域的要求和理解

Joel Spolsky（Stackoverflow 的联合创始人，Fog Creek（现为 Glitch）创始人，Joel-on-Software 博客的作者）曾经讲述过他与微软联合创始人比尔·盖茨做的一次评审。

时任首席执行官花了很长时间问了一些看似随机的问题，以及一些更难的问题。问题变得越来越难，越来越尖锐，直到他问了一个“杀手”问题。Spolsky 一回答，盖茨就通过了评审。但这样做的原因是什么？比尔·盖茨希望确保受评人员控制材料，他通过在评审领域提出越来越难的问题来做到这一点。

这是评审的重要一部分。作为一名评审人员，您应了解正在评审的特性或错误修复情况，并能够就实现细节提出好的问题。这通常不是代码评审的主要重点（即，通常不应占用评审的大量时间），但它应该是评审的一部分。

类别2——非功能性需求

非功能性需求包括系统中不直接服务于产品需求但允许产品正常工作（并符合法律法规）的部分，在事情进行的不顺利时，我们可以分析问题。