首页/开发/代码质量与评审速查/评审优先级:先核心后外围
代码评审64 条

评审优先级:先核心后外围

评审时间有限时按影响面排序:核心逻辑与数据路径优先,其次接口与兼容性、测试质量,最后命名与文档、风格细节。

  • 半小时评审时间:先保证核心逻辑与高风险路径被认真读
  • 风险标签辅助:涉及资金、权限、数据的改动优先深审
  • 剩余时间扫其余:存疑处标注问题而不是放过
  • 时间不足时明确告知:核心已看,其余明天继续
评审不是限时抢答:没看完的改动宁可慢半天,也不给虚假批准。
排序依据影响面与出错代价:资金/权限/数据优先于外观与风格
时间不足明确标注未覆盖部分并约定续审时间,不静默放过

易错点

  • 平均用力:高风险代码与文档改动耗费同样的注意力
  • 时间不够就快速批准:埋下最深的隐患

数据来源与参考

  • Google Engineering Practices · Speed of Code Reviews(质量与速度平衡)
​‌‌​​‌​​​‌‌​‌‌‌‌​‌‌​​​‌‌​‌‌‌​​‌‌​​‌​‌‌‌​​‌‌‌​​​‌​‌‌​‌​​‌​‌‌‌​​​‌​‌‌​‌​​‌​‌‌​​​‌‌​‌‌‌​‌​​​‌‌​‌‌‌‌​​‌​‌‌‌​​‌‌​​​‌‌​‌‌​‌‌‌‌​‌‌​‌‌​‌