コンストラクタに引数が必要なクラスのDIについて

Question

コンストラクタに引数（処理中に値が決定する）が必要なクラスをDIする場合
どのように実装したらよいのでしょうか。

１．基本的にシングルトンなので動的な引数でインスタンス化は出来ないのでしょうか。

２．そのような場合はコンテナ管理させずにnewしてあげるのが一般的なのでしょうか。

@Controller
public class HogeController {
    @Autowired
    private HogeService service;

    void test(){
        int i = 1;
        String s = "test";

// 本当は以下のようなことがしたいのですが、どのようにすれば良いのでしょうか。
        HogeService service = new HogeService(i, s);

        service.print();
    }
}

@Service
public class HogeService {
    final int a;
    final String b;

    HogeService (int a, Sting b){
        this.a = a;
        this.b = b;
    }

    public void print(){
        System.out.println(i);
        System.out.println(s);
    }
}

追加（解決策？）

• new()した方が良いような気もしますがコンテナに登録されたオブジェクトにコンストラクタに引数が渡せました。

@Controller
public class HogeController {
    void test(){
        AnnotationConfigApplicationContext context = new AnnotationConfigApplicationContext(Test.class);
        TestDI hogeA = (TestDI) context.getBean("getHogeInstance", 1, "@@@");
        TestDI hogeB = (TestDI) context.getBean("getHogeInstance", 2, "◆◆");

        hogeA.print();
        hogeB.print();
    }
}

@Service
public class Test {
    @Bean
    @Scope("prototype")
    HogeService getHogeInstance(int i, String s) {
        return new HogeService (i, s);
    }
}

public class HogeService {
    final int a;
    final String b;

    HogeService (int a, Sting b){
        this.a = a;
        this.b = b;
    }

    public void print(){
        System.out.println(i);
        System.out.println(s);
    }
}

Answer 1

サンプル

@Configuration
public class ApplicationConfiguration {

  @Bean
  @Scope("prototype") //As we want to create several beans with different args, right?
  String hello(String name) {
    return "Hello, " + name;
  }
}

//and later in your application

AnnotationConfigApplicationContext context = new AnnotationConfigApplicationContext(ApplicationConfiguration.class);
String helloCat = (String) context.getBean("hello", "Cat");
String helloDog = (String) context.getBean("hello", "Dog");

APIリンク

上記をマネすれば、引数が渡せるのでは？

Comments

[msss]

これは@Autowiredして値を設定するのと何が違うのでしょうか。

[msss]

回答ありがとうございます。
上記の方法を用いてコンテナに登録されたオブジェクトにコンストラクタからnewすることが出来ました。
これが良い方法かどうかはまた別途質問しようと思います。
ありがとうございました。

Answer 2

HogeControllerのインスタンスが生成された時点でコンテナから対象のインスタンスが取得されるため、メソッド中で初期値を設定するのは不可能です。

setterなどで設定してあげるか（immutableでやりたいのであればこの方法は使えません。）
コンテナ管理させないか。

ですかね。とりあえずなんでもかんでもコンテナ管理はやめておいたほうがいいです。
コンテナ管理をさせたい理由と実現したいことのイメージがあれば別にアドバイスが出来るかもしれません。

Comments

[msss]

回答ありがとうございます。
※コードを少し修正しました（コンストラクタ引数をfinalにしました）

特にコンテナ管理させたいわけではないです。
上記の場合はnew()しておくのが一般的という回答でよろしいでしょうか。

[7tsuno]

そうですね。一般的なオブジェクトでコンテナ管理させない場合はコンテナ管理させないほうがいいです。
つまり、new()で取得するのが一般的（一般的かどうかといったら自分はstatic factoryのほうが好きですが）です。
コンテナ管理する対象はロジックだったり、値をリクエスト間で引き継ぎたいオブジェクトだったりです。
ちなみに一般的には@Serviceを付与するのはでサービス層のクラス（ビジネスロジック等）です。

[msss]

◇ロジックの処理も、その業務を行うのに必須の前提条件が必要な場合もあると思います。
そのような場合は全てのメソッドに前提条件の値を渡すことになるのでしょうか。

◇ちなみに、7tsuno様はコンテナ管理させる場合とさせない場合をどのように切り分けられているのでしょうか。

[7tsuno]

◇ロジックの処理も、その業務を行うのに必須の前提条件が必要な場合もあると思います。
ロジックにフィールドを持たせたいということでしょうか。基本的にはコンテナ管理されたロジックにフィールドを持たせるのは危険です。（スコープにもよりますが他のスレッドやリクエストに値が変更される可能性があるため。）
全てのメソッドに前提条件の値を渡すことになると思います。

◇コンテナ管理させる場合とさせない場合
設計にもよりますが、Controller, Helper, Service,Form, DTOを使うような場合、
https://terasolunaorg.github.io/guideline/public_review/Overview/ApplicationLayering.html
Controller, Helper, Serviceはフィールドを持たないがコンテナ管理。
Form, DTOはnewで取得
という風に切り分けますね。

[7tsuno]

追加で、1セッションで共通で扱いたいオブジェクトや1アプリケーションで共通で扱いたいオブジェクトがあればコンテナ管理させます。
件のようなオブジェクトはすぐに生存期間を終わらせていいものなのでコンテナ管理はさせません。

[msss]

とても分かりやすい回答ですっきりしました。
ありがとうございます！