对修改后的代码做 Code Review 时应检查哪些方面？

核心回答

先确认修改是否实现需求、有没有破坏原有行为，再检查实现质量。可以按范围、行为链、关键风险、验证证据四步审查，最后看复杂度、性能、命名和文档；不能只按格式规范扫一遍。

先看范围与行为

对照需求检查修改了哪些文件、是否混入无关重构，沿一次实际请求查看参数、校验、处理、持久化与返回。再查受影响调用方和异常分支，确认接口语义、字段和状态转换仍兼容。

重点检查什么

• 业务正确性：计算、边界、空值和状态判断是否符合要求，失败是否被误报成功。
• 权限与数据边界：身份和资源归属是否验证，是否增加敏感信息暴露或绕过既有校验。
• 事务与并发：检查与写入之间是否有竞态，部分失败是否回滚，重复调用会不会重复产生副作用。
• 依赖与性能：新增循环或远程调用是否放大请求量，超时和资源释放是否完整，是否引入不必要依赖。
• 可维护性：实现是否过度抽象，命名与注释是否清楚，相关文档或配置是否需要同步。

用证据形成结论

检查测试是否覆盖本次新增和改变的行为，特别是错误路径与回归条件。必要时运行最小用例或查看实际页面；并发问题还要推演交错顺序，不能只凭一次运行正常就排除。

意见应落到位置、触发条件和可观察影响。例如“两个请求同时读取相同余额，再分别写回，可能覆盖一次扣款”，比“并发不安全”更便于修复。没有发现明确问题时，可以说明已检查的范围及尚未运行的验证，不为凑数量添加风格偏好。

对应教程

参考资料