| 文档 | How to do a code review（评审者指南） |
|---|---|
| 顺序 | 先整体意图，后代码细节 |
评审关注层次（自上而下）
| 层次 | 要回答的问题 |
|---|---|
| 设计 | 改动是否放在系统的正确位置，与既有架构是否一致 |
| 功能 | 行为是否符合意图；边界、异常、并发路径是否考虑 |
| 复杂度 | 有没有更简单的实现；是否为假想需求过度设计 |
| 测试 | 测试是否有效、能否失败；关键路径是否覆盖 |
| 命名 | 命名是否表达意图，是否与领域术语一致 |
| 风格 | 格式细节交给格式化工具，评论时用 nit 前缀 |
不要从风格细节开始评：先确认整体方向正确，否则评审会失焦。