DDD Code Reviewer

DDD 代码审查 + 反模式检测 + 5 维度合规评分。验证代码是否遵循 DDD 最佳实践，输出结构化审查报告。

Workflow

DDD 代码审查三步走：

Step 1: 代码扫描 — 反模式检测清单逐项检查 + 源代码扫描
Step 2: 评分计算 — 5 维度逐项评分（分层/领域/命名/结构/测试）
Step 3: 报告输出 — 结构化审查报告（评分 + 反模式列表 + 修复建议）

每次审查必须走完三步，输出完整报告。只口述问题不输出文档，团队无法追踪改进。

When to Use

触发词（触发即用）

代码审查 DDD review 反模式检测 anti-pattern 贫血模型 充血模型检查 架构审查 分层合规 代码质量评分 layering compliance rich domain check review my DDD code

适用前提

不适用场景

Audience

This skill is designed for: Backend developers (implementing DDD architectures), Software architects (evaluating and selecting patterns), Tech leads (reviewing team implementations), and DDD beginners (learning domain-driven design fundamentals).

Rules

1. Every code review must include a 5-dimension scoring report.
2. P0 anti-patterns (anemia model, layer violation, cross-aggregate ref) block merge.
3. Domain layer must have zero framework dependencies — verified via ArchUnit.
4. Review reports must include fix suggestions ordered by priority (P0→P1→P2).

Anti-Pattern Checklist

12 种 DDD 反模式，按 P0（阻塞合并）/P1（下个版本前修复）/P2（持续改进）分级。

• P0 — 必须修复: 贫血模型、上帝 Service、跨聚合直接引用、领域层框架依赖、循环依赖、Repository 返回非聚合根
• P1 — 应该修复: Controller 业务逻辑、Application Service 有 SQL、值对象可变
• P2 — 持续改进: 聚合过大、缺少领域事件、跨聚合事务

完整反模式速查表见 references/checklist.md，含 Java 代码示例和修复路径。

Layered Compliance Matrix

分层依赖检查规则（基于 ArchUnit）：

    ┌─────────────────┬─────┬─────┬─────┬─────┐
    │ 层 / 可依赖     │ Infra│ Dom │ App │ Adap│
    ├─────────────────┼─────┼─────┼─────┼─────┤
    │ Infrastructure  │  ✓  │  ✗  │  ✗  │  ✗  │
    │ Domain          │  ✗  │  ✓  │  ✗  │  ✗  │
    │ Application     │  ✓  │  ✓  │  ✓  │  ✗  │
    │ Adapter         │  ✓  │  ✓  │  ✓  │  ✓  │
    └─────────────────┴─────┴─────┴─────┴─────┘

Domain 层零依赖规则（P0）：

• ✗ import org.springframework.stereotype.Service
• ✗ import javax.persistence.Entity
• ✗ import org.apache.ibatis.annotations.Mapper
• ✓ import java.util.Optional
• ✓ import java.math.BigDecimal

ArchUnit 完整配置见 references/archunit-config.md。

Rich Domain Model Validation

Rich models encapsulate behavior in entities (pass); anemic models expose state via getters/setters (fail). Key: behavior in Entity vs behavior in Service.

完整代码示例和重构对比见 examples/rich-model-refactoring.md，含 3 个实战案例。

Scoring System

5 维度评分

总分 = Σ(维度得分 × 权重)

分数等级

评分细则和计算示例见 references/scoring-criteria.md、examples/scoring-example.md。

Review Report Template

每次审查输出结构化报告，包含：

# DDD Code Review Report

## Overall Score: 78/100 (🟡 B)

### 分层合规 (24/30)
| 检查项 | 结果 | 说明 |
|--------|:----:|------|
| Domain 零依赖 | ✅ | 无框架依赖 |
| App 层无 SQL | ❌ | OrderAppService L45 直接调用 Mapper |
| 依赖方向 | ✅ | 无反向依赖 |

### 领域模型质量 (22/30)
| 检查项 | 结果 | 说明 |
|--------|:----:|------|
| 充血模型 | ⚠️ | User 实体仍为贫血模型 |
| 值对象 | ⚠️ | Money/Email 已用，Phone 仍为 String |

### 反模式清单
| 严重级别 | 反模式 | 位置 | 修复建议 |
|:------:|--------|------|---------|
| P0 | 上帝 Service | OrderService.java:45-320 | 拆分为 OrderPricingService + OrderFulfillmentService |

### 改进建议（按优先级）
1. [P0] 将 OrderAppService 中的 SQL 移到 Repository 实现
2. [P1] 将 User 实体改为充血模型
3. [P2] 为关键业务操作补充领域事件

完整模板见 references/report-template.md、示例见 examples/review-report-example.md。

Gotchas

常见审查陷阱见 references/gotchas.md。

FAQ

Security & Safety

This skill is pure documentation. It contains no executable scripts, collects no user data, accesses no external services or networks.

Keywords

DDD 代码审查、反模式检测、贫血模型、充血模型、上帝 Service、分层合规、依赖方向、ArchUnit、领域事件、值对象不可变、聚合设计、5 维度评分、代码质量门禁

References

Examples

Next Steps

审查完成后，根据结果进入后续步骤：

• 架构级别评估 → ddd-architecture-evaluator
• 修复架构目录结构 → 对应架构 Skill（ddd-architecture-layered / ddd-architecture-hexagonal / ddd-architecture-cola 等）
• 提升测试覆盖率 → ddd-testing-strategist
• 输出架构文档 → ddd-architecture-doc