データの受け渡しについて

Question

前提・実現したいこと

削除処理でのデータの受け取りについてです。
削除ボタンがsubmitされてformにposのデータが格納されているのは確認できました。RecordManagerでremoveメソッドを定義しましたがgetPos()に値が渡っておらず正常な削除処理が行われていません。
removeメソッドのselectList.get(j).getPos()に値がわたっていないので(全て0の状態)、selectList内にposを定義するか、そもそも別の物(idやname)で値を受け取る方法を考えてみましたが結果は変わらずでした。

該当のソースコード

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 = "選択された明細行を削除しました。";

～省略～

		/** 削除処理 */
//ここでformに格納されているのを確認できた。
		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));
		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));
		list.add(new Item("B00101", "無線綴ノートA4", 100, 0, 0));
		list.add(new Item("B00201", "リングノートA4", 120, 0, 0));
		list.add(new Item("B00301", "領収書", 350, 0, 0));
		list.add(new Item("C00101", "はさみ（青）", 128, 0, 0));
		list.add(new Item("C00201", "ステープラー", 338, 0, 0));
		list.add(new Item("C00301", "２穴パンチ", 128, 0, 0));
		list.add(new Item("C00401", "ゼムクリップ", 98, 0, 0));
		list.add(new Item("C00501", "消しゴム", 58, 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()){
//System.out.println(pos);
				/** 合計からpos番目の小計を引く */
				total = total - selectList.get(j).getSubtotal();

				selectList.remove(pos);


				break;

			}

//selectListのどの商品を選んでも値が0で値が渡ってきます。
.
		System.out.println(selectList.get(j).getPos());
		}
	}

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




public SalesForm() {

}

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="${status.index}" /></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>
//selectListにposを定義したのがこれです。
						<th class="pos" style="display:none">${status.index}</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>

Item.java

public class Item {
	private String id;
	private String name;
	private int price;
	private int quantity;
	private int subtotal;
	private int pos;

	@Override
	public int hashCode() {
		final int prime = 31;
		int result = 1;
		result = prime * result + ((id == null) ? 0 : id.hashCode());
		return result;
	}

	@Override
	public boolean equals(Object obj) {
		if (this == obj) {
			return true;
		}
		if (obj == null) {
			return false;
		}
		if (!(obj instanceof Item)) {
			return false;
		}
		Item other = (Item) obj;
		if (id == null) {
			if (other.id != null) {
				return false;
			}
		} else if (!id.equals(other.id)) {
			return false;
		}
		return true;
	}
/*それぞれのgetterとsetterを生成してあります。*/

	public Item(String id, String name, int price, int quantity, int subtotal, int pos) {
		this.id = id;
		this.name = name;
		this.price = price;
		this.quantity = quantity;
		this.subtotal = subtotal;
        this.pos = pos;
	}
}

Answer 1

selectList.get(j).getPos() で 取得できる pos の値は どこで格納されているのでしょうか。
それさえ調べれば、なぜ0になってしまうのかがわかるでしょう。

Listで指定した位置のものを削除したいのであれば、selectList.get(j) で削除できるので、わざわざ位置を示す変数を Item に持つ必要はないのでは。

Comments

[naotaro0704]

回答ありがとうございます。

値自体が取得されてないから、どこにも格納されず0になってしまう
と考えていたのですが、間違っていますか。

[A-pZ]

そもそも、selectList.get(j) で取得されている Item がもつ、posの値は"どこで"定義していますか？
Itemクラスの実装がこのソースコードに書かれていないので判別はできませんが、selectListの中にあるItemがもつposの値が0のままだからなのではないでしょうか。

[naotaro0704]

Item.javaを追加しました。
自分では定義してあると思っているのですが、これは定義されているといえないのでしょうか。

[A-pZ]

removeメソッドで削除しているのは selectListの内容なので、一度 selectListの中身をデバッグモードなどで確認されてはいかがでしょうか。HTMLのth:eachで繰り返している内容を見ても
<th class="pos" style="display:none">${status.index}</th> と、実際の値を取得するのではなく、繰り返し処理用の変数から値を表示しているので、posには値が入っているかを確認できません。
ここからは推察ですが、selectListに格納するItemでposの値を定義していないからではないでしょうか。