# Law Of Demeter

> デメテルの法則（最小知識の原則）に基づくコードレビューと設計支援。オブジェクトの連鎖呼び出し （Train Wreck）を検出し、直接の友人とのみ会話する設計へ変換する。結合度の低減と 変更容易性の向上を促進する。コードレビュー、新規実装、リファクタリング時に オブジェクト間の結合が深い場合に使用。 対象言語: Java, Kotlin, Scala, TypeScript, Python, Ruby, Go, Rust。 トリガー：「デメテルの法則」「連鎖呼び出しを減らしたい」「Train Wreckを直して」 「結合度を下げたい」「ドット連鎖が多い」「最小知識の原則」「Law of Demeter」 といったオブジェクト間結合関連リクエストで起動。

- Skill: `j5ik2o/law-of-demeter` (Agent Skill, multi-file: 2 files)
- Install (CLI): `npx skillmds@latest add j5ik2o/law-of-demeter`
- Raw SKILL.md: https://api.skillmd.com/api/skills/j5ik2o/law-of-demeter/raw
- Safety review: pending
- Works with: Claude Code, Claude.ai, OpenAI Codex
- Category: Coding & Dev Tools
- Author: j5ik2o (https://skillmd.com/u/j5ik2o)
- Updated: 2026-09-10
- Page: https://skillmd.com/skills/j5ik2o/law-of-demeter

---


# Law of Demeter

直接の友人とだけ話せ。見知らぬ者に話しかけるな。

## 核心原則

**メソッドは「直接の友人」のメソッドだけを呼び出し、「友人の友人」には手を出さない。**

Karl Liebherr（1987年、ノースイースタン大学）が提唱。正式名称は「最小知識の原則（Principle of Least Knowledge）」。

| アプローチ | 特徴 | 問題 |
|-----------|------|------|
| 連鎖呼び出し | `a.getB().getC().doX()` | 内部構造に依存、変更に脆い |
| 委譲 | `a.doX()` | 結合度が低い、変更に強い |

## 4つのルール

メソッド `M` が呼び出してよいのは、以下の4種類のメソッドのみ：

| # | 許可される呼び出し先 | 説明 |
|---|---------------------|------|
| 1 | 自身（`this` / `self`）のメソッド | 自分のクラスに定義されたメソッド |
| 2 | `M` の引数として渡されたオブジェクトのメソッド | パラメータ経由の直接の友人 |
| 3 | `M` 内で生成したオブジェクトのメソッド | 自分が作ったオブジェクトは友人 |
| 4 | 自身のインスタンス変数（フィールド）のメソッド | 保持しているオブジェクトは友人 |

**禁止**: 上記メソッド呼び出しの**戻り値**のメソッドを呼び出すこと（＝友人の友人）

## 判断フロー

```
メソッド内で obj.method() を呼んでいる
    ↓
obj はどこから来たか？
    ├─ this/self のフィールド → ✅ 許可（ルール4）
    ├─ メソッドの引数 → ✅ 許可（ルール2）
    ├─ メソッド内で new/生成した → ✅ 許可（ルール3）
    ├─ this/self 自身 → ✅ 許可（ルール1）
    └─ 別のメソッド呼び出しの戻り値 → ❌ 違反（友人の友人）
```

## アンチパターン検出

### Train Wreck（列車事故）

連鎖的なドット呼び出しでオブジェクトの内部構造をたどるパターン：

```
❌ order.getCustomer().getAddress().getCity()
❌ user.getProfile().getSettings().getTheme().getName()
❌ app.getConfig().getDatabase().getConnection().execute(query)
❌ invoice.getLineItems().get(0).getProduct().getCategory()
```

### 検出基準

| パターン | 問題 |
|---------|------|
| ドットが2つ以上の連鎖 | 構造依存（ただし流暢APIは例外） |
| getter連鎖 + 末尾の操作 | 取得したオブジェクトの操作 = 友人の友人 |
| getter連鎖 + if文 | 遠いオブジェクトの状態で分岐 |

## 変換パターン

### 1. 委譲メソッドの導入

```java
// ❌ 違反: 友人(order)の友人(customer)の友人(address)に話しかけている
City city = order.getCustomer().getAddress().getCity();

// ✅ 修正: 各レベルに委譲メソッドを追加
City city = order.getShippingCity();

// Order
public City getShippingCity() {
    return customer.getShippingCity();
}

// Customer
public City getShippingCity() {
    return address.getCity();
}
```

### 2. 目的に応じたメソッド名

```java
// ❌ 違反: 内部構造を露出した名前
Email email = order.getCustomer().getEmail();

// ✅ 修正: 目的を表すメソッドを提供
Email email = order.getNotificationEmail();
```

### 3. パラメータとして渡す

```java
// ❌ 違反: 遠いオブジェクトを取得して使用
void processOrder(Order order) {
    PaymentGateway gateway = order.getCustomer().getPaymentGateway();
    gateway.charge(order.getTotal());
}

// ✅ 修正: 必要なオブジェクトを引数で受け取る
void processOrder(Order order, PaymentGateway gateway) {
    gateway.charge(order.getTotal());
}
```

### 4. 振る舞いをオブジェクトに移動

```java
// ❌ 違反: 外部で判定
if (order.getCustomer().getAddress().getCountry().equals("JP")) {
    applyJapaneseTax(order);
}

// ✅ 修正: 判定ロジックをオブジェクトに持たせる
Order orderUpdated = order.applyApplicableTax();

// Order
public Order applyApplicableTax() {
    return customer.applyTaxFor(this);
}
```

## 例外：違反ではないケース

### 1. 流暢API / ビルダーパターン

```java
// ✅ 許可: 流暢APIは同一オブジェクトを返す
User user = User.builder()
    .name("Alice")
    .email("alice@example.com")
    .build();
```

**理由**: 各メソッドが `this` を返すため、友人の友人ではなく同じ友人と会話し続けている。

### 2. データ構造（DTO / Value Object）

```java
// ✅ 許可: DTOは振る舞いを持たない単なるデータ構造
City city = addressDto.getCity();
int zip = addressDto.getZipCode();
```

**理由**: DTOは内部構造の隠蔽を目的としない。ただし、ドメインオブジェクトでは違反。

### 3. ストリーム / コレクション操作

```java
// ✅ 許可: ストリームAPIの連鎖
List<String> names = users.stream()
    .filter(u -> u.isActive())
    .map(u -> u.getName())
    .collect(Collectors.toList());
```

**理由**: ストリームAPIは流暢APIの一種であり、内部構造の探索ではない。

### 4. 標準ライブラリの連鎖

```python
# ✅ 許可: 標準ライブラリの連鎖
result = text.strip().lower().replace(" ", "_")
```

**理由**: 同一型（String）のメソッド連鎖であり、内部構造への依存ではない。

## 過剰適用の警告

### 「ラッパーメソッドの爆発」に注意

```java
// 過剰: すべてのフィールドに委譲メソッドを作成
class Order {
    Name getCustomerName() { return customer.getName(); }
    Email getCustomerEmail() { return customer.getEmail(); }
    Phone getCustomerPhone() { return customer.getPhone(); }
    City getCustomerCity() { return customer.getAddress().getCity(); }
    // ... 延々と続く
}
```

**対処**: 本当に必要な操作だけを委譲する。全フィールドの委譲は設計の問題を示す。

### 判断基準

| 状況 | 対応 |
|------|------|
| 委譲メソッドが3個以内 | 適切 |
| 委譲メソッドが4個以上 | 設計を見直す（責務の分割を検討） |
| 同じ委譲先への委譲が大量 | そのオブジェクトを直接使うべきか検討 |

## 関連原則

| 原則 | 関係 |
|------|------|
| Tell, Don't Ask | 補完関係。TDAは「命じよ」、LoD は「友人にだけ」 |
| カプセル化 | LoD はカプセル化を構造的に強制する手段 |
| 疎結合 | LoD の遵守は結合度を機械的に低下させる |
| 単一責任原則 | 委譲メソッドの爆発は SRP 違反のサイン |
| breach-encapsulation-naming | getter を明示的に制限する命名規約 |

## レビュー観点

コードレビュー時の確認ポイント：

1. **ドット連鎖**: `a.b().c()` 形式の連鎖が2段以上ないか
2. **getter連鎖**: 取得→取得→操作のパターンはないか
3. **遠いオブジェクトへの依存**: メソッドが知るべきでないオブジェクトを参照していないか
4. **例外の確認**: 流暢API / DTO / ストリームなら許容

## 詳細ガイドライン

言語別の実装パターン、リファクタリング手順の詳細は [references/patterns.md](references/patterns.md) を参照。

## 関連スキル（併読推奨）
このスキルを使用する際は、以下のスキルも併せて参照すること：
- `tell-dont-ask`: 振る舞い面の補完原則（状態を問い合わせず命じる）
- `first-class-collection`: コレクションへのデメテルの法則適用パターン
- `breach-encapsulation-naming`: 連鎖呼び出しが避けられない場合の命名規約

