DI コンテナがコードに出現するのは dependency lookup が行われている兆候

dakatsuka @d_akatsuka

DIコンテナをモック化するとかなりテストがしやすい気がする

Takuto Wada @t_wada

@d_akatsuka テストに DI コンテナが登場するのは、 dependency injection ではなく dependency lookup になってしまっている兆候ではないでしょうか (もっと粒度の大きいテストの話だったらすみません)

dakatsuka @d_akatsuka

@t_wada 複数のリポジトリからデータを取得してごにょごにょするサービスクラスを作っているのですが、リポジトリの取得がコンテナ経由になってしまうんですよね…。Smyfony2は良いFWだと思うのですが、如何せん情報が少なくて試行錯誤してます

dakatsuka @d_akatsuka

奥が深い

Takuto Wada @t_wada

@d_akatsuka サービスクラスにコンストラクタやセッターを定義して、 DI 定義で constructor injection や setter injection するのはいかがでしょう http://t.co/hIf0boFS1w

dakatsuka @d_akatsuka

@t_wada 今はSetter injectionでコンテナを突っ込む形にしているのですが、コンテナではなく必要なものだけ（例えばdoctrine）をinjectするべきという事でしょうか

dakatsuka @d_akatsuka

ContainerAwareを発見して先走った感

Takuto Wada @t_wada

@d_akatsuka そのとおりです。DI の大事なところは lookup の知識をプロダクトコードから排して全体の複雑性を下げることです。必要なものを探すのではなく、与えてもらう構造を作ります。今回で言えば、リポジトリ二つのセッターがあればそれで済む、というのが良い状態ですね

dakatsuka @d_akatsuka

@t_wada 勉強になりました！ありがとうございます

Hidenori Goto | COO and CTO at KabuK Style @hidenorigoto

@d_akatsuka @t_wada Symfony2/Doctrineの場合ちょっと特殊でエンティティリポジトリはDoctrine EntityManager経由で取得しないといけないんですよね。

Hidenori Goto | COO and CTO at KabuK Style @hidenorigoto

@d_akatsuka @t_wada (つづき) なので、サービスクラスにEntityManagerをインジェクトし、サービスクラス内ではEntityManagerからリポジトリオブジェクトを取得する形で折り合いをつけています。

Takuto Wada @t_wada

@hidenorigoto @d_akatsuka なんと、そうでしたか…！ では EM を inject するしかないんですかね。なかなか悩ましいですね。

dakatsuka @d_akatsuka

@t_wada @hidenorigoto お、Doctrineは特殊なんですね。。。大変参考になりました。setEntityManage()を作って試してみますー

dakatsuka @d_akatsuka

DIへの理解が深まった

Atsuhiro Kubo @iteman

@t_wada @hidenorigoto @d_akatsuka DI 定義の factory_service を使えばインジェクションできますよ。

Atsuhiro Kubo @iteman

@t_wada @hidenorigoto @d_akatsuka ただ、サービスクラス等で結局 $em->flush() 等を使うので EntityManager だけでもいいと思いますが、より明示的にしたければそれぞれをインジェクションするのが良いかと思います。

Takuto Wada @t_wada

@iteman @hidenorigoto @d_akatsuka 補足ありがとうございます。私はできるだけ lookup を排したいので、明示的にそれぞれ injection するのが好みだなと思いました。

Atsuhiro Kubo @iteman

Doctrine ベースのリポジトリを DI サービスとして定義する例 https://t.co/foU59zqExL #symfony #doctrine

Atsuhiro Kubo @iteman

Doctine リポジトリのインジェクションは少々テクニカルなので状況に合わせて選択すると良いと思います。

Atsuhiro Kubo @iteman

@t_wada @hidenorigoto @d_akatsuka そうですね、私も明示的にインジェクションする方が良いと考えています。

Kakutani Shintaro @kakutani

@t_wada ここ数日またブクマが増えてたのはこれか…

Takuto Wada @t_wada

@kakutani 突発的に DI おじさん業を執り行った結果、本家へ影響があったとは…!

🍻 @kompiro

@t_wada Dependency lookupをするとなんでいかんのか、っていう点もこの話の流れに必要な気がしました。JavaでいうPOJOではなくなるからイカンっていうこと(移植性)や、テストのセットアップフェーズにコンテナも作らないかん(スローテストへの第一歩)とか。

Takuto Wada @t_wada

@kompiro そのとおりですね。大事な視点なので togetter に加えますね!

🍻 @kompiro

@t_wada ありがとうございます。先人が踏んだ地雷を避けられるとみんなハッピーになれるから、こういうノウハウは共有したいですねー。って検索したらこんなスライドが…。http://t.co/COYoYcCHm0 いろいろありがとうございます。