JavaのArrayListの値がうまく取得できない

Question

前提・実現したいこと

Java（SpringFramework）でショッピングサイトのようなシステムを
作っています。

エラーメッセージ等はでないのですが、
JavaBeans型のListをfor文で値を取得する際に、同じもののみが
取得されてしまって困っています。

該当のソースコード

OrderItemBean orderItemBean = new OrderItemBean();
List<Item> items = itemRepository.findAllById(basketIdList);
List<OrderItemBean> oiBean = new ArrayList<OrderItemBean>();

	for (int i = 0; i < basketNum; i++) {
		Item item = items.get(i);
		orderItemBean.setId(item.getId());
		orderItemBean.setName(item.getName());
		orderItemBean.setPrice(item.getPrice());
		orderItemBean.setImage(item.getImage());
		orderItemBean.setOrderNum(basketOrderNumList.get(i));
		orderItemBean.setSubtotal(subtotalList.get(i));
		
		oiBean.add(i, orderItemBean);
		}

for (int num = 0; num < basketNum; num++) {
		System.out.println("oiBeanのname取得"+oiBean.get(num).getName());
		
		//下記は試しに記述してみたコード
		System.out.println(oiBean.get(0).getName());
		System.out.println(oiBean.get(1).getName());
		}

```

### 試したこと
「OrderItemBeanはInteger idやString name,Integer priceなどが詰まったjavaBeansです。

OrderItemBean型のArrayList,oiBeanを定義して
oiBeanの中にorderItemBeanにセッターで中身をいれたjavaBeansをリスト
として入れていきます。
その後for文でひとつづつ取得しようとしてみました。」

###### SYSOUTの結果等
sysoutの結果は例えばorderItemBean.setName(item.getName());が
オレンジ、りんご、辞書の順にセットしていくと結果が
オレンジ、オレンジ、オレンジになっています。
一つ目のfor文内でitem.getName()をsysoutしてみると出てくるものが
itemsの主キーに設定しているidの昇順にりんご(id=1)、辞書(id=2)、オレンジ(id=3)の順に出力されています。
まずはオレンジが三回出力されるのを解決するにはどうすれば良いでしょうか
宜しくお願い致します。

Answer 1

for文で各ループで毎回同じオブジェクトに対して操作をしているため、Listには毎回同じオブジェクトを追加していることになります。したがって、最後に入れた要素が全部ずらっと並ぶことになります。
ループごとにオブジェクトを生成して、異なるオブジェクトに対する操作にしないといけません。

    for (int i = 0; i < basketNum; i++) {
        Item item = items.get(i);
        OrderItemBean orderItemBean = new OrderItemBean();  // <-ここ
        orderItemBean.setId(item.getId());
        orderItemBean.setName(item.getName());
        orderItemBean.setPrice(item.getPrice());
        orderItemBean.setImage(item.getImage());
        orderItemBean.setOrderNum(basketOrderNumList.get(i));
        orderItemBean.setSubtotal(subtotalList.get(i));

        oiBean.add(i, orderItemBean);
    }

Comments

[aechan]

まったく気が付きませんでした。。
おかげさまで解決しました、ありがとうございます！

Answer 2

oiBean.add(i, orderItemBean);でoiBeanに追加しているのは、1行目でnewしたorderItemBeanを追加しています。for内で何度addしても同じオブジェクトを追加しているだけです。

以下を一つ目のfor()の中に入れてください。

OrderItemBean orderItemBean = new OrderItemBean();