「List<JavaBeans>」型を返却するメソッドを作りたい

Question

前提

Junitにて色々なBeanのListデータが必要になりました。
しかし、Beanの型は異なるけど、「Beanを生成して特定のフィールド値を設定する。それをListに詰める。」は同じで、似たような処理を書くのは面倒くさいです。
なので、「List<JavaBeans>」型を返却するメソッドを作成することにしました。

実現したいこと

対象Bean名で分岐させ各々で処理する方法を思いつきましたが、対象Beanが増えるとその分を追記しなければなりません。そこで、for文の箇所を共通化する方法はありますか？

該当のソースコード

実施環境に今いないので、思い出しながら下記ソースを書きました。
なので、正確ではないことはご了承ください。

// List作成メソッド　(arrayがリストに詰めるためのデータ）
public List<T> createList(Class<T> clazz, String[] array){
    // 戻り値
    List<T> list = new ArrayList<>();

    // 名前によって分岐
    switch (clazz.getSimpleName()) {
    case "ABean":
        // ABeanの場合
        for (String s: array) {
            ABean bean = new ABean();
            bean.setName(s);
            list.add(clazz.cast(bean));
        }
        break;

    case "BBean":
        // BBeanの場合
        for (String s: array) {
            BBean bean = new BBean();
            bean.setAddress(s);
            list.add(clazz.cast(bean));
        }
        break;

    case default:
        break;
    }

    retrun list;
}

public class ABean () {
    private name;
}

public class BBean () {
    private address;
}

Comments

[jimbe]

どのようなテストをしようとされているのか分かりませんが…。
JUnit のコードをヘタに複雑にすると、テスト対象の問題なのかテストコードの問題なのかが分かり難くなりそうです。
何でしたら(実行環境にいられましたら)、その「面倒くさい」と思われる程肥大化(?)してしまったコードをまず作ってみて、それをご質問に出してみては如何でしょうか。(100行程度もあれば十分かと。)

Answer 1

リフレクションで setter を呼んじゃう、とかはどうですか。

	public static void main(String[] args) throws Exception {

		String[] array = {"aaaa", "bbbb", "cccc"};

		List<ABean> aBeans = createList(ABean.class, "setName",    array);
		List<BBean> bBeans = createList(BBean.class, "setAddress", array);
	}

	public static <T> List<T> createList(Class<T> clazz, String setter, String[] array) throws Exception {

		List<T> list = new ArrayList<>();

		for (String s : array) {

			T bean = clazz.getDeclaredConstructor().newInstance();
			clazz.getMethod(setter, String.class).invoke(bean, s);
			list.add(bean);
		}

		return list;
	}

Answer 2

こんな風にも出来ますが…結局のところ for 文がその他のコードになるだけで、 "同じようなコードが並ぶ" ことには変わらないようにも思います。

import java.util.ArrayList;
import java.util.List;

public class Main {
  public static void main(String[] args) {

    List<ABean> alist = getList(new String[]{"あ","い","う"}, s -> {
      ABean bean = new ABean();
      bean.setName(s);
      return bean;
    });

    List<BBean> blist = getList(new String[]{"か","き","く"}, s -> {
      BBean bean = new BBean();
      bean.setAddress(s);
      return bean;
    });

    System.out.println("alist="+alist);
    System.out.println("blist="+blist);
  }

  interface Factory<T> {
    T create(String s);
  }
  static <T> List<T> getList(String[] array, Factory<T> factory) {
    List<T> list = new ArrayList<T>(array.length);
    for(String s : array) list.add(factory.create(s));
    return list;
  }
}

class ABean {
  private String name;
  void setName(String name) { this.name = name; }
  @Override public String toString() { return name; }
}
class BBean {
  private String address;
  void setAddress(String address) { this.address = address; }
  @Override public String toString() { return address; }
}

実行結果

alist=[あ, い, う]
blist=[か, き, く]

Answer 3

雑ですがこんなのでどうでしょうか。
ラムダから型を取るのは大変らしいので、よくある可変長引数のtipsで^^;

import java.util.ArrayList;
import java.util.List;
import java.util.function.BiConsumer;

public class Main {
    public static void main(String[] args) {
        var foo = createList(ABean::setName, new String[]{ "foo", "bar", "baz", });
        var hoge = createList(BBean::setAddress, new String[]{ "hoge", "fuga", "piyo", });
    }

    @SuppressWarnings("unchecked")
    private static <T> List<T> createList(BiConsumer<T, String> setter, String[] array, T... dummy) {
        var list = new ArrayList<T>();
        var clazz = (Class<T>) dummy.getClass().getComponentType();

        try {
            for (var s : array) {
                var bean = clazz.getDeclaredConstructor().newInstance();
                setter.accept(bean, s);
                list.add(bean);
            }
        } catch (ReflectiveOperationException e) {
            throw new RuntimeException(e);
        }

        return list;
    }
}

class ABean {
    private String name;

    public String getName() { return name; }

    public void setName(String name) { this.name = name; }
}

class BBean {
    private String address;

    public String getAddress() { return address; }

    public void setAddress(String address) { this.address = address; }
}

Comments

[TN8001]

さすがに納得いかんなぁ。実用するなら一番いい実装だと思ったんだけどｗ

使う側の文字数も少なくて、文字列で指定するよりメソッド参照のほうが堅牢なのに...
varがあかんかったんかなぁ？ｗ（型を書けば８でも通るはず）

Answer 4

各Beanによって処理が違いますし、fatになるだけです。
無理に共通化する方が冗長で分かりづらくなると思います。

元々呼び出し元も全然違うのですから、そのBeanだけを扱うクラス（モデルクラスになるかな？）
を作って呼び出し元にあわせたほうが良いです。なるべく疎結合に。