更新によるvalue値の振り直し

Question

前提・実現したいこと

選択した商品を削除するプログラムを作成しています。
add.jspのラジオボタンのvalue値を${selectList.pos}にしています。このvalue値だと商品を削除していったときにIndexOutOfBoundsExceptionが発生するときがあります。削除するごとに売上明細のラジオボタンのvalue値を更新して値を振り直す方法はありますか。

IndexOutOfBoundsExceptionが発生しているところは、SalesSystemController.javaのRecordManager.remove(form.getPos());とRecordManager.javaのselectList.remove(pos);の部分とエラーメッセージには出ています。

該当のソースコード

SalesSystemController.java

	/** 削除処理 */

		System.out.println(form.getPos()+"from");

		/**
		 * fromのposを取得しmodelに格納する。
		 */
		model.addAttribute("pos",form.getPos());
		/**
		 * RecordManaagerのremoveメソッドを実行させる。
		 */
		RecordManager.remove(form.getPos());

		/** 商品の残りの数による条件分岐 */

		/** 売上明細の残りが1件以上の処理 */
		if (RecordManager.getSelectList().size() > 0) {
			/**
			 * DELETEをmessage4という名前でmodelに登録する
			 */
			model.addAttribute("message4", DELMSG);
			/**
			 * RecordManagerのselectListを取得し、allListという名前でmodelに登録する。(売上明細)
			 */
			model.addAttribute("selectList", RecordManager.getSelectList());
			/**
			 * RecordManagerのitemListを取得 itemListという名前でmodelに登録する。
			 */
			 model.addAttribute("ItemList", RecordManager.getItemList());
			/**
			 * RecordManagerのtotalを取得しtotalという名前でmodelに登録する。(合計)
			 */
			model.addAttribute("total", RecordManager.getTotal());
			/**
			 * 明細追加画面へ
			*/
			return ADD;
		}
		/** 売上明細の残りが0件のときの処理 */
		else {
			/**
			 * DELETEをmessage4という名前でmodelに登録する
			 */
			model.addAttribute("message4", DELMSG);
			/**
			 * 初期画面へ
			 */
			return INIT;
		}
	}

RacordManager.java

public final class RecordManager<selectList> {

	/** 商品データ */
	private static List<Item> list = new ArrayList<>();
	private static List<Item> selectList = new ArrayList<>();
	private static Item item;
	/** totalを定義 */
	private static int total = 0;
	private static long salesId;
	private static int pos;
	private static String index;



	static {
		list.add(new Item("A00101", "油性ボールペン", 60, 0, 0, 0));
		list.add(new Item("A00201", "極細ボールペン", 120, 0, 0, 0));
		list.add(new Item("A00301", "蛍光ペン６色セット", 420, 0, 0, 0));
		list.add(new Item("A00401", "シャープペンシル", 100, 0, 0, 0));
		list.add(new Item("A00501", "鉛筆H（１ダース）", 400, 0, 0, 0));
		list.add(new Item("B00101", "無線綴ノートA4", 100, 0, 0, 0));
		list.add(new Item("B00201", "リングノートA4", 120, 0, 0, 0));
		list.add(new Item("B00301", "領収書", 350, 0, 0, 0));
		list.add(new Item("C00101", "はさみ（青）", 128, 0, 0, 0));
		list.add(new Item("C00201", "ステープラー", 338, 0, 0, 0));
		list.add(new Item("C00301", "２穴パンチ", 128, 0, 0, 0));
		list.add(new Item("C00401", "ゼムクリップ", 98, 0, 0, 0));
		list.add(new Item("C00501", "消しゴム", 58, 0, 0, 0));
	}

	/** 商品名 */
	public static List<Item> selectItem(String id, int quantity,int pos) {

		for (int i = 0; i < list.size(); i++) {

			if (id.equals(list.get(i).getId())) {
				pos = selectList.size();

				/** Itemを再定義 */
				Item item = new Item(id,list.get(i).getName(),  list.get(i).getPrice(), quantity,
						quantity * list.get(i).getPrice(),pos);

				selectList.add(item);

				/** subtotalを定義 */
				int subtotal = quantity * list.get(i).getPrice();
				total = total + subtotal;

				break;
			}

		}

		return selectList;

	}



