中文代码审查
使用场景
对代码进行全面审查,生成中文审查报告。
审查维度
1. 功能正确性
- 逻辑是否正确
- 边界条件是否处理
- 异常情况是否覆盖
- 返回值是否一致
2. 代码质量
- 命名是否清晰(变量、函数、类)
- 函数是否过长(建议 < 50行)
- 是否有重复代码
- 圈复杂度是否过高
3. 性能
- 是否有 N+1 查询
- 是否有不必要的循环
- 内存是否有泄漏风险
- 是否有可以缓存的计算
4. 安全
- 输入是否做了校验
- SQL是否使用参数化查询
- 敏感信息是否硬编码
- 权限是否正确检查
5. 可维护性
- 是否有必要的注释
- 是否有单元测试
- 错误处理是否完善
- 日志是否足够排查问题
报告模板
# 代码审查报告
## 概述
- 审查文件:xxx.py
- 审查日期:2026-06-06
- 总体评分:B+(良好)
## 发现的问题
### 🔴 严重问题(必须修复)
1. **SQL注入风险** (第45行)
- 问题:直接拼接SQL字符串
- 建议:使用参数化查询
- 示例:`cursor.execute("SELECT * FROM users WHERE id = %s", (user_id,))`
### 🟡 建议改进
1. **函数过长** (第100-250行)
- 问题:process_data 函数超过150行
- 建议:拆分为多个小函数
### 🟢 优点
1. 错误处理完善
2. 日志记录详细
评分标准
- A+:优秀,可直接合并
- A:良好,小问题可忽略
- B+:良好,建议修改后合并
- B:一般,需要修改
- C:较差,需要重构
- D:严重问题,需要重写
