【ミライトデザイン社内勉強会#16】「IDDD本から理解するドメイン駆動設計」輪読会~第7章「ドメインサービス」~

### 前回の記事
[【ミライトデザイン社内勉強会#15】「IDDD本から理解するドメイン駆動設計」輪読会~第6章「値オブジェクト」~ - Qiita](https://qiita.com/tkek321/items/e30cedab379f431ceb79)

## [実践DDD本 第7章「ドメインサービス」～複数の物を扱うビジネスルール～ \(1/4\)：CodeZine（コードジン）](https://codezine.jp/article/detail/10318)

### オブジェクトが状態を持つ、持たないというのは下記のようなイメージであってる？
[p1](https://codezine.jp/article/detail/10318)

- 状態をもつ
    - 引数が同じでも、addメソッドを呼び出すたびに値が変わる
    
```php
class 計算
{
    private $num = 0;
    
    public function add(int $n): int 
    {
        return $this->num += $n;
    }
}
```

- 状態を持たない
    - 引数が同じ場合、addメソッドを何回呼んでも値は変わらない


```php
class 計算
{   
    public function add(int $n1, int $n2): int 
    {
        return $n1 + $n2;
    }
}
```
---

- イメージ的にはこれで問題無いと思う


### `サービスには状態を持たせないこと。`とあるがドメインサービスが状態を持ってはいけない理由が知りたい
[p1](https://codezine.jp/article/detail/10318)

- ドメインサービスはロジックをまとめているからじゃない？
- Entitiyとか値オブジェクトでやっていた振る舞いだけをサービスとして分離したクラスなので、状態を持たせない
    - 状態をもつとエンティティや値オブジェクトと役割が被ってしまう
- ドメインサービスはDIとかでユースケースに注入することが多くあるので、他の箇所で使用している可能性がある。それに状態として持たせておくと状態を変更した時の影響がどこに及ぶのかがわからなくなるのではないか。
    - ドメインサービスに限らず、 DIで渡してくるものは状態を持たせない方がいい

### Entityが内部にプロパティとして他のEntityを持っていることはあるの？

- https://codezine.jp/article/detail/10776
> <img width="500" alt="image.png (55.2 kB)" src="https://img.esa.io/uploads/production/attachments/14942/2021/02/04/76867/7f247bfc-85e6-4ea0-971d-43923c55d1b1.png">

- 上記の図を見ると、EntityがEntityを持っていることはありそう。
    - 集約ルートのEntityは他のEntitiyは持てるけど、別集約は持てなさそう。
    - Entity = 集約ルート というわけではなさそう
- Entityには他の別集約の識別子だけも持っていて、復元をする。みたいな実装もあると思う
-  [戦術的DDD基本原則まとめ - Qiita](https://qiita.com/haazime/items/6119097071149a362f7f#aggregates%E9%9B%86%E7%B4%84)
   > 集約のメンバはエンティティと値オブジェクトのみ

### 「リファクタリングによるドメインサービスの導入」で「集約の内部からリポジトリを呼び出すことはあまり望ましくない」とが記されているが、その理由を知りたい。
[p3](https://codezine.jp/article/detail/10318?p=3)

- [実践ドメイン駆動設計 | ヴォーン・ヴァーノン, 高木 正弘 | コンピュータ・IT | Kindleストア | Amazon](https://www.amazon.co.jp/%E5%AE%9F%E8%B7%B5%E3%83%89%E3%83%A1%E3%82%A4%E3%83%B3%E9%A7%86%E5%8B%95%E8%A8%AD%E8%A8%88-%E3%83%B4%E3%82%A1%E3%83%BC%E3%83%B3%E3%83%BB%E3%83%B4%E3%82%A1%E3%83%BC%E3%83%8E%E3%83%B3-ebook/dp/B00UX9VJGW)

> 経験上、集約の内部からリポジトリ(12)を使うことは、できる限り避けるべきだ。 -- ヴァーン・ヴァーノン. 実践ドメイン駆動設計 (Japanese Edition) (Kindle の位置No.6493-6494). Kindle 版.

> ドメイン内のサービスは、必要に応じてリポジトリを使える。しかし、集約のインスタンスからリポジトリにアクセスすることは、お勧めできない。 

- 経験上🤔

- 集約の中であえてリポジトリを呼ぶメリットがない
    - Entitiyの中で外部から値を持ってきたりする処理自体がない
    - ドメインがインフラに依存してるからダメ
    - ドメインオブジェクトの操作は集約からしかできないようであるべき

- [ドメイン駆動設計 - DDDで集約の内部からリポジトリを使うことを避ける理由 - スタック・オーバーフロー](https://ja.stackoverflow.com/questions/14388/ddd%E3%81%A7%E9%9B%86%E7%B4%84%E3%81%AE%E5%86%85%E9%83%A8%E3%81%8B%E3%82%89%E3%83%AA%E3%83%9D%E3%82%B8%E3%83%88%E3%83%AA%E3%82%92%E4%BD%BF%E3%81%86%E3%81%93%E3%81%A8%E3%82%92%E9%81%BF%E3%81%91%E3%82%8B%E7%90%86%E7%94%B1)



### リファクタ前のバックログアイテムは値オブジェクトとなっているが、プロダクト集約の処理をドメインサービスに移したことでバックログアイテムが集約になっているがこれはコアドメインの変更がされたと思っていい？
[p3](https://codezine.jp/article/detail/10318?p=3)
> ![](https://cz-cdn.shoeisha.jp/static/images/article/10318/10318_002_s.png)
> Productクラス（集約）からBacklogItemクラス（集約）を取り出す

と書いているので、記述ミス？
 ---

- この辺りはこの記事だけではわかりにくいと思うので、原本を合わせて読んだ方がわかりやすいかも
    1. 元々はバックログアイテムは値オブジェクトとしてプロダクトが保有している
        - `ビジネス優先度を合計するメソッド`もプロダクトにあった（この時点では何も問題ない）
    2. プロダクトの集約が肥大化してきたのでリファクタを行った
        - この時バックログアイテムを集約に変更して、プロダクトから分離した
    3. リファクタ後のプロダクト、バックログアイテム、どちらとも`ビジネス優先度を合計するメソッド`の置き場所としてはイマイチだったので、サービスクラスを作成した
        - プロダクトに`ビジネス優先度を合計するメソッド`を設置するとプロダクトEntitiyからリポジトリにアクセスをして、バックログアイテムを取得する必要がある。などの問題があったのでサービスクラスを作成し、そこに`ビジネス優先度を合計するメソッド`を置いた

原本では↑みたいな流れになっている

### ドメインサービスとはユースケースから呼ばれるサービスのこと？
#### ドメインサービスとアプリケーションサービスの違いも知りたい
[p1](https://codezine.jp/article/detail/10318)

- ユースケースから呼ばれるサービスをドメインサービスと呼ぶのではなくて、ドメインサービスをユースケースから呼んでいる
- UseCase(Applicationサービス)から呼ばれるサービスというわけではなくて、Entityに持たせられなかった振る舞いを持たせている
- Applicationサービス ≒  UseCase
    - Applicationサービスの中のUseCaseっていう認識
    - 厳密には違うのかもしれないけど、やってることは同じ
        - Applicationサービスで、ドメインサービスやエンティティ、値オブジェクトを利用する

### ドメインサービスをどういう時に使うのかあまりイメージわかない、アンチパターンを知りたい

[p1](https://codezine.jp/article/detail/10318)
> ドメインサービス：エンティティや値オブジェクトの責務ではないドメインモデルのロジック（複数のドメインオブジェクトを使って計算する処理やファサード）

[p3](https://codezine.jp/article/detail/10318?p=3)
> DDDに慣れていない場合、トランザクションスクリプトで処理を書いてしまう可能性があります。
> この問題は、使う必要がない箇所でドメインサービスを利用してしまうため、エンティティや値オブジェクトが空っぽの「ドメインモデル貧血症（5章）」を引き起こす危険性があります。

> ドメインサービスの「ミニレイヤ」を作ってしまう失敗
> 検討することなく最初からドメインサービスのレイヤを作ってしまうと、このレイヤが肥大化する傾向にあります。

- 基本はドメインサービスは使わない（使う必要が無いなら使わない）
- ドメインサービスを使う基準は、集約をまたいで処理を書きたくなったら
- ドメインサービスはどのEntitiyにも属さなかった振る舞いを仕方なくドメインサービスに切り出している

## 次回
[【ミライトデザイン社内勉強会#17】「IDDD本から理解するドメイン駆動設計」輪読会~第8章「ドメインイベント」~ - Qiita](https://qiita.com/tkek321/items/6cd7d5f15718119b34a4)