	/** 選択した売上明細を削除 */
	public static  void remove(int pos) {
		/** int型jを定義しselectListの要素内で繰り返し処理*/
		for (int j = 0; j < selectList.size();j++) {

			/** 選択したposと同じ順番の要素をselectList内から探す*/
			if (pos == selectList.get(j).getPos()){
//System.out.println(pos+"I");


				/** 合計からpos番目の小計を引く */
				total = total - selectList.get(j).getSubtotal();

System.out.println(selectList.get(j).getPos()+"R");

			/** selectListのpos番目を削除 */
				selectList.remove(pos);

				break;

			}

		}
	}

SalesForm.java

public class SalesForm implements Serializable {
	private static final long serialVersionUID = 1L;



	/** 商品id */
	private String id;
	/** 商品名 */
	private String name;
	/** 単価 */
	private int price;
	/** 個数 */
	@NotNull
	@Min(1)
	private int quantity=1;
	/** 小計 */
	private int subtotal;
	@NotNull
	private int pos;




public SalesForm() {

}
/** getterとsetterを定義*/

Item.java

public class Item {

	/** 商品id */
	private String id;
	/** 商品名 */
	private String name;
	/** 単価 */
	private int price;
	/** 個数 */
	private int quantity;
	/** 小計 */
	private int subtotal;

	private int pos;
/** hashCord()とequals()を定義*/
/** getterとsetterを定義*/

add.jsp

<body>
	<form:form modelAttribute="salesForm" action="/sales/system">
		<div class="header">
			<span class="titleName">直太郎オンラインショップ</span>
			<div class="date"><%=LocalDate.now()%></div>
		</div>
		<div class="main">
			<div>
				商品:
				<form:select path="id">

				<form:options items="${ItemList}" itemValue="id" itemLabel="name" />

				</form:select>

				<br />
			</div>
			<div>
				<span class="itemQuantity">点数:</span>
				<form:input path="quantity" size="20" />
			</div>

			<div class="add">
				<div>
					<input type="submit" name="addition" value="明細追加" />
				</div>
			</div>
		</div>

		<br>
		<div class="message">
			<font color="#ff0000"> <c:out value="${message1}" />
			</font> <font color="#1e90ff"> <c:out value="${message2}" /> <font
				color="#1e90ff"> <c:out value="${message4}" />
			</font>
			</font> <font color="#ff0000"> <c:out value="${message5}" />
			</font>
		</div>
		<br>



		<div class="">
			<span class="details">売上明細</span>

			<table class="product">

				<tr>
					<th>削除</th>
					<th>商品ID</th>
					<th>商品名</th>
					<th>単価</th>
					<th>点数</th>
					<th>小計</th>
				</tr>
				<c:forEach items="${selectList}" var="selectList" varStatus="status">
					<tr>
						<th class="itemradiobtn"><form:radiobutton path="pos"
								label="" value="${selectList.pos}" /></th>
						<th class="id">${selectList.id}</th>
						<th class="name">${selectList.name}</th>
						<th class="price">${selectList.price}</th>
						<th class="quantity">${selectList.quantity}</th>
						<th class="subtotal">${selectList.subtotal}</th>
						<th class="pos">${selectList.pos}</th>
					</tr>
				</c:forEach>
			</table>
		</div>

		<div class="total">
			合計：
			<fmt:formatNumber groupingUsed="true">
				<c:out value="${total}" />
			</fmt:formatNumber>
			円
		</div>


		<div class="footer">

			<input type="submit" name="remove" value="削除"
				onclick="remove_row(this)" /> <input type="submit" name="firm"
				value="確定" />
		</div>

	</form:form>

</body>

Comments

[m.ts10806]

どこでIndexOutOfBoundsExceptionがおきているのでしょうか
質問本文に明示してください

[tetutetu]

削除処理を行ってからページを再表示するまでのどこかのタイミングで
selectList内の各Itemのposにインデックス番号を再セットする処理を挟むのが場当たり的な対応になるかなと思いますが

こう何度も場当たり的な対応を繰り返すぐらいなら、Map等を使って再設計した方がよっぽど良いものになるのではと思うのですが…。

Answer 1

selectList.remove(pos); はListの位置を指定して削除するので、もしposにこだわるのであれば、
selectListの中身をforループで取り出し、itemのposと同じだった場合にその要素を削除すると良いでしょう。

例えば

for (Item item:selectList) {
  int itemPosition = item.getPos();
  if ( pos == itemPosition ) {
    selectList.remove(item);
    break;
  }
}

などでしょうか。

https://docs.oracle.com/javase/jp/8/docs/api/java/util/List.html#remove-java.lang.Object-