adspecs 0.1.12

package/.adspecs/templates/06-/344/273/243/347/240/201/350/257/204/345/256/241/346/212/245/345/221/212/346/250/241/346/235/277.md

@@ -0,0 +1,1012 @@
+# 代码评审报告模板
+
+> **版本**: v1.0
+> **创建日期**: 2026-03-16
+> **适用人员**: 代码评审员、开发工程师、AI助手
+> **目的**: 记录代码评审情况,包括代码质量分析、设计原则检查、性能优化建议、安全漏洞检测,确保代码质量和可维护性
+
+---
+
+## 📝 使用说明
+
+本模板用于记录代码评审的执行情况,包括:
+
+1. **评审概览** - 评审基本信息、评审范围、评审标准
+2. **代码规范检查** - 命名规范、注释规范、格式规范
+3. **设计原则检查** - SOLID原则、DRY原则、KISS原则
+4. **性能优化分析** - 性能瓶颈、优化建议
+5. **安全漏洞检测** - SQL注入、XSS、CSRF等
+6. **测试覆盖率分析** - 单元测试覆盖率、测试用例质量
+7. **代码复杂度分析** - 圈复杂度、认知复杂度
+8. **代码重复度分析** - 重复代码检测
+9. **依赖分析** - 依赖管理、版本冲突
+10. **评审结果** - 评审通过率、问题统计
+11. **改进建议** - 改进建议、后续工作
+
+---
+
+## 🎯 模板结构
+
+### 第1部分：评审概览
+
+```markdown
+## 1. 评审概览
+
+### 1.1 基本信息
+- **评审报告编号**: [如:CRR-20260316-001]
+- **需求名称**: [对应的需求名称]
+- **需求编号**: [对应的需求编号]
+- **所属模块**: [对应的模块名称]
+- **评审人员**: [评审人员姓名]
+- **开发人员**: [开发人员姓名]
+- **评审日期**: [YYYY-MM-DD]
+- **评审版本**: [v1.0]
+
+### 1.2 评审目标
+**本次代码评审要达到什么目标？**
+[用1-2句话描述评审目标,如:检查代码质量、验证设计原则、发现潜在问题、提供改进建议]
+
+### 1.3 评审范围
+**包含哪些评审内容？**
+[列出本次评审包含的内容,如:代码规范、设计原则、性能优化、安全漏洞、测试覆盖、代码复杂度、代码重复度、依赖分析]
+
+**不包含哪些评审内容？**
+[明确说明本次评审不包含的内容,如:业务逻辑验证、功能测试]
+
+### 1.4 评审代码
+
+**评审代码列表**:
+- [文件1]: [文件路径] - [代码行数] - [说明]
+- [文件2]: [文件路径] - [代码行数] - [说明]
+- [文件3]: [文件路径] - [代码行数] - [说明]
+
+**代码统计**:
+- **总文件数**: [数量]
+- **总代码行数**: [数量]
+- **Java代码行数**: [数量]
+- **Vue代码行数**: [数量]
+- **SQL代码行数**: [数量]
+
+### 1.5 评审标准
+**通过标准**:
+- 代码规范符合要求
+- 设计原则遵循良好
+- 无安全漏洞
+- 测试覆盖率≥80%
+- 代码复杂度合理
+- 无严重问题
+
+**失败标准**:
+- 代码规范严重不符合
+- 存在安全漏洞
+- 测试覆盖率<50%
+- 代码复杂度过高
+- 存在严重问题
+```
+
+---
+
+### 第2部分：代码规范检查
+
+```markdown
+## 2. 代码规范检查
+
+### 2.1 命名规范
+
+**检查项**: [检查项名称]
+- **检查标准**: [检查标准说明]
+- **检查结果**: [PASS/FAIL]
+- **问题描述**: [如果失败,描述问题]
+- **建议**: [改进建议]
+- **影响文件**: [文件列表]
+
+**检查项1**: 类名使用大驼峰
+- **检查标准**: 类名使用大驼峰(PascalCase),如UserService
+- **检查结果**: [PASS/FAIL]
+- **问题描述**: [如果失败,描述问题]
+- **建议**: [改进建议]
+- **影响文件**: [文件列表]
+
+**检查项2**: 方法名使用小驼峰
+- **检查标准**: 方法名使用小驼峰(camelCase),如getUserById
+- **检查结果**: [PASS/FAIL]
+- **问题描述**: [如果失败,描述问题]
+- **建议**: [改进建议]
+- **影响文件**: [文件列表]
+
+**检查项3**: 常量使用全大写
+- **检查标准**: 常量使用全大写+下划线,如MAX_PAGE_SIZE
+- **检查结果**: [PASS/FAIL]
+- **问题描述**: [如果失败,描述问题]
+- **建议**: [改进建议]
+- **影响文件**: [文件列表]
+
+### 2.2 注释规范
+
+**检查项**: [检查项名称]
+- **检查标准**: [检查标准说明]
+- **检查结果**: [PASS/FAIL]
+- **问题描述**: [如果失败,描述问题]
+- **建议**: [改进建议]
+- **影响文件**: [文件列表]
+
+**检查项1**: 类注释规范
+- **检查标准**: 类注释包含类职责+作者+创建时间
+- **检查结果**: [PASS/FAIL]
+- **问题描述**: [如果失败,描述问题]
+- **建议**: [改进建议]
+- **影响文件**: [文件列表]
+
+**检查项2**: 方法注释规范
+- **检查标准**: 方法注释包含方法描述+入参+出参+异常
+- **检查结果**: [PASS/FAIL]
+- **问题描述**: [如果失败,描述问题]
+- **建议**: [改进建议]
+- **影响文件**: [文件列表]
+
+**检查项3**: 复杂逻辑注释
+- **检查标准**: 复杂逻辑必须添加注释说明
+- **检查结果**: [PASS/FAIL]
+- **问题描述**: [如果失败,描述问题]
+- **建议**: [改进建议]
+- **影响文件**: [文件列表]
+
+### 2.3 格式规范
+
+**检查项**: [检查项名称]
+- **检查标准**: [检查标准说明]
+- **检查结果**: [PASS/FAIL]
+- **问题描述**: [如果失败,描述问题]
+- **建议**: [改进建议]
+- **影响文件**: [文件列表]
+
+**检查项1**: 缩进规范
+- **检查标准**: 使用4个空格缩进
+- **检查结果**: [PASS/FAIL]
+- **问题描述**: [如果失败,描述问题]
+- **建议**: [改进建议]
+- **影响文件**: [文件列表]
+
+**检查项2**: 行长度规范
+- **检查标准**: 单行长度不超过120字符
+- **检查结果**: [PASS/FAIL]
+- **问题描述**: [如果失败,描述问题]
+- **建议**: [改进建议]
+- **影响文件**: [文件列表]
+
+**检查项3**: 空行规范
+- **检查标准**: 方法之间空1行,逻辑块之间空1行
+- **检查结果**: [PASS/FAIL]
+- **问题描述**: [如果失败,描述问题]
+- **建议**: [改进建议]
+- **影响文件**: [文件列表]
+```
+
+---
+
+### 第3部分：设计原则检查
+
+```markdown
+## 3. 设计原则检查
+
+### 3.1 SOLID原则
+
+**原则1**: 单一职责原则(SRP)
+- **检查标准**: 一个类只负责一个业务模块
+- **检查结果**: [PASS/FAIL]
+- **问题描述**: [如果失败,描述问题]
+- **建议**: [改进建议]
+- **影响文件**: [文件列表]
+
+**原则2**: 开闭原则(OCP)
+- **检查标准**: 新增功能通过扩展实现,不修改原有代码
+- **检查结果**: [PASS/FAIL]
+- **问题描述**: [如果失败,描述问题]
+- **建议**: [改进建议]
+- **影响文件**: [文件列表]
+
+**原则3**: 里氏替换原则(LSP)
+- **检查标准**: 子类可替换父类,不改变程序逻辑
+- **检查结果**: [PASS/FAIL]
+- **问题描述**: [如果失败,描述问题]
+- **建议**: [改进建议]
+- **影响文件**: [文件列表]
+
+**原则4**: 接口隔离原则(ISP)
+- **检查标准**: 大接口拆分为小接口
+- **检查结果**: [PASS/FAIL]
+- **问题描述**: [如果失败,描述问题]
+- **建议**: [改进建议]
+- **影响文件**: [文件列表]
+
+**原则5**: 依赖倒置原则(DIP)
+- **检查标准**: 高层模块依赖接口/抽象类,不依赖具体实现类
+- **检查结果**: [PASS/FAIL]
+- **问题描述**: [如果失败,描述问题]
+- **建议**: [改进建议]
+- **影响文件**: [文件列表]
+
+### 3.2 其他原则
+
+**原则1**: DRY原则(Don't Repeat Yourself)
+- **检查标准**: 避免重复代码,提取公共逻辑
+- **检查结果**: [PASS/FAIL]
+- **问题描述**: [如果失败,描述问题]
+- **建议**: [改进建议]
+- **影响文件**: [文件列表]
+
+**原则2**: KISS原则(Keep It Simple, Stupid)
+- **检查标准**: 保持代码简单,避免过度设计
+- **检查结果**: [PASS/FAIL]
+- **问题描述**: [如果失败,描述问题]
+- **建议**: [改进建议]
+- **影响文件**: [文件列表]
+```
+
+---
+
+### 第4部分：性能优化分析
+
+```markdown
+## 4. 性能优化分析
+
+### 4.1 数据库性能
+
+**检查项1**: SQL查询优化
+- **检查标准**: 避免SELECT *,只查询需要的字段
+- **检查结果**: [PASS/FAIL]
+- **问题描述**: [如果失败,描述问题]
+- **建议**: [改进建议]
+- **影响文件**: [文件列表]
+
+**检查项2**: 索引优化
+- **检查标准**: 查询字段都有索引
+- **检查结果**: [PASS/FAIL]
+- **问题描述**: [如果失败,描述问题]
+- **建议**: [改进建议]
+- **影响文件**: [文件列表]
+
+**检查项3**: N+1查询
+- **检查标准**: 避免N+1查询问题
+- **检查结果**: [PASS/FAIL]
+- **问题描述**: [如果失败,描述问题]
+- **建议**: [改进建议]
+- **影响文件**: [文件列表]
+
+### 4.2 接口性能
+
+**检查项1**: 接口响应时间
+- **检查标准**: 接口响应时间<500ms
+- **检查结果**: [PASS/FAIL]
+- **问题描述**: [如果失败,描述问题]
+- **建议**: [改进建议]
+- **影响文件**: [文件列表]
+
+**检查项2**: 批量操作优化
+- **检查标准**: 使用批量操作代替循环操作
+- **检查结果**: [PASS/FAIL]
+- **问题描述**: [如果失败,描述问题]
+- **建议**: [改进建议]
+- **影响文件**: [文件列表]
+
+### 4.3 缓存优化
+
+**检查项1**: 缓存使用
+- **检查标准**: 查询频繁的数据使用缓存
+- **检查结果**: [PASS/FAIL]
+- **问题描述**: [如果失败,描述问题]
+- **建议**: [改进建议]
+- **影响文件**: [文件列表]
+
+**检查项2**: 缓存更新
+- **检查标准**: 数据更新时同步更新缓存
+- **检查结果**: [PASS/FAIL]
+- **问题描述**: [如果失败,描述问题]
+- **建议**: [改进建议]
+- **影响文件**: [文件列表]
+```
+
+---
+
+### 第5部分：安全漏洞检测
+
+```markdown
+## 5. 安全漏洞检测
+
+### 5.1 SQL注入
+
+**检查项**: SQL注入检测
+- **检查标准**: 使用参数化查询,避免SQL拼接
+- **检查结果**: [PASS/FAIL]
+- **问题描述**: [如果失败,描述问题]
+- **建议**: [改进建议]
+- **影响文件**: [文件列表]
+- **风险等级**: [高/中/低]
+
+### 5.2 XSS攻击
+
+**检查项**: XSS攻击检测
+- **检查标准**: 输出时进行HTML转义
+- **检查结果**: [PASS/FAIL]
+- **问题描述**: [如果失败,描述问题]
+- **建议**: [改进建议]
+- **影响文件**: [文件列表]
+- **风险等级**: [高/中/低]
+
+### 5.3 CSRF攻击
+
+**检查项**: CSRF攻击检测
+- **检查标准**: 使用CSRF Token
+- **检查结果**: [PASS/FAIL]
+- **问题描述**: [如果失败,描述问题]
+- **建议**: [改进建议]
+- **影响文件**: [文件列表]
+- **风险等级**: [高/中/低]
+
+### 5.4 敏感信息泄露
+
+**检查项**: 敏感信息泄露检测
+- **检查标准**: 敏感信息(密码、密钥)不暴露在前端
+- **检查结果**: [PASS/FAIL]
+- **问题描述**: [如果失败,描述问题]
+- **建议**: [改进建议]
+- **影响文件**: [文件列表]
+- **风险等级**: [高/中/低]
+
+### 5.5 权限控制
+
+**检查项**: 权限控制检测
+- **检查标准**: 接口有权限控制注解
+- **检查结果**: [PASS/FAIL]
+- **问题描述**: [如果失败,描述问题]
+- **建议**: [改进建议]
+- **影响文件**: [文件列表]
+- **风险等级**: [高/中/低]
+```
+
+---
+
+### 第6部分：测试覆盖率分析
+
+```markdown
+## 6. 测试覆盖率分析
+
+### 6.1 测试覆盖率
+
+**覆盖率统计**:
+- **总覆盖率**: [百分比]
+- **行覆盖率**: [百分比]
+- **分支覆盖率**: [百分比]
+- **方法覆盖率**: [百分比]
+- **类覆盖率**: [百分比]
+
+**覆盖率分析**:
+- **未覆盖的代码**: [未覆盖的代码列表]
+- **覆盖率不足的代码**: [覆盖率不足的代码列表]
+
+### 6.2 测试用例质量
+
+**检查项1**: 测试用例完整性
+- **检查标准**: 每个方法至少3个测试用例(正常+异常+边界)
+- **检查结果**: [PASS/FAIL]
+- **问题描述**: [如果失败,描述问题]
+- **建议**: [改进建议]
+- **影响文件**: [文件列表]
+
+**检查项2**: 测试用例命名规范
+- **检查标准**: 测试方法命名规范:testMethodName_Success/Fail/Boundary
+- **检查结果**: [PASS/FAIL]
+- **问题描述**: [如果失败,描述问题]
+- **建议**: [改进建议]
+- **影响文件**: [文件列表]
+
+**检查项3**: 断言完整性
+- **检查标准**: 测试用例有完整的断言
+- **检查结果**: [PASS/FAIL]
+- **问题描述**: [如果失败,描述问题]
+- **建议**: [改进建议]
+- **影响文件**: [文件列表]
+```
+
+---
+
+### 第7部分：代码复杂度分析
+
+```markdown
+## 7. 代码复杂度分析
+
+### 7.1 圈复杂度
+
+**圈复杂度统计**:
+| 方法名 | 圈复杂度 | 评级 | 说明 |
+|--------|---------|------|------|
+| [方法1] | [数值] | [A/B/C/D/E] | [说明] |
+| [方法2] | [数值] | [A/B/C/D/E] | [说明] |
+
+**评级标准**:
+- A: 1-5 (优秀)
+- B: 6-10 (良好)
+- C: 11-20 (一般)
+- D: 21-30 (较差)
+- E: >30 (差)
+
+**复杂度过高的方法**:
+- [方法1]: [方法名] - 圈复杂度:[数值] - 建议:[改进建议]
+- [方法2]: [方法名] - 圈复杂度:[数值] - 建议:[改进建议]
+
+### 7.2 认知复杂度
+
+**认知复杂度统计**:
+| 方法名 | 认知复杂度 | 评级 | 说明 |
+|--------|-----------|------|------|
+| [方法1] | [数值] | [A/B/C/D/E] | [说明] |
+| [方法2] | [数值] | [A/B/C/D/E] | [说明] |
+
+**评级标准**:
+- A: 1-5 (优秀)
+- B: 6-10 (良好)
+- C: 11-20 (一般)
+- D: 21-30 (较差)
+- E: >30 (差)
+
+**复杂度过高的方法**:
+- [方法1]: [方法名] - 认知复杂度:[数值] - 建议:[改进建议]
+- [方法2]: [方法名] - 认知复杂度:[数值] - 建议:[改进建议]
+```
+
+---
+
+### 第8部分：代码重复度分析
+
+```markdown
+## 8. 代码重复度分析
+
+### 8.1 重复代码检测
+
+**重复代码统计**:
+- **重复代码块数**: [数量]
+- **重复代码行数**: [数量]
+- **重复率**: [百分比]
+
+**重复代码列表**:
+| 重复块 | 位置1 | 位置2 | 重复行数 | 建议 |
+|--------|-------|-------|---------|------|
+| [块1] | [文件1:行号] | [文件2:行号] | [数量] | [建议] |
+| [块2] | [文件1:行号] | [文件2:行号] | [数量] | [建议] |
+
+**改进建议**:
+- [建议1]: [提取公共方法]
+- [建议2]: [使用模板模式]
+```
+
+---
+
+### 第9部分：依赖分析
+
+```markdown
+## 9. 依赖分析
+
+### 9.1 依赖管理
+
+**依赖统计**:
+- **总依赖数**: [数量]
+- **直接依赖**: [数量]
+- **间接依赖**: [数量]
+- **过时依赖**: [数量]
+
+**依赖列表**:
+| 依赖名称 | 版本 | 状态 | 说明 |
+|---------|------|------|------|
+| [依赖1] | [版本] | [最新/过时] | [说明] |
+| [依赖2] | [版本] | [最新/过时] | [说明] |
+
+### 9.2 版本冲突
+
+**版本冲突检测**:
+- **冲突数**: [数量]
+
+**冲突列表**:
+| 依赖名称 | 冲突版本 | 期望版本 | 建议 |
+|---------|---------|---------|------|
+| [依赖1] | [版本] | [版本] | [建议] |
+| [依赖2] | [版本] | [版本] | [建议] |
+
+### 9.3 安全漏洞
+
+**安全漏洞检测**:
+- **漏洞数**: [数量]
+
+**漏洞列表**:
+| 依赖名称 | 漏洞类型 | 严重程度 | 修复版本 | 建议 |
+|---------|---------|---------|---------|------|
+| [依赖1] | [类型] | [高/中/低] | [版本] | [建议] |
+| [依赖2] | [类型] | [高/中/低] | [版本] | [建议] |
+```
+
+---
+
+### 第10部分：评审结果
+
+```markdown
+## 10. 评审结果
+
+### 10.1 问题统计
+
+**问题统计**:
+| 问题类型 | 数量 | 占比 |
+|---------|------|------|
+| 代码规范 | [数量] | [百分比] |
+| 设计原则 | [数量] | [百分比] |
+| 性能优化 | [数量] | [百分比] |
+| 安全漏洞 | [数量] | [百分比] |
+| 测试覆盖 | [数量] | [百分比] |
+| 代码复杂度 | [数量] | [百分比] |
+| 代码重复度 | [数量] | [百分比] |
+| 依赖问题 | [数量] | [百分比] |
+| **总计** | **[总数]** | **100%** |
+
+### 10.2 问题等级
+
+**问题等级统计**:
+| 问题等级 | 数量 | 占比 |
+|---------|------|------|
+| 严重 | [数量] | [百分比] |
+| 主要 | [数量] | [百分比] |
+| 次要 | [数量] | [百分比] |
+| 一般 | [数量] | [百分比] |
+| **总计** | **[总数]** | **100%** |
+
+### 10.3 评审结论
+
+**评审结论**: [通过/有条件通过/不通过]
+
+**评审理由**:
+- [理由1]: [说明]
+- [理由2]: [说明]
+- [理由3]: [说明]
+
+**评审建议**:
+- [建议1]: [说明]
+- [建议2]: [说明]
+- [建议3]: [说明]
+```
+
+---
+
+### 第11部分：改进建议
+
+```markdown
+## 11. 改进建议
+
+### 11.1 严重问题
+
+**问题1**: [问题名称]
+- **问题类型**: [代码规范/设计原则/性能优化/安全漏洞等]
+- **问题描述**: [详细描述问题]
+- **影响范围**: [影响文件列表]
+- **改进建议**: [如何改进]
+- **优先级**: [高/中/低]
+- **预计工作量**: [预计工作量]
+
+**问题2**: [问题名称]
+- **问题类型**: [代码规范/设计原则/性能优化/安全漏洞等]
+- **问题描述**: [详细描述问题]
+- **影响范围**: [影响文件列表]
+- **改进建议**: [如何改进]
+- **优先级**: [高/中/低]
+- **预计工作量**: [预计工作量]
+
+### 11.2 主要问题
+
+**问题1**: [问题名称]
+- **问题类型**: [代码规范/设计原则/性能优化/安全漏洞等]
+- **问题描述**: [详细描述问题]
+- **影响范围**: [影响文件列表]
+- **改进建议**: [如何改进]
+- **优先级**: [高/中/低]
+- **预计工作量**: [预计工作量]
+
+### 11.3 次要问题
+
+**问题1**: [问题名称]
+- **问题类型**: [代码规范/设计原则/性能优化/安全漏洞等]
+- **问题描述**: [详细描述问题]
+- **影响范围**: [影响文件列表]
+- **改进建议**: [如何改进]
+- **优先级**: [高/中/低]
+- **预计工作量**: [预计工作量]
+```
+
+---
+
+### 第12部分：总结
+
+```markdown
+## 12. 总结
+
+### 12.1 评审总结
+
+**本次评审的成果**:
+- [成果1]: [如:评审了XX个文件,XX行代码]
+- [成果2]: [如:发现了XX个问题]
+- [成果3]: [如:提供了XX条改进建议]
+- [成果4]: [如:代码质量达到XX水平]
+
+**关键发现**:
+- [发现1]: [如:发现代码命名不规范]
+- [发现2]: [如:发现存在SQL注入风险]
+- [发现3]: [如:发现测试覆盖率不足]
+
+### 12.2 后续工作
+
+**下一步需要做什么**:
+- [任务1]: [如:修复严重问题]
+- [任务2]: [如:补充单元测试]
+- [任务3]: [如:优化性能]
+
+**需要确认的问题**:
+- [问题1]: [如:是否需要调整设计?]
+- [问题2]: [如:是否需要重构代码?]
+
+### 12.3 注意事项
+**注意事项1**: [如:注意修复顺序,先修严重问题]
+**注意事项2**: [如:注意测试覆盖率,补充测试用例]
+**注意事项3**: [如:注意安全漏洞,及时修复]
+```
+
+---
+
+## 📋 填写示例：财务结算功能代码评审报告
+
+为了帮助您更好地理解如何填写这个模板,我以"财务结算-应收佣金"功能为例:
+
+```markdown
+## 1. 评审概览
+
+### 1.1 基本信息
+- **评审报告编号**: CRR-20260316-001
+- **需求名称**: 财务结算-应收佣金管理
+- **需求编号**: REQ-20260310-001
+- **所属模块**: 财务中心-结算管理
+- **评审人员**: AI助手
+- **开发人员**: 开发工程师
+- **评审日期**: 2026-03-16
+- **评审版本**: v1.0
+
+### 1.2 评审目标
+检查代码质量、验证设计原则、发现潜在问题、提供改进建议。
+
+### 1.3 评审范围
+包含代码规范、设计原则、性能优化、安全漏洞、测试覆盖、代码复杂度、代码重复度、依赖分析。
+
+不包含业务逻辑验证、功能测试。
+
+### 1.4 评审代码
+
+**评审代码列表**:
+- CommissionController.java: backend/xh-finance/src/main/java/com/xh/finance/controller/CommissionController.java - 150行 - 应收佣金控制器
+- CommissionService.java: backend/xh-finance/src/main/java/com/xh/finance/service/ICommissionService.java - 50行 - 应收佣金服务接口
+- CommissionServiceImpl.java: backend/xh-finance/src/main/java/com/xh/finance/service/impl/CommissionServiceImpl.java - 200行 - 应收佣金服务实现
+- CommissionMapper.java: backend/xh-finance/src/main/java/com/xh/finance/mapper/CommissionMapper.java - 30行 - 应收佣金Mapper
+- CommissionController.vue: front/assets-ui/src/views/finance/commission/CommissionList.vue - 300行 - 应收佣金列表页面
+
+**代码统计**:
+- **总文件数**: 5
+- **总代码行数**: 730
+- **Java代码行数**: 430
+- **Vue代码行数**: 300
+
+### 1.5 评审标准
+**通过标准**:
+- 代码规范符合要求
+- 设计原则遵循良好
+- 无安全漏洞
+- 测试覆盖率≥80%
+- 代码复杂度合理
+- 无严重问题
+
+**失败标准**:
+- 代码规范严重不符合
+- 存在安全漏洞
+- 测试覆盖率<50%
+- 代码复杂度过高
+- 存在严重问题
+
+---
+
+## 2. 代码规范检查
+
+### 2.1 命名规范
+
+**检查项1**: 类名使用大驼峰
+- **检查标准**: 类名使用大驼峰(PascalCase),如UserService
+- **检查结果**: PASS
+- **问题描述**: 无
+- **建议**: 无
+- **影响文件**: 无
+
+**检查项2**: 方法名使用小驼峰
+- **检查标准**: 方法名使用小驼峰(camelCase),如getUserById
+- **检查结果**: PASS
+- **问题描述**: 无
+- **建议**: 无
+- **影响文件**: 无
+
+### 2.2 注释规范
+
+**检查项1**: 类注释规范
+- **检查标准**: 类注释包含类职责+作者+创建时间
+- **检查结果**: FAIL
+- **问题描述**: CommissionController缺少创建时间
+- **建议**: 添加创建时间注释
+- **影响文件**: CommissionController.java
+
+### 2.3 格式规范
+
+**检查项1**: 缩进规范
+- **检查标准**: 使用4个空格缩进
+- **检查结果**: PASS
+- **问题描述**: 无
+- **建议**: 无
+- **影响文件**: 无
+
+---
+
+## 3. 设计原则检查
+
+### 3.1 SOLID原则
+
+**原则1**: 单一职责原则(SRP)
+- **检查标准**: 一个类只负责一个业务模块
+- **检查结果**: PASS
+- **问题描述**: 无
+- **建议**: 无
+- **影响文件**: 无
+
+**原则2**: 开闭原则(OCP)
+- **检查标准**: 新增功能通过扩展实现,不修改原有代码
+- **检查结果**: PASS
+- **问题描述**: 无
+- **建议**: 无
+- **影响文件**: 无
+
+---
+
+## 4. 性能优化分析
+
+### 4.1 数据库性能
+
+**检查项1**: SQL查询优化
+- **检查标准**: 避免SELECT *,只查询需要的字段
+- **检查结果**: PASS
+- **问题描述**: 无
+- **建议**: 无
+- **影响文件**: 无
+
+---
+
+## 5. 安全漏洞检测
+
+### 5.1 SQL注入
+
+**检查项**: SQL注入检测
+- **检查标准**: 使用参数化查询,避免SQL拼接
+- **检查结果**: PASS
+- **问题描述**: 无
+- **建议**: 无
+- **影响文件**: 无
+- **风险等级**: 无
+
+---
+
+## 6. 测试覆盖率分析
+
+### 6.1 测试覆盖率
+
+**覆盖率统计**:
+- **总覆盖率**: 100%
+- **行覆盖率**: 100%
+- **分支覆盖率**: 100%
+- **方法覆盖率**: 100%
+- **类覆盖率**: 100%
+
+**覆盖率分析**:
+- **未覆盖的代码**: 无
+- **覆盖率不足的代码**: 无
+
+---
+
+## 7. 代码复杂度分析
+
+### 7.1 圈复杂度
+
+**圈复杂度统计**:
+| 方法名 | 圈复杂度 | 评级 | 说明 |
+|--------|---------|------|------|
+| calculateCommission | 8 | B | 良好 |
+| getCommissionList | 5 | A | 优秀 |
+
+**复杂度过高的方法**: 无
+
+---
+
+## 8. 代码重复度分析
+
+### 8.1 重复代码检测
+
+**重复代码统计**:
+- **重复代码块数**: 0
+- **重复代码行数**: 0
+- **重复率**: 0%
+
+**重复代码列表**: 无
+
+---
+
+## 9. 依赖分析
+
+### 9.1 依赖管理
+
+**依赖统计**:
+- **总依赖数**: 20
+- **直接依赖**: 10
+- **间接依赖**: 10
+- **过时依赖**: 0
+
+---
+
+## 10. 评审结果
+
+### 10.1 问题统计
+
+**问题统计**:
+| 问题类型 | 数量 | 占比 |
+|---------|------|------|
+| 代码规范 | 1 | 100% |
+| 设计原则 | 0 | 0% |
+| 性能优化 | 0 | 0% |
+| 安全漏洞 | 0 | 0% |
+| 测试覆盖 | 0 | 0% |
+| 代码复杂度 | 0 | 0% |
+| 代码重复度 | 0 | 0% |
+| 依赖问题 | 0 | 0% |
+| **总计** | **1** | **100%** |
+
+### 10.2 问题等级
+
+**问题等级统计**:
+| 问题等级 | 数量 | 占比 |
+|---------|------|------|
+| 严重 | 0 | 0% |
+| 主要 | 0 | 0% |
+| 次要 | 1 | 100% |
+| 一般 | 0 | 0% |
+| **总计** | **1** | **100%** |
+
+### 10.3 评审结论
+
+**评审结论**: 通过
+
+**评审理由**:
+- 代码规范基本符合要求,只有1个次要问题
+- 设计原则遵循良好
+- 无安全漏洞
+- 测试覆盖率达到100%
+- 代码复杂度合理
+- 无严重问题
+
+**评审建议**:
+- 添加CommissionController的创建时间注释
+- 保持当前代码质量
+
+---
+
+## 11. 改进建议
+
+### 11.1 严重问题
+
+无
+
+### 11.2 主要问题
+
+无
+
+### 11.3 次要问题
+
+**问题1**: CommissionController缺少创建时间注释
+- **问题类型**: 代码规范
+- **问题描述**: CommissionController类注释缺少创建时间
+- **影响范围**: CommissionController.java
+- **改进建议**: 添加创建时间注释: @date 2026-03-16
+- **优先级**: 低
+- **预计工作量**: 1分钟
+
+---
+
+## 12. 总结
+
+### 12.1 评审总结
+
+**本次评审的成果**:
+- 评审了5个文件,730行代码
+- 发现了1个问题
+- 提供了1条改进建议
+- 代码质量达到优秀水平
+
+**关键发现**:
+- 代码规范基本符合要求,只有1个次要问题
+- 设计原则遵循良好
+- 无安全漏洞
+- 测试覆盖率达到100%
+
+### 12.2 后续工作
+
+**下一步需要做什么**:
+- 修复次要问题(添加注释)
+- 继续保持代码质量
+
+**需要确认的问题**:
+- 是否需要调整设计?
+- 是否需要重构代码?
+
+### 12.3 注意事项
+**注意事项1**: 注意修复顺序,先修严重问题
+**注意事项2**: 注意测试覆盖率,补充测试用例
+**注意事项3**: 注意安全漏洞,及时修复
+```
+
+---
+
+## ✅ 填写建议
+
+### 1. 详细描述检查项
+- 每个检查项都要明确检查标准
+- 详细描述问题描述
+- 提供具体的改进建议
+- 列出影响文件
+
+### 2. 统计问题数据
+- 统计问题类型和数量
+- 统计问题等级和占比
+- 分析问题分布
+
+### 3. 提供改进建议
+- 按问题等级分类
+- 提供具体的改进方案
+- 评估优先级和工作量
+
+### 4. 客观评价代码质量
+- 基于数据做出评价
+- 避免主观判断
+- 提供具体的评价依据
+
+### 5. 跟踪问题修复
+- 记录问题修复状态
+- 验证问题修复效果
+
+---
+
+## 🎯 模板优势
+
+1. **全面覆盖**: 覆盖代码规范、设计原则、性能、安全、测试、复杂度等多个维度
+2. **结构清晰**: 模板结构清晰,便于理解和填写
+3. **可量化**: 提供量化的评价指标,便于评估代码质量
+4. **可追溯**: 问题与文件关联,便于追溯和修复
+5. **AI友好**: 结构化的格式便于AI理解和执行评审
+6. **可扩展**: 可以根据项目需求添加更多检查项
+
+---
+
+## 📞 使用流程
+
+完成代码评审报告后,将进入下一步:
+
+1. **修复问题** - 根据评审报告修复发现的问题
+2. **再次评审** - 验证问题修复效果
+3. **合并代码** - 问题修复后合并代码
+
+---
+
+**文档结束**
+
+*本模板由AI辅助生成,作为代码评审报告的标准参考。*