首页/开发/代码质量与评审速查/评审者读 diff 的策略
代码评审64 条

评审者读 diff 的策略

读 diff 的两种模式:按文件线性通读,或按意图切片(核心逻辑、测试、配套);大 diff 先看统计信息建立全貌再逐块读。

  • 先看背景:PR 描述与提交信息,建立意图框架
  • 按意图分组阅读:核心实现 → 测试 → 配置与文档
  • 大改动先过滤文件:先读关键路径,再批量扫其余
  • 读不懂时直接问:不要让猜测进入评审结论
带着意图读 diff 更容易发现错位:实现与意图是否一致。
读后自检:合上页面能否复述改动做了什么、影响哪些行为;说不清说明尚未读透,继续读或把疑问标出来。

易错点

  • 从文件开头逐行顺序读:注意力耗尽在后半段,关键问题漏过
  • 只看改动行不看上下文:局部正确但整体语义可能被破坏

数据来源与参考

  • Google Engineering Practices(评审方法建议)
​‌‌​​‌​​​‌‌​‌‌‌‌​‌‌​​​‌‌​‌‌‌​​‌‌​​‌​‌‌‌​​‌‌‌​​​‌​‌‌​‌​​‌​‌‌‌​​​‌​‌‌​‌​​‌​‌‌​​​‌‌​‌‌‌​‌​​​‌‌​‌‌‌‌​​‌​‌‌‌​​‌‌​​​‌‌​‌‌​‌‌‌‌​‌‌​‌‌​‌