首页/开发/代码质量与评审速查/十分钟快速评审清单(速查卡)
代码评审64 条

十分钟快速评审清单(速查卡)

时间有限时的评审兜底清单:从意图、正确性、测试、安全到兼容性逐项快扫,找不出问题也要能说明看过什么。

这份清单解决什么

时间有限时用它做兜底:从意图、正确性、并发、测试、安全、兼容、回滚到残留逐项快扫。清单不替代认真评审,但能保证高风险区域被看到,也让你在批准时说得清自己看过什么。每项结论都要来自真实阅读,而不是逐格打钩。

基本信息

适用时机改动较大、评审时间被压缩时先扫一遍
定位防遗漏的兜底工具与自查表
用法每项结论来自实际阅读,读不到的要说明
局限支付、权限、数据迁移等高风险改动仍需完整评审

快速评审兜底清单

项快速检查
意图描述与实现是否一致,是否解决了声称的问题
正确性边界、空值、错误路径;对外部依赖的假设
并发共享状态与锁;回调与超时
测试新行为有测试;测试能失败;修复带复现用例
安全输入校验、权限挂载、密钥与敏感数据
兼容接口与数据格式变化是否可控
回滚出问题怎么退;迁移是否可重入
残留调试代码、注释掉的代码、TODO 是否清干净

快扫的顺序

  • 先读描述再扫文件清单:判断范围是否与描述声称的一致。
  • 定位行为变化的那几行重点看,其余按结构快速过一遍。
  • 测试文件看断言与失败路径,而不是只看用例数量。
  • 疑问先记下来,最后统一判断哪些是阻塞项。
  • 时间不足以覆盖全部时明确说明只看了哪部分,别默认批准。

快扫常用命令

快扫常用命令
git diff --stat              # 先看规模与文件清单
git diff -- '*.py'           # 按语言或目录分组逐块看
# 重点看:行为变化、错误分支、依赖与配置改动
git diff -- '*test*'         # 测试改动单独扫一遍
清单兜底不等于形式化:每项结论都要来自真实阅读,读到的内容能复述出来才算过关。
快扫后直接批准:清单没跑完就别给结论,宁可说明只看了哪部分。

易错点

  • 快扫后直接批准:清单没跑完就给结论,等于用形式感代替评审。
  • 清单全绿就不再深看:高风险改动(权限、资金、迁移)需要完整评审。
  • 只扫代码不扫配置与依赖:环境相关的问题多数藏在 diff 的边角。
  • 把清单当作团队门禁指标:逐格打钩但不看内容,反而掩盖了风险。

数据来源与参考

  • Google Engineering Practices · What to look for in a code review(汇总)
​‌‌​​‌​​​‌‌​‌‌‌‌​‌‌​​​‌‌​‌‌‌​​‌‌​​‌​‌‌‌​​‌‌‌​​​‌​‌‌​‌​​‌​‌‌‌​​​‌​‌‌​‌​​‌​‌‌​​​‌‌​‌‌‌​‌​​​‌‌​‌‌‌‌​​‌​‌‌‌​​‌‌​​​‌‌​‌‌​‌‌‌‌​‌‌​‌‌​‌