リストからラジオボタンで選んだ内容を削除する処理を実装したい

以下のadd.jspで[明細追加]ボタンを押すと、プルダウンで選択した商品データがリストとして追加されていきます。

そこで追加されたリストの一番左にラジオボタンを設置して、ラジオボタンで選択した商品をリストから削除する処理を実装したいです。（ラジオボタンの設置まではできてます）

以下のソースでは本質問では関係ない部分は文字数を減らすため削除しています。

リストからの削除は以下のメソッドで実装したいです。
RecordManager.javaのremoveToAllListメソッド

java
1    /**
2     * リストに商品情報を 1 件削除する
3     */
4    public static void removeToAllList(Item newItem) {
5    	Item item = new Item(newItem.getId(), newItem.getName(), newItem.getPrice(),newItem.getQuantity(),newItem.getSubtotal());
6        allList.remove(item);
7    }

よろしくおねがいします。

add.jsp

jsp
1<%@ page session="false" language="java"
2	contentType="text/html; charset=UTF-8" pageEncoding="UTF-8"%>
3<%@ page import="java.time.LocalDate"%>
4<%@ taglib uri="http://java.sun.com/jsp/jstl/core" prefix="c"%>
5<%@ taglib uri="http://www.springframework.org/tags/form" prefix="form"%>
6<!DOCTYPE html>
7<html>
8<head>
9<link rel="stylesheet" type="text/css"
10	href="/sales/resources/css/common.css" />
11<title>売上システム(初期画面)</title>
12</head>
13<body>
14	<form:form modelAttribute="salesForm" action="/sales/system" var="list">
15		<div class="header">
16			<span class="titleName">オンラインショップ</span>
17			<div class="date"><%=LocalDate.now()%></div>
18		</div>
19		<div class="main"></div>
20		<div>明細追加画面</div>
21		<div>
22			<form:select path="goodsName" items="${nameList}" />
23			<div>
24				<input type="text" name="quantity" value="0" size="20" />
25			</div>
26			<div>
27				<input type="submit" name="add" value="明細追加" />
28			</div>
29			<table>
30				<tr>
31					<th>削除</th>
32					<th>商品ID</th>
33					<th>商品名</th>
34					<th>単価</th>
35					<th>点数</th>
36					<th>小計</th>
37				</tr>
38				<c:forEach items="${allList}" var="item" varStatus="status">
39					<tr class="even">
40						<td><input type="radio" name="remove" /></td>
41						<td>${item.id}</td>
42						<td>${item.name}</td>
43						<td>${item.price}</td>
44						<td>${item.quantity}</td>
45						<td>${item.subtotal}</td>
46					</tr>
47				</c:forEach>
48			</table>
49			<input type="submit" name="toInit" value="確定" />
50		</div>
51		<div>
52			<input type="submit" name="remove" value="削除" />
53		</div>
54	</form:form>
55</body>
56</html>

SalesSystemController.java

