JSPのエラー時の値再表示について

現在研修課題でオンラインショップのサイトを作成しています。
初期画面で商品と商品数を入力、明細画面に遷移し、削除や追加が行える。
確定ボタンで、確定画面が表示される。
終了ボタンで、初期画面に戻るというものです。
その際、明細画面にてエラー時の値表示で躓いてしまいました。

1. 前提・実現したいこと

商品数の入力は「1以上100以下の整数」で行っており、
101や0といった整数のエラー表示は実現しています。

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

テキストボックスの値をエラー値のみ表示したいのですが、
1.1やA、空白といった場合のエラー値（整数以外）の表示が以下の様に正しく表示されません。
＜1.1と入力した場合＞

＜Aを入力した場合＞

＜空白を入力した場合＞

3. 該当のソースコード

SalesForm.java
1package jp.practice.sales;
2
3import java.util.ArrayList;
4import java.util.List;
5import javax.validation.constraints.NotBlank;
6import javax.validation.constraints.Positive;
7
8public class SalesForm {
9
10	/** 商品ID */
11	private String id;
12
13	/** 商品名 */
14	private String goodsName;
15
16	/** 単価 */
17	private int price;
18
19	/** 売上点数 */
20	@NotBlank
21	@Positive
22	private int quantity;
23
24	/** 小計 */
25	private int subtotal;
26
27	/** 商品点数 */
28	private int point;
29
30	/** 削除番号 */
31	private int delNumber;
32
33	/** 合計 */
34	private int total;
35
36~getter、setterは省略~

SalesSystemController.java
1package jp.practice.sales;
2
3import java.text.NumberFormat;
4import javax.servlet.http.HttpServletRequest;
5import org.springframework.stereotype.Controller;
6import org.springframework.ui.Model;
7import org.springframework.validation.BindingResult;
8import org.springframework.validation.annotation.Validated;
9import org.springframework.web.bind.annotation.RequestMapping;
10import org.springframework.web.bind.annotation.SessionAttributes;
11import org.springframework.web.bind.support.SessionStatus;
12
13@Controller
14@SessionAttributes(types = SalesForm.class)
15@RequestMapping(value = "/system")
16/** メッセージ表示 商品追加エラー時 */
17	private static final String ADDNGMSG = "点数には1以上100以下の整数を入力してください。";
18
19@RequestMapping(params = "add")
20	public String add(@Validated SalesForm form, BindingResult result, Model model, SessionStatus sessionStatus, HttpServletRequest request) {
21		model.addAttribute("nameList", RecordManager.makeNameList());
22		model.addAttribute("allList", form.getAllList());
23
24		model.addAttribute("id", form.getId());
25
26		int delCheck = form.getDelNumber();
27		delCheck = 0;
28        model.addAttribute("delNumber", delCheck);
29
30		Item item = RecordManager.findItem(form.getGoodsName());
31
32		form.setNewItem(new Item(item.getId(), form.getGoodsName(),
33				item.getPrice(), form.getQuantity(), item.getSubtotal()));
34
35		Item newItem = form.getNewItem();
36
37		int subTotal = newItem.getPrice() * newItem.getQuantity();
38		newItem.setSubtotal(subTotal);
39
40		NumberFormat total = NumberFormat.getNumberInstance();
41
42		final int quentityUnder = 0;
43		final int quentityOver  = 101;
44
45		if (result.hasErrors()) {
46			model.addAttribute("total", total.format(form.total(form.getAllList())));
47			model.addAttribute("alarmMessage", ADDNGMSG);
48			return ADD;
49		} else if (quentityUnder < form.getQuantity() && form.getQuantity() < quentityOver) {
50			form.addToAllList(newItem);
51			form.setGoodsName("A00101");
52			model.addAttribute("total", total.format(form.total(form.getAllList())));
53			model.addAttribute("quantity", "1");
54			model.addAttribute("message", ADDMSG);
55			return ADD;
56		} else if (form.zero2List()) {
57			sessionStatus.setComplete();
58			model.addAttribute("alarmMessage", ADDNGMSG);
59			return INIT;
60		} else {
61			model.addAttribute("total", total.format(form.total(form.getAllList())));
62			request.setAttribute("quantity", form.getQuantity());
63			model.addAttribute("alarmMessage", ADDNGMSG);
64			return ADD;
65		}
66	}

add.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<%@page import="java.time.format.DateTimeFormatter"%>
7<%@ page import="java.util.*"%>
8<%@ page import="java.time.*"%>
9<!DOCTYPE html>
10<html>
11<script>
12function offradio() {
13	  var elem = document.querySelectorAll('input[type="radio"]');
14	  for(var i=0; i<elem.length; i++) {
15	    elem[i].checked = false;
16	  }
17	}
18</script>
19<head>
20<link rel="stylesheet" type="text/css"
21	href="/sales/resources/css/common.css" />
22<title>売上システム(明細追加画面)</title>
23</head>
24<body>
25	<form:form modelAttribute="salesForm" method="post" action="/sales/system" var="list">
26		<div class="header">
27			<span class="titleName">オンラインショップ</span>
28			<%
29				DateTimeFormatter format = DateTimeFormatter.ofPattern("M月d日(E)");
30			%>
31			<div class="date"><%=format.format(LocalDate.now())%></div>
32		</div>
33		<div class="main">
34			<div>
35				<span class="goodsName">商品名：</span>
36				<form:select path="goodsName">
37				<form:options items="${nameList}"/>
38				</form:select>
39			</div>
40			<div>
41				<span class="point">&emsp;点数：</span>
42				<form:input path="quantity" items="${allList}" size="17"
43					maxlength="3" value="${quantity}" />
44			</div>
45			<div>
46				<input type="submit" name="add" value="明細追加" onclick="offradio();"/>
47			</div>
48			<div class="message">
49				<p>
50					<c:out value="${message}" />
51				</p>
52			</div>
53			<div class="alarmMessage">
54				<p>
55					<c:out value="${alarmMessage}" />
56				</p>
57			</div>
58		</div>
59		<div class="footer">
60			売上明細
61			<table>
62				<tr>
63					<th width="40">削除</th>
64					<th width="80">商品ID</th>
65					<th width="170">商品名</th>
66					<th width="40">単価</th>
67					<th width="40">点数</th>
68					<th width="60">小計</th>
69				</tr>
70				<c:forEach items="${allList}" var="item" step="1" varStatus="status">
71					<tr class="even">
72						<form:hidden path="quantity" />
73						<td style="text-align: center;">
74							<form:radiobutton name="button" path="delNumber" value="${status.count}" />
75						</td>
76						<td style="text-align: center;"><c:out value="${item.id}" /></td>
77						<td style="text-align: center;"><c:out value="${item.name}" /></td>
78						<td style="text-align: right;"><c:out value="${item.price}" /></td>
79						<td style="text-align: right;"><c:out value="${item.quantity}" /></td>
80						<td style="text-align: right;"><c:out value="${item.subtotal}" /></td>
81					</tr>
82				</c:forEach>
83			</table>
84			<div class="total">
85				合計：
86				<c:out value="${total}" />
87				円
88			</div>
89			<div>
90				<input type="submit" name="remove" value="削除"/>
91				<input type="submit" name="fix" value="確定" />
92			</div>
93		</div>
94	</form:form>
95</body>

4. 自分で調べたことや試したこと

ネットでの検索や同じ課題を作成した方に相談してみましたが同じような現象が起こっておらず解決に至りませんでした。皆様のお力をお借りしたいです。追加で必要なコード等あれば追加します。

y_waiwai

2022/05/10 06:34

正しく表示されてるように見えますが、どういうふうに正しくないんでしょうか

CatCat

2022/05/10 06:49 編集

101や0といった整数は問題なく表示されますが、 1.1やAといった整数以外の数値を入力すると2つ目の画像の様に複数値が表示されてしまいます。ーー例ーー本来表示したい数値「1.1」実際に表示される数値「1.1、1、1」説明不足で申し訳ありません。

y_waiwai

2022/05/10 06:57

Aとか空白の場合はどういう表示になるんでしょう。できればそこらへんの説明は、質問文の方に追記してください

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

回答1件

<form:input path="quantity" items="${allList}" size="17" maxlength="3" value="${quantity}" />

ここは点数を1つ入力できればいいだけなのに、itemsというリストを操作するときの属性が記述されています。

<form:input path="quantity" maxlength="3" value="${quantity}" />

これだけでいけるのではないでしょうか？

https://springhack.com/

投稿2022/05/13 12:26

noricgeographic

総合スコア21

CatCat

2022/05/16 01:13

回答ありがとうございます。指摘して頂いた通り編集し、再度試してみましたが、結果は変わらず複数値が表示されるままでした。

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

あなたの回答

tips

プレビュー

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

質問の解決につながる回答をしましょう。サンプルコードなど、より具体的な説明があると質問者の理解の助けになります。また、読む側のことを考えた、分かりやすい文章を心がけましょう。

まだベストアンサーが選ばれていません

会員登録して回答してみよう

アカウントをお持ちの方は

15分調べてもわからないことは
teratailで質問しよう！

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう！