DIコンテナを使おうとしてサービスロケータにならないようにするにはどうしたらよいのでしょうか、具体例がわかりません

Question

以下の記事を読んでいて、書いてあることまでは理解できたつもりなのですが
DI・DIコンテナ、ちゃんと理解出来てる・・？ - Qiita

最後、「こういう使い方をするとサービスロケータになっちゃうよ」ということまではわかったつもりなのですが
（お恥ずかしながらサービスロケータはこの記事を拝見して初めて知りましたが）

では具体的にどう書くと、ベターなのでしょうか。
例えばDIコンテナから取得してメンバに持つのをやめて、各メソッドで都度取得する、ような感じでしょうか。

例などご教示いただけるとありがたいです。

Answer 1

例えばDIコンテナから取得してメンバに持つのをやめて、各メソッドで都度取得する、ような感じでしょうか。

概ねそんな感じです。

参考リンクから抜粋

コンテナをファクトリでないオブジェクトの 外側 で使う場合、あなたはコンテナをDIのために使っている。
コンテナをファクトリでないオブジェクトの 内側 で使う場合、あなたはコンテナをサービスロケータとして使っている。

DI

class Client
{
    private $service;

    public function __construct(ServiceInterface $service)
    {
        $this->service = $service;
    }

    public function doSomething()
    {
        $this->service->doSomething();
    }

    ...
}

サービスロケータ

use Pimple\Container;

class Client
{
    private $container;

    public function __construct(Container $container)
    {
        $this->container = $container;
    }

    public function doSomething()
    {
        $this->container['service_interface']->doSomething();
    }

    ...
}

Comments

[d415uk35470]

ありがとうございます。

ただまだ疑問があります。
書いていただいた実装で、Clientクラスが、コンテナに依存しない、コンテナは外側で使われていることはわかりました。

ただ、もともと挙げたqiitaの記事の途中で、Clientクラスのコンストラクタの引数がどんどん増えてしまう問題の対処としてDIコンテナの話題がでてきたので、その問題をどう解決したらよいかわかっていません。

いま書いていただいている実装だと、Clientのコンストラクタの引数が受け取る数はどんどん増えていきますよね？

[d415uk35470]

あ、、、わかりました！Clientクラスの実装は一緒（引数が増えていく）で、それを生成するコードもDIコンテナにあるんですね。

ただこれだとDIコンテナがどんどん肥大化していくので、結局Builderパターンとかにしてうまく分割したほうが・・・という気もなんとなくしましたが、そこから先はまだこれから勉強します。

まずは、ありがとうございました。