rowankid Code Review Skill — 代码审查技能
通用检查维度
逻辑正确性
- 核心业务逻辑是否正确？
- 边界条件是否处理？（空值、最大值、并发场景）
- 返回值是否符合接口契约？
异常处理
- 外部调用（数据库/HTTP/文件）是否有 try-catch？
- 异常是否有意义地记录到日志？
- 是否有兜底逻辑？
安全性
- SQL 是否使用参数化查询？
- 是否有越权风险？
- 敏感信息是否避免打印到日志？
性能
- 是否存在 N+1 查询？
- 循环内是否有数据库 / 网络调用？
- 大数据量场景是否有分页？
可读性
- 函数 / 变量命名是否清晰？
- 关键逻辑是否有注释？
- 函数长度是否合理（< 50 行建议）？
输出模板
# 代码审查意见 — [MR/PR 标题]
审查人: | 日期: | 结论: [通过 / 修改后通过 / 打回]
## 必须修改
| 位置（文件:行号） | 问题描述 | 修改建议 |
|-----------------|---------|---------|
## 建议优化
-
## 做得好的地方
-