| 重点 | 行为等价证据 / 动机单一 / 无新坏味道 |
|---|---|
| 证据 | 测试通过、对照说明、必要时的前后输出 |
| 要求 | 重构与功能分离的 PR 结构 |
评审清单
评审清单
□ 改动前后调用方与输出契约一致吗
□ 每一步都能独立解释吗（提交历史）
□ 是否留下临时过渡物（转发、双写）需清理
□ 测试是否随结构调整更新重构改动的评审要点
评审重构 PR 时看三点：行为是否等价（测试证据）、动机是否单一（一次只解决一类问题）、是否引入新坏味道（如更长的参数表）；重构与功能混提时请作者拆分。
易错点
- 只看风格不看行为等价证据：回归风险被放过
- 要求一次性拆分全部重构：评审成本转嫁给作者与评审者
数据来源与参考
- Google Engineering Practices（小型 CL 与评审重点）、《重构》第 2 版评审实践