首页/开发/代码质量与评审速查/Google 代码评审标准:以改进代码为目标
代码评审64 条

Google 代码评审标准:以改进代码为目标

Google 工程实践文档给出的评审决策标准:只要改动让代码库整体变好就应放行,不要求完美;当改动让代码明确劣化时才阻止。

文档The Standard of Code Review(Google Engineering Practices)
决策批准让代码库整体变好的改动
阻止条件改动让代码质量明确下降
  • 评审目的是随时间保持代码库健康,而不是追求完美
  • 没有完美的代码,只有更好的代码;'还可以更好'不是阻塞理由
  • 改动确实改善系统时:评审者不应为小瑕疵挡下它
  • 反之,若改动让质量下降(例如删掉测试、引入明显耦合),应明确指出并阻止
这条标准同时约束双方:评审者别为小事阻塞,作者也别把未成型的代码丢给评审。

易错点

  • 把'可以更好'当成阻塞理由,让改动长期挂着无法合并
  • 对明确劣化(去掉测试、绕过校验)放行,质量债被静默累积

数据来源与参考

  • Google Engineering Practices · The Standard of Code Review(google/eng-practices)
​‌‌​​‌​​​‌‌​‌‌‌‌​‌‌​​​‌‌​‌‌‌​​‌‌​​‌​‌‌‌​​‌‌‌​​​‌​‌‌​‌​​‌​‌‌‌​​​‌​‌‌​‌​​‌​‌‌​​​‌‌​‌‌‌​‌​​​‌‌​‌‌‌‌​​‌​‌‌‌​​‌‌​​​‌‌​‌‌​‌‌‌‌​‌‌​‌‌​‌