java
1package jp.practice.sales;
2
3import org.springframework.stereotype.Controller;
4import org.springframework.ui.Model;
5import org.springframework.web.bind.annotation.RequestMapping;
6import org.springframework.web.bind.annotation.SessionAttributes;
7
8/**
9 * Handles requests for the application home page.
10 */
11@Controller
12@RequestMapping(value = "/system")
13@SessionAttributes
14public class SalesSystemController {
15
16    /**
17     * 初期画面から呼ばれる。選択した商品の明細追加画面を表示する。
18     * @param form フォームオブジェクト
19     * @param model モデルオブジェクト
20     * @return Viewとしてadd.jspを指定
21     */
22    @RequestMapping(params = "add")
23    public String add(SalesForm form, Model model) {
24        Item item = RecordManager.findItem(form.getGoodsName());
25        //RecordManager.findItem(form.getGoodsName());
26        form.setId(item.getId());
27        form.setGoodsName(item.getName());
28        form.setPrice(item.getPrice());
29        form.setQuantity(item.getQuantity());
30        form.setSubtotal(item.getSubtotal());
31        RecordManager.addToAllList(item);
32        model.addAttribute("nameList", RecordManager.makeNameList());
33        model.addAttribute("allList", RecordManager.getallList());
34        return "add";
35    }
36
37    /**
38     * 細追加画面から呼ばれる。選択した商品を削除し明細追加画面を表示する。
39     * @param form フォームオブジェクト
40     * @param model モデルオブジェクト
41     * @return Viewとしてadd.jspを指定
42     */
43    @RequestMapping(params = "remove")
44    public String remove(SalesForm form, Model model) {
45    	Item item = RecordManager.findItem(form.getGoodsName());
46        form.setId(item.getId());
47        form.setGoodsName(item.getName());
48        form.setPrice(item.getPrice());
49        form.setQuantity(item.getQuantity());
50        form.setSubtotal(item.getSubtotal());
51        RecordManager.removeToAllList(item);
52        model.addAttribute("nameList", RecordManager.makeNameList());
53        model.addAttribute("allList", RecordManager.getallList());
54        return "add";
55    }
56

SalesForm.java

java
1package jp.practice.sales;
2
3import java.util.ArrayList;
4import java.util.List;
5
6public class SalesForm {
7
8	/** 商品ID */
9    private String id;
10    /** 商品名 */
11	private String goodsName;
12    /** 単価 */
13    private int price;
14    /** 売上点数 */
15    private int quantity;
16    /** 小景 */
17    private int subtotal;
18	/** 商品点数 */
19	private int point;
20	/** 削除番号 */
21	private int delNumber;
22
23	List<Item> allList = new ArrayList<>();
24
25	public SalesForm() {
26
27	}
28	/**
29	 * @return id
30	 */
31	public String getId() {
32		return id;
33	}
34
35	/**
36	 * @param id セットする id
37	 */
38	public void setId(String id) {
39		this.id = id;
40	}
41	/**
42	 * @return price
43	 */
44	public int getPrice() {
45		return price;
46	}
47
48	/**
49	 * @param price セットする price
50	 */
51	public void setPrice(int price) {
52		this.price = price;
53	}
54
55	/**
56	 * @return quantity
57	 */
58	public int getQuantity() {
59		return quantity;
60	}
61
62	/**
63	 * @param quantity セットする quantity
64	 */
65	public void setQuantity(int quantity) {
66		this.quantity = quantity;
67	}
68
69	/**
70	 * @return subtotal
71	 */
72	public int getSubtotal() {
73		return subtotal;
74	}
75
76
77	/**
78	 * @return goodsName
79	 */
80	public String getGoodsName() {
81		return goodsName;
82	}
83
84	/**
85	 * @param goodsName セットする goodsName
86	 */
87	public void setGoodsName(String goodsName) {
88		this.goodsName = goodsName;
89	}
90
91	/**
92	 * @return point
93	 */
94	public int getPoint() {
95		return point;
96	}
97
98	/**
99	 * @param point セットする point
100	 */
101	public void setPoint(int point) {
102		this.point = point;
103	}
104
105	/**
106	 * @return delNumber
107	 */
108	public int getDelNumber() {
109		return delNumber;
110	}
111
112	/**
113	 * @param delNumber セットする delNumber
114	 */
115	public void setDelNumber(int delNumber) {
116		this.delNumber = delNumber;
117	}
118
119	/**
120	 * @return allList
121	 */
122	public List<Item> getAllList() {
123		return allList;
124	}
125}

RecordManager.java

java
1package jp.practice.sales;
2
3import java.util.ArrayList;
4import java.util.List;
5
6/**
7 * 商品データ全体を保持するクラス
8 */
9public final class RecordManager {
10
11	/** 明細データ */
12	static List<Item> allList = new ArrayList<>();
13
14    /** 商品データ */
15    private static final List<Item> list = new ArrayList<Item>();
16    static {
17        list.add(new Item("A00101", "油性ボールペン", 60, 0, 0));
18        list.add(new Item("A00201", "極細ボールペン", 120, 0, 0));
19        list.add(new Item("A00301", "蛍光ペン６色セット", 420, 0, 0));
20        list.add(new Item("A00401", "シャープペンシル", 100, 0, 0));
21        list.add(new Item("A00501", "鉛筆H（１ダース）", 400, 0, 0));
22        list.add(new Item("B00101", "無線綴ノートA4", 100, 0, 0));
23        list.add(new Item("B00201", "リングノートA4", 120, 0, 0));
24        list.add(new Item("B00301", "領収書", 350, 0, 0));
25        list.add(new Item("C00101", "はさみ（青）", 128, 0, 0));
26        list.add(new Item("C00201", "ステープラー", 338, 0, 0));
27        list.add(new Item("C00301", "２穴パンチ", 128, 0, 0));
28        list.add(new Item("C00401", "ゼムクリップ", 98, 0, 0));
29        list.add(new Item("C00501", "消しゴム", 58, 0, 0));
30    }
31
32    /**
33     * 引数で指定された商品名に一致する商品データを返却
34     * @param name 検索キーとなる商品名
35     * @return 検索結果の商品データ
36     */
37    public static Item findItem(String goodsName) {
38    	int index = list.indexOf(new Item("", goodsName, 0,0,0));
39    	return list.get(index);
40    }
41
42    /**
43     * 商品データを返す。
44     * @return 商品データ
45     */
46    public static List<Item> getNameist() {
47        return list;
48    }
49
50    /**
51     * 明細データを返す。
52     * @return 明細データ
53     */
54    public static List<Item> getallList() {
55        return allList;
56    }
57
58    /**
59     * リストに商品情報を 1 件追加する
60     */
61    public static void addToAllList(Item newItem) {
62    	Item item = new Item(newItem.getId(), newItem.getName(), newItem.getPrice(),newItem.getQuantity(),newItem.getSubtotal());
63        allList.add(item);
64    }
65
66    /**
67     * リストに商品情報を 1 件削除する
68     */
69    public static void removeToAllList(Item newItem) {
70    	Item item = new Item(newItem.getId(), newItem.getName(), newItem.getPrice(),newItem.getQuantity(),newItem.getSubtotal());
71        allList.remove(item);
72    }
73
74

Item.java

java
1package jp.practice.sales;
2
3public class Item {
4
5	/** 商品ID */
6    private String id;
7    /** 商品名 */
8    private String name;
9    /** 単価 */
10    private int price;
11    /** 売上点数 */
12    private int quantity;
13    /** 小景 */
14    private int subtotal;
15
16	public Item(String id, String name, int price, int quantity, int subtotal) {
17		this.id = id;
18		this.name = name;
19		this.price = price;
20		this.quantity = quantity;
21		this.subtotal = subtotal;
22	}
23
24	/**
25	 * @return id
26	 */
27	public String getId() {
28		return id;
29	}
30
31	/**
32	 * @param id セットする id
33	 */
34	public void setId(String id) {
35		this.id = id;
36	}
37
38	/**
39	 * @return name
40	 */
41	public String getName() {
42		return name;
43	}
44
45	/**
46	 * @param name セットする name
47	 */
48	public void setName(String name) {
49		this.name = name;
50	}
51
52	/**
53	 * @return price
54	 */
55	public int getPrice() {
56		return price;
57	}
58
59	/**
60	 * @param price セットする price
61	 */
62	public void setPrice(int price) {
63		this.price = price;
64	}
65
66	/**
67	 * @return quantity
68	 */
69	public int getQuantity() {
70		return quantity;
71	}
72
73	/**
74	 * @param quantity セットする quantity
75	 */
76	public void setQuantity(int quantity) {
77		this.quantity = quantity;
78	}
79
80	/**
81	 * @return subtotal
82	 */
83	public int getSubtotal() {
84		return subtotal;
85	}
86
87
88
89	/**
90	 * @param subtotal セットする subtotal
91	 */
92	public void setSubtotal(int subtotal) {
93		this.subtotal = subtotal;
94	}
95
96
97
98
99}

行動規範の内容に同意します

回答1件

ベストアンサー

削除対象のItemインスタンスが、allListに格納されているItemと完全に同じインスタンスならこの実装でも良いかと思いますが、addToAllListの処理で、new Item()を作っており、同じインスタンスではないので削除できません。

削除処理を実装するのであれば、そのItemが一意に定められる項目（この実装では id です）が同じ値のものを削除すればよいでしょう。

java
1	public static void removeToAllList(String targetId) {
2		for (Item item : allList) {
3			if (item.getId().equals(targetId)) {
4				allList.remove(item);
5				return;
6			}
7		}
8	}

投稿2020/08/28 17:39

A-pZ

総合スコア12011

icecleam

2020/08/28 17:55

ご回答ありがとうございます。 removeToAllListを上記のように修正したのですが、ラジオボタンで指定した商品ではなく一番直近で追加した商品データが削除されるようになりました。これをラジオボタンで選択したものを削除できるようにしたいのですが、どのようにしたらいいでしょうか。。上記に伴うremoveメソッド内の修正 RecordManager.removeToAllList(item); ↓ RecordManager.removeToAllList(item.getId());