代码审查：去做就对了

来源 (中文翻译版): https://blog.codinghorror.com/code-reviews-just-do-it/ 抓取时间: 2026-07-21 16:28:37

代码审查：去做就对了

Jeff Atwood

2006年1月21日 — 2分钟阅读 — 评论 在人性化同行评审中，Karl Wiegers 以一个强有力的声明开篇：

同行评审——即由软件交付成果的作者以外的人检查其缺陷和改进机会的活动——是可用的最强大的软件质量工具之一。同行评审方法包括检查、走查、同行桌面检查以及其他类似活动。在体验了近十五年同行评审的好处之后，我永远不会在一个不进行同行评审的团队中工作。 在 Vertigo 参与代码审查一段时间后，我相信同行代码审查是你可以做的改进代码的最重要的一件事。 如果你现在没有与另一位开发人员一起进行代码审查，你正在错过代码中的许多 bug，并且正在剥夺自己一些关键的职业发展机会。就我而言，在与一位开发人员同事一起检查我的代码之前，我的代码不算完成。 但不要只听我的话。McConnell 在《代码大全》中提供了大量关于代码审查有效性的证据： …软件测试本身的有效性有限——单元测试的平均缺陷检测率仅为 25%，功能测试为 35%，集成测试为 45%。相比之下，设计和代码检查的平均有效性分别为 55% 和 60%。审查结果的案例研究令人印象深刻：

• 在一个软件维护组织中，引入代码审查之前，55% 的单行维护变更都是错误的。引入审查后，只有 2% 的变更有错误。考虑所有变更时，引入审查后 95% 的变更第一次就是正确的。在引入审查之前，第一次正确的比例不到 20%。
• 在同一组人开发的 11 个程序中，前 5 个是在没有审查的情况下开发的。剩下的 6 个是在有审查的情况下开发的。所有程序发布到生产环境后，前 5 个每 100 行代码平均有 4.5 个错误。经过检查的 6 个程序每 100 行代码平均只有 0.82 个错误。审查减少了超过 80% 的错误。
• Aetna 保险公司通过使用检查发现了程序中 82% 的错误，并能够将其开发资源减少 20%。
• IBM 的 50 万行 Orbit 项目使用了 11 个级别的检查。它提前交付，并且只有通常预期错误的约 1%。
• 对 AT&T 一个拥有 200 多人的组织的研究报告称，在该组织引入审查后，生产力提高了 14%，缺陷减少了 90%。
• 喷气推进实验室估计，通过在早期阶段发现和修复缺陷，每次检查节省约 25,000 美元。

进行代码审查的唯一障碍是找到你尊重的开发人员来进行审查，并抽出时间进行审查。一旦开始，我想你会很快发现你在代码审查中花费的每一分钟都会得到十倍的回报。 如果你的组织对代码审查还不熟悉，我强烈推荐 Karl 的书《软件中的同行评审：实用指南》。Karl 在他的网站上提供的示例章节也是很好的入门读物。 软件开发 代码审查 同行评审 代码质量 职业发展

作者 Jeff Atwood

室内爱好者。Stack Overflow、Discourse 和 RGMII 的联合创始人。免责声明：我不知道我在说什么。让我们善待彼此。在 https://infosec.exchange/@codinghorror 找到我 ✉️ 订阅 ⏲️ 正在为你注册。 ❗ 出了点问题。请再试一次。 ✅ 成功！检查你的收件箱（以及你的垃圾邮件文件夹，以防万一）。