评审关注点
| 维度 | 检查问题 |
|---|---|
| 正确性 | 边界条件、错误处理、并发与时区是否覆盖 |
| 安全 | 输入校验、权限校验、日志与密钥是否脱敏 |
| 可维护性 | 命名、职责划分、重复代码、注释是否到位 |
| 测试 | 是否补充用例，断言是否验证真实行为 |
| 兼容性 | 接口与数据结构变更是否有迁移说明 |
评审用语示例
评审用语示例
- 这里 page 为 0 时会怎样？建议补一个用例
- 这个查询在列表页会执行多少次？是否需要批量
- nit: 命名建议用 retryCount（不阻塞）把「必须修改」与「建议/nit」明确区分，作者才知道哪些是合入前置条件。