代码审查是软件开发流程中的重要部分，这一环节可以有效提升代码质量，减少 Bug 存在。然而，对于很多程序员而言，进行代码审查无疑是痛苦的，甚至有人认为它拖慢了进度、不利于快速上线......那么，代码审查是否有必要？
本文将以科技大厂 Google 为例，分享 Google 内部的做法以及代码审查工具，希望我们可以从中吸取一些有益的经验。
许多前 Google 工程师都提到他们非常怀念 Critique。这是 Google 自己的代码审查工具，也是他们最恋恋不舍的一款 Google 内部工具。
有人在 Reddit 的一篇文章中列举了他们非常看好的功能，比如“attention set”等。
那么，Critique 究竟是什么，它有何优点？又如何与 Google 的代码审查流程配合使用呢？
为什么Critique在Google内部备受推崇？我们可以从这学到什么？
不求完美，但要持续改进：尽管经验丰富的开发者可能觉得经验较少的开发者的代码达不到他们的个人标准，但Google鼓励持续改进，而非追求完美。开发者需要不断进步，过于苛刻的审查不利于将来的提升。
遵循代码指南：在涉及代码风格的问题上，必须严格遵循和参考Google风格指南（）。
分享知识：鼓励审查者通过代码审查分享有关语言特性、代码库以及其他相关的知识。通常，这些准则会附上“支持文档”，比如链接到Google/Abseil的C++每周技巧。
严格的标准，但保持轻量级：Google希望审查者在24小时以内审查代码变更，并鼓励只有一名审查者，目的是节省所有相关人员的时间。在Google，大多数代码变更都很小，且只有一名审查者，除了授权提交的确认外，没有其他评论。一周内，70%的变更都会在初次审核申请后的24小时内确认提交。
礼貌和职业操守：信任和尊重的文化至关重要。反馈应专业，避免个人批评。审查者应该对作者的方法持开放态度，仅在必要时提供替代方案，并将每条评论视为学习的机会。
我想特别说一说这一点，虽然看似很明显，但在实践中并不容易做到。人们往往对自己编写的代码有特殊的感情，而审查者的严厉程度也有可能超过自己的预期。与面对面交流相比，书面文字可以减少沟通的语气带来的影响。
一个变更列表（变更列表是Google的拉取请求，相当于PR）要想通过审查，就不能有残留的未解决评论，而且至少有一名审查者给出“我觉得没问题”的意见，并获得两种类型的审核批准：
Critique是Google的代码审核工具，可帮助工程师高效地审核和提交代码变更。
重要的是，2023年Google最新发布的消息显示，他们内部拥有全面的人工智能驱动的代码审查工具（如上图所示）。
当审查者在代码上留下评论时，Critique将显示一些建议，而这些建议都来自机器学习，这意味着代码的作者只需点击一个按钮就可以解决评论。
根据Google的研究论文，最近的进展表明Google正在尽可能地通过人工智能驱动的代码审查工具提高开发人员的生产力。
在Google的内部代码编辑器Cider中创建一个CL（变更列表，即PR），该编辑器“紧密集成”了Critique以及其他Google内部工具，从而提高了开发人员的生产力。
预审工具：在审核开始之前，在Critique的帮助下最后一次打磨代码，然后显示差异、构建和测试结果，并进行风格检查。
差分比较和可视化：比较结果带有高亮显示语法、交叉引用、行内差异、忽略空白，以及检测并隐藏了被移动的代码。
分析结果：显示静态分析器的结果，突出显示重要发现并提供修复建议。包括“预提交”，即Critique中运行的自动化测试，可强制执行项目特定的不变量。
Critique整合了分析作者的反馈渠道。 审核者可以点击分析生成的评论上的“请修复”，表示作者应该解决这个问题。 代码的作者和审核者都可以点击“没有帮助”，用于标记对审查过程没有提供实质性帮助的分析结果。
当PR准备好，可以开始审核时，代码的作者会添加审查者，并正式将代码发送给他们进行审查。当PR或变更列表已送审核时，如果尚未在当前代码快照上运行“预提交”，则运行这些测试。这意味着，参与审查的所有人都能知道代码是否有问题。CQ9电子平台 CQ9传奇
代码审查也可以匿名，即代码作者的身份对审查者保持匿名。然而，Google并未发现匿名审查和“真实”的审查之间有太大的差异。
未解决的评论表示代码的作者必须解决的问题。代码变更作者在回复评论时，可以将其标记为“已解决”。
已解决的评论包括可选或信息性的评论，表示可能不需要变更作者采取任何行动。
此外，还有一个审查状态的仪表板和一个attention set，可以让代码审查者知道当前正在等待谁的回复。
如上所述，Google的代码审查至少需要一名审核者给出“我觉得没问题”的意见。此外，代码变更不能有残留的未解解决评论，但代码作者可以在回复时自行将评论标记为已解决。最后，还需要相关代码库部分所有者的批准，以及可读性批准。
“Google研究人员发现，Critique的用途不仅限于代码审查。代码变更的作者使用Critique来检查代码的差分，并浏览分析工具的结果。在有些情况下，代码审查是变更开发过程的一部分：审核者可能会发送未完成的变更，以决定如何完成实现。此外，在代码变更被批准很久之后，开发人员还会使用Critique来检查提交的变更历史。”
Google特意针对公司的代码审查进行了一项研究。下面是他们的论文中一些有趣的统计数据。
在Google内部，对Critique感到满意的软件工程师数量高达97%。
我询问了7名Google员工，为什么相较于过去使用过的工具（比如GitHub），他们更喜欢Critique。
静态分析：Google拥有一套功能完备的静态分析工具，可自动为代码提供可操作的反馈。这为代码作者和审查者节约了很多时间，因为审查者不必在显而易见的问题吹毛求疵。
仅关注最新变更的文件：只关注代码的最新“快照”。不关心以前的快照、提交和代码变更，这样用户界面更清晰。
机器学习提供建议：Google最新的机器学习提供支持的建议极大地加快了代码审查的速度。
与Google其他工具的紧密集成：Critique与Google的IDE以及其他内部工具（如bug） 的集成非常好，能够提高生产力。包括代码、评论以及任务票的链接。
令人满意的游戏机制：虽然Critique不是为了游戏化而构建的，但有的Google员工报告说，他们很喜欢看到Critique“变绿” ，因为这意味着PR已准备好提交了，即通过了所有测试，且得到了审核者的批准。
一项针对88名Mozilla开发人员的定性研究发现，静态分析集成是代码审查中最常请求的功能。 有了自动分析，审查者就可以专注于代码变更的可理解性和可维护性，而不会因琐碎的评论（例如格式）分心。
虽然如今许多工具都具备这些功能，但我认为正是因为与各种工具的紧密集成，以及Google的特定工作流和代码库的极端“个性化”，使Critique备受喜爱。
与此同时，这也意味着并不是每家公司都能完整地复制Critique以及其他相关工具。例如，他们的一些工具主要是为了解决单一代码库结构带来的挑战。
虽然Critique永远不会成为开源项目，但Gerrit（）是一款类似于Critique的工具，这是由Google创建并维护的开源代码审查工具。
然而，我认为Google确实为提高开发者的生产力做了很多努力和思考。我们可以从中吸取很多有益的经验。CQ9传奇电子 CQ9电子
