ドメイン駆動設計(DDD)の原則に従った実装がなされているかを検証します。AIDLC(AI-Driven Life Cycle)ドキュメントと実装コードの一貫性を確認し、ドメインモデル違反を早期に検出します。
スキル呼び出し時に検証対象を指定できます:
```
/ddd-check
```
または特定のファイル/ディレクトリのみ検証:
```
/ddd-check --domain-only
/ddd-check --file src/domain/entities/cart.py
```
**検証内容**:
**確認ファイル**:
```bash
main/aidlc-docs/construction/unit_01_ai_dialog_public/docs/ubiquitous_language.md
main/backend/src/domain/**/*.py
```
**チェック項目**:
#### 1. コアドメイン用語の使用
| 日本語 | 英語 | コード内での使用例 |
|--------|------|-------------------|
| 買い目 | BetSelection | `class BetSelection` (値オブジェクト) |
| カート | Cart | `class Cart` (エンティティ) |
| カートアイテム | CartItem | `class CartItem` (エンティティ) |
| 相談セッション | ConsultationSession | `class ConsultationSession` (集約ルート) |
| メッセージ | Message | `class Message` (エンティティ) |
| データフィードバック | DataFeedback | `class DataFeedback` (値オブジェクト) |
| 掛け金フィードバック | AmountFeedback | `class AmountFeedback` (値オブジェクト) |
**検証方法**:
```bash
grep -r "class BetSelection" main/backend/src/domain/value_objects/
grep -r "class Cart" main/backend/src/domain/entities/
```
**違反パターン**:
#### 2. サポートドメイン用語の使用
| 日本語 | 英語 | 扱い |
|--------|------|------|
| レース | Race | 外部データ(Read Model) |
| 開催場 | Venue | 外部データ |
| 出走馬 | Runner | 外部データ |
| 騎手 | Jockey | 外部データ |
| オッズ | Odds | 外部データ |
**検証方法**:
**違反パターン**:
**ディレクトリ構造チェック**:
```
main/backend/src/
├── domain/ # ドメイン層(ビジネスロジック)
│ ├── entities/ # エンティティ(識別子を持つ)
│ ├── value_objects/ # 値オブジェクト(イミュータブル)
│ ├── services/ # ドメインサービス
│ ├── ports/ # インターフェース(ポート)
│ ├── identifiers/ # 識別子(ID型)
│ └── enums/ # 列挙型
├── application/ # アプリケーション層(ユースケース)
│ └── use_cases/
├── infrastructure/ # インフラ層(外部システム連携)
│ ├── providers/ # プロバイダー(アダプター)
│ └── repositories/
└── api/ # API層(Lambda ハンドラー)
└── handlers/
```
**検証ルール**:
#### 依存関係の方向
```
API層 → アプリケーション層 → ドメイン層 ← インフラ層
```
**許可される依存**:
**禁止される依存**:
**検証方法**:
```bash
grep -r "from src.infrastructure" main/backend/src/domain/
grep -r "from src.api" main/backend/src/domain/
```
#### エンティティのルール
**必須条件**:
1. 識別子(ID)を持つ
2. ライフサイクルがある(生成→更新→削除)
3. 同一性は識別子で判断(`__eq__` の実装)
**検証パターン**:
```python
@dataclass
class Cart:
"""カートエンティティ."""
cart_id: CartId # ✅ 識別子を持つ
items: list[CartItem] # ✅ 可変な状態
def add_item(self, item: CartItem) -> None:
"""アイテムを追加.""" # ✅ 振る舞いを持つ
self.items.append(item)
```
**違反パターン**:
```python
@dataclass(frozen=True)
class Cart:
cart_id: CartId
items: list[CartItem]
```
#### 値オブジェクトのルール
**必須条件**:
1. イミュータブル(`frozen=True`)
2. 識別子を持たない
3. 等価性は値で判断
**検証パターン**:
```python
@dataclass(frozen=True)
class BetSelection:
"""買い目(値オブジェクト)."""
bet_type: BetType
horse_numbers: HorseNumbers
amount: Money
# ✅ 振る舞いは新しいインスタンスを返す
def change_amount(self, new_amount: Money) -> "BetSelection":
return BetSelection(
bet_type=self.bet_type,
horse_numbers=self.horse_numbers,
amount=new_amount,
)
```
**違反パターン**:
```python
@dataclass
class BetSelection:
bet_type: BetType
horse_numbers: HorseNumbers
amount: Money
@dataclass(frozen=True)
class BetSelection:
selection_id: str # ❌ 識別子は不要
bet_type: BetType
```
**集約のルール**:
1. 集約ルートを通じてのみ内部エンティティにアクセス
2. トランザクション境界 = 集約境界
3. 集約間の参照は識別子のみ
**検証例**:
#### 正しい集約設計
```python
@dataclass
class ConsultationSession:
"""相談セッション(集約ルート)."""
session_id: SessionId # 識別子
user_id: UserId
messages: list[Message] # 内部エンティティ
def add_message(self, message: Message) -> None:
"""メッセージを追加.""" # ✅ 集約ルート経由
self.messages.append(message)
```
**違反パターン**:
```python
@dataclass
class Message:
message_id: MessageId
consultation_session: ConsultationSession # ❌ 集約全体を参照
```
**正しいパターン**:
```python
@dataclass
class Message:
message_id: MessageId
session_id: SessionId # ✅ 識別子のみ
```
**ドメインサービスの条件**:
1. ステートレス(状態を持たない)
2. 複数のエンティティにまたがるロジック
3. エンティティや値オブジェクトに置けないロジック
**検証パターン**:
```python
class FeedbackGenerator:
"""フィードバック生成ドメインサービス."""
def __init__(self, ai_client: AiClient) -> None:
self._ai_client = ai_client # ✅ 依存性注入
def generate_feedback(
self,
bet_selection: BetSelection,
race_data: RaceData,
runner_data: list[RunnerData],
) -> DataFeedback:
"""買い目のフィードバックを生成."""
# ✅ 複数のドメインオブジェクトを使用するロジック
...
```
**違反パターン**:
```python
class FeedbackGenerator:
def __init__(self) -> None:
self._cache: dict = {} # ❌ 状態を持つ
class CartService:
def add_item(self, cart: Cart, item: CartItem) -> None:
cart.items.append(item) # ❌ これはCartエンティティのメソッドにすべき
```
**ポート(インターフェース)の検証**:
```python
class RaceDataProvider(ABC):
"""レースデータ取得インターフェース."""
@abstractmethod
def get_race(self, race_id: RaceId) -> RaceData | None:
pass # ✅ インターフェースのみ
class JraVanRaceDataProvider(RaceDataProvider):
"""JRA-VAN からデータ取得."""
def get_race(self, race_id: RaceId) -> RaceData | None:
# 外部API呼び出し
...
```
**違反パターン**:
```python
class RaceDataProvider(ABC):
def get_race(self, race_id: RaceId) -> RaceData | None:
# ❌ デフォルト実装を含む
return None
```
```
✅ DDD設計原則チェック完了
検証項目:
🎉 DDD原則に従った実装がなされています。
```
```
🔴 DDD設計原則違反を検出
検証項目:
---
🔴 違反 #1: レイヤー依存関係違反
ファイル: src/domain/entities/cart.py:15
違反内容: ドメイン層がインフラ層をインポート
コード:
```python
from src.infrastructure.repositories import CartRepository # ❌
```
理由: ドメイン層はインフラ層に依存してはいけない
修正案:
1. CartRepository をポート(インターフェース)として domain/ports/ に定義
2. domain層では CartRepository ポートを使用
3. infrastructure層で CartRepository を実装
---
⚠️ 警告 #1: 集約の境界が曖昧
ファイル: src/domain/entities/message.py:10
警告内容: 集約を越えた直接参照
コード:
```python
consultation_session: ConsultationSession # ⚠️
```
理由: 集約間の参照は識別子のみが推奨
推奨修正:
```python
session_id: SessionId # ✅ 識別子のみで参照
```
---
次のアクション:
```
Leave a review
No reviews yet. Be the first to review this skill!