首页/开发/代码质量与评审速查/反模式:橡皮图章式评审
代码评审64 条

反模式:橡皮图章式评审

橡皮图章指不看或草草扫一眼就批准:评审信号失真,缺陷照常流入主干;常见成因是流程强制批准与评审量过载。

  • 症状:评审耗时极短、评论为空、批准紧跟提交
  • 成因:强制人数要求、评审队列过长、评审人过载
  • 对策:控制评审队列,轮值评审人,给评审留出时间预算
  • 度量:把空批准率与评审时长分布透明化

空批准自检(批准前)

空批准自检(批准前)
1. 不看描述,能说清改动做什么、为什么吗?
2. 能说出读过的关键文件,错误与边界路径看过没有?
3. 能说出验证方式:测试覆盖什么、失败会怎样?
三题都答不上:先补读再批准,别把批准当盖章。
批准时留一句结论:看过哪些文件、验证了哪些行为。

抽查已批准记录时看什么

抽查项异常表现
时间间隔数百行改动在几分钟内被批准,审读时间与规模不匹配
对话状态批准时仍有未回复的提问,问题被跳过而非解决
人员分布批准长期集中在个别人,队列过载或责任集中
评审不是签字仪式:无内容的批准对质量零贡献,还稀释了责任。

易错点

  • 用批准数量考核团队:催生形式化批准
  • 把空批准当高效:质量债在合并后集中爆发

数据来源与参考

  • SmartBear《Best Kept Secrets of Peer Code Review》
  • Google Engineering Practices · Speed of Code Reviews
​‌‌​​‌​​​‌‌​‌‌‌‌​‌‌​​​‌‌​‌‌‌​​‌‌​​‌​‌‌‌​​‌‌‌​​​‌​‌‌​‌​​‌​‌‌‌​​​‌​‌‌​‌​​‌​‌‌​​​‌‌​‌‌‌​‌​​​‌‌​‌‌‌‌​​‌​‌‌‌​​‌‌​​​‌‌​‌‌​‌‌‌‌​‌‌​‌‌​‌