IndexOutOfBoundsExceptionの対処法について

Question

前提・実現したいこと

javaでの削除処理についてです。
ラジオボタン選択時に削除処理を行うプログラムを作っています。明細追加で商品をいくつか追加した後削除処理を行う際にエラーが発生してしまっています。
対処法を教えていただきたいです。

発生している問題・エラーメッセージ

IndexOutOfBoundsExceptionが発生してしまっています

java.lang.IndexOutOfBoundsException: Index: 4, Size: 4
	java.util.ArrayList.rangeCheck(ArrayList.java:653)
	java.util.ArrayList.remove(ArrayList.java:492)
	jp.practice.sales.RecordManager.remove(RecordManager.java:77)
	jp.practice.sales.SalesSystemController.remove(SalesSystemController.java:210)
	

該当のソースコード

SalesSystemController.java

@Controller
@RequestMapping(value = "/system")
public class SalesSystemController {

	private static final Logger logger = LoggerFactory.getLogger(SalesSystemController.class);

	/** Stringの定数として次の3つを定義する */
	private static final String INIT = "init";
	private static final String ADD = "add";
	private static final String FIX = "fix";
	/** メッセージと定義 */
	private static final String ERRMSG = "1以上の数字を入力してください。";
	private static final String ADDMSG = "明細に追加しました。";
	private static final String FIXMSG = "以下のように売上登録しました。";
	private static final String DELMSG = "選択された明細行を削除しました。";
//	private static final String REMMSG = "明細行を選択してください。";

	/** 初期画面を呼び出す処理 */
	/** アノテーション情報として/startを受け入れる */
	@RequestMapping(value = "/start")
	/** SalesFormの引数をformに、Modelの引数をmodelに指定する*/
	public String init(SalesForm form, Model model) {


	

	/** 明細追加画面での削除押下時の処理 */
	@RequestMapping(params = "remove")
	public String remove(SalesForm form, BindingResult result, Model model) {

		/**
		 * RecordManagerのitemListを取得 itemListという名前でmodelに登録する。
		 */
		 model.addAttribute("ItemList", RecordManager.getItemList());
		/**
		 * formのquantityを取得し、addDataという名前でmodelに登録する。
		 */
		 model.addAttribute("addData", form.getQuantity());


		/** 削除処理 */

		System.out.println(form.getId());
		/**
		 * 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("allList", 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;
		}
	}

RecordManager.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;

	static {
		list.add(new Item("A00101", "油性ボールペン", 60, 0, 0));
		list.add(new Item("A00201", "極細ボールペン", 120, 0, 0));
		list.add(new Item("A00301", "蛍光ペン６色セット", 420, 0, 0));
		list.add(new Item("A00401", "シャープペンシル", 100, 0, 0));
		list.add(new Item("A00501", "鉛筆H（１ダース）", 400, 0, 0));
	
	}

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

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

			if (id.equals(list.get(i).getId())) {

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

				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()){
				/** 合計からpos番目の小計を引く */
				total = total - selectList.get(pos).getSubtotal();

				break;
			}
			/** selectListのpos番目を削除 */
			selectList.remove(pos);
			System.out.println(getPos());
		}
	}

SalesForm.java

ublic 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;
    private int pos;

    public SalesForm() {

    }

/** 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="${allList}" var="allList" varStatus="status">
                    <tr>
                        <th class="itemradiobtn"><form:radiobutton path="pos"
                                label="" value="${status.index}"/></th>
                        <th class="id">${allList.id}</th>
                        <th class="name">${allList.name}</th>
                        <th class="price">${allList.price}</th>
                        <th class="quantity">${allList.quantity}</th>
                        <th class="subtotal">${allList.subtotal}</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>
</html>

試したこと

IndexOutOfBoundsExceptionについて調べると、indexの上限が256でindex256、size256のときは、上限を引き上げる方法などが出てきましたがやり方が256までいっていないので違うやり方があるのだろうと思いました。
Index: 4, Size: 4ということなので、inedxは0,1,2,3で4が指定されたらエラーが起きるのはわかりました。

Answer 1

配列やリストの範囲外を指定するからその例外が出るんですから、
前もってそいつのサイズを見て、範囲外かどうかを判定すればどうでしょう

Comments

[naotaro0704]

回答ありがとうございます。
実際に判断する豊富としてはどうしたらよいでしょうか。

[y_waiwai]

その例外が出る行で、配列やリストのsizeを見て、インデックスと比較すればいいだけでは。

まあ、そもそも範囲外のアクセスが出る時点でそのコードがおかしいってことなんでそこからどうにかしないといけないですが。