JavaでListの情報をDtoに一行でいれたい

Question

###前提・実現したいこと
Javaで実装したいのですが
List<DTOA> list = new ArrayList<DTOA>();
for (int i=0;i<10;i++) {
DTOA dtoA = new DTOA();
dtoA.getCount(i);
list.add(dtoA);
}

DTOB dtoB = new DTOB();
dtoB.setCount1(list.get(0).getCount());
dtoB.setCount2(list.get(1).getCount());
dtoB.setCount3(list.get(2).getCount());
dtoB.setCount4(list.get(3).getCount());
dtoB.setCount5(list.get(4).getCount());
dtoB.setCount6(list.get(5).getCount());
dtoB.setCount7(list.get(6).getCount());
dtoB.setCount8(list.get(7).getCount());
dtoB.setCount9(list.get(8).getCount());
dtoB.setCount10(list.get(9).getCount());

このdtoBにいれるところを、Streamなどを使用して1行でかけたりしないでしょうか？
よろしくお願いします。

Comments

[matobaa]

小さなプログラムだと逆に遅くなりそうなやりかたならありそうなのですけど、Streamなどを使用して1行で書きたい理由ってなんでしょうか? setCount999がたーーくさんあるとか?

[java40]

setCountがたくさんあるので、ソースを見たときにもっとコンパクトに綺麗にできないかと思いまして、ただ速度が落ちるようならあきらめようと思います。

[退会済みユーザー]

いくらがんばって考えてもも10行前後になるや。

※ リフレクションなのでおそくなる

setCountN の設計を根本から変えないとダメかと

[退会済みユーザー]

DXOを作ってそこにつらつら書いた方がいいかもしれない。

Answer 1

できたできた。

DTOB 側に、番号で指定されたメソッドを返すメソッドを書いておけば:

// DTOB.java
IntConsumer[] intConsumers = new IntConsumer[] {
    this::setCount1, this::setCount2, this::setCount3,
    this::setCount4, this::setCount5, this::setCount6,
    this::setCount7, this::setCount8, this::setCount9, this::setCount10 };
public IntConsumer getFunction(int index) { return intConsumers[index-1]; }

こんな感じで呼び出せます:

    IntStream.rangeClosed(1, 10).parallel().forEach(
        i -> {dtoB.getFunction(i).accept(list.get(i-1).getCount());}
    );

parallel() は別に使わなくてもいいですけど。

Stream#zip() を使えば、もっとすっきり書けるはず。

メソッド数がとても多ければ parallel の効果がでるかもしれない……

Comments

[java40]

有難う御座います。
JavaでのStream#zip()はなかったはずなので、どうしようかと思いまして。
試してみます。

[matobaa]

いちおうStream使ってみましたが、やっぱりharuka-kanataさんが言うようにsetCountN の設計を見直すか、DXOを書くのがきれいだと思います。

Stream#zip() 欲しいです! いまはないけど