Code Review 代码审查
按照团队规范对代码进行质量、安全和可维护性审查。适用于 Vue3、FastAPI、Python 技术栈。
何时使用
- 用户请求代码审查
- 审查 Pull Request 或代码变更
- 用户提到「帮我看看这段代码」「review 一下」「代码质量」
审查清单
1. 正确性与逻辑
- 逻辑正确，处理边界情况
- 无明显的空指针/未定义访问
- 异步逻辑正确（async/await 配对）
- 错误处理完整，有 try/except 或 .catch()
2. 安全
- 无 SQL 注入（使用参数化查询）
- 无 XSS（Vue 已转义，注意 v-html）
- 敏感信息不硬编码（用 get_env 或配置）
- 接口有权限校验（如需要）
3. 代码风格与可维护性
- 符合项目约定（router/service/dao 分层）
- 函数职责单一，长度适中
- 命名清晰（变量、函数、路由）
- 有必要的注释和日志
4. 性能
- 无 N+1 查询（批量查询或 join）
- 大列表考虑分页
- 前端避免不必要的重复渲染
5. 测试
- 关键逻辑有测试覆盖
- 边界情况有测试
反馈格式
- 🔴 必须修复：影响正确性、安全或严重性能问题
- 🟡 建议改进：可读性、可维护性、最佳实践
- 🟢 可选优化：锦上添花
项目规范参考
- 后端：app/dao、app/service、app/router、app/model（entity/dto/vo）
- 数据库：使用 AsyncSqlSessionTemplate（insert/update/query_one/query_list）
- 报错：
from tkms.exception.api import ApiException - 配置：
from tkms import get_env - 前端：Vue3 + Ant Design，components/views/services 分层