Code Review

Code Architecture Review

代码架构评审技能，从架构角度审查代码质量和设计。

评审维度

1. 设计模式应用

正面模式（应采用）

反面模式（应避免）

2. SOLID 原则检查

□ S - Single Responsibility (单一职责)
  - 类是否只有一个变化原因？
  - 方法是否保持简短？

□ O - Open/Closed (开闭原则)
  - 对扩展开放，对修改封闭？
  - 使用继承还是组合？

□ L - Liskov Substitution (里氏替换)
  - 子类能否替换父类？
  - 是否违反继承契约？

□ I - Interface Segregation (接口隔离)
  - 接口是否臃肿？
  - 客户端是否被迫依赖未使用的方法？

□ D - Dependency Inversion (依赖反转)
  - 依赖抽象而非具体？
  - 是否使用依赖注入？

3. 代码复杂度

4. 架构层次检查

□ 是否遵循确定的分层架构？
□ 依赖方向是否正确（外层依赖内层）？
□ 是否有跨层直接调用？
□ 核心业务逻辑是否在正确的层？

5. 错误处理

□ 是否有统一的异常处理机制？
□ 异常是否被适当捕获和记录？
□ 是否避免了吞掉异常（空 catch）？
□ 自定义异常是否有意义？

6. 数据访问

□ 是否使用 Repository 模式？
□ 是否存在 N+1 查询问题？
□ 事务边界是否清晰？
□ 是否考虑缓存策略？

代码评审报告模板

# 代码架构评审报告

## 1. 评审概要
- **评审范围**：[文件/模块列表]
- **评审时间**：[日期]
- **整体评级**：[A/B/C/D]

## 2. 主要发现

### 严重问题 (必须修复)
| 问题 | 位置 | 描述 | 建议 |
|------|------|------|------|
|      |      |      |      |

### 中等问题 (建议修复)
| 问题 | 位置 | 描述 | 建议 |
|------|------|------|------|
|      |      |      |      |

### 轻微问题 (可选修复)
| 问题 | 位置 | 描述 | 建议 |
|------|------|------|------|
|      |      |      |      |

## 3. 架构亮点
[做得好的设计决策]

## 4. 改进建议优先级
1. [最优先]
2. [次优先]
3. [后续]

## 5. 总结
[总体评价]

评审 Checklist

评审前
□ 理解需求和业务背景
□ 了解技术栈和约束
□ 确定评审重点

评审中
□ 从整体到局部，从粗到细
□ 记录所有发现（不只是问题，也包括亮点）
□ 标注问题优先级

评审后
□ 整理评审报告
□ 与开发者对齐问题
□ 确认后续行动计划

常见架构问题速查