jsp間で画面遷移するときの条件の設定方法

Question

以下のadd.jsp画面で商品をプルダウンから追加し、それをラジオボタンで選択し、その状態で削除ボタンを押下すると選択した商品をリストから削除していきます。
そこでリストが空になった時に初期画面（init.jsp）に遷移するようにしたいのですが、以下のコードだとうまく動きません。
問題の処理は[SalesSystemController.javaのremoveメソッド]だと思うので、そこの処理を載せておきます。。
すみませんが、宜しくお願いします。

SalesSystemController.javaのremoveメソッド

else if(RecordManager.calTotal(RecordManager.getallList()) == 0){
            model.addAttribute("nameList", RecordManager.makeNameList());
            return "init";

SalesSystemController.java

package jp.practice.sales;

import javax.servlet.http.HttpServlet;
import javax.servlet.http.HttpServletRequest;

import org.springframework.stereotype.Controller;
import org.springframework.ui.Model;
import org.springframework.validation.BindingResult;
import org.springframework.web.bind.annotation.RequestMapping;
import org.springframework.web.bind.annotation.SessionAttributes;

/**
 * Handles requests for the application home page.
 */
@Controller
@RequestMapping(value = "/system")
@SessionAttributes(types = SalesForm.class)
public class SalesSystemController extends HttpServlet {
    private static final String ERRMSG = "点数には1以上100以下の整数を入力してください。";
    private static final String ADDMSG = "明細に追加しました。";
    private static final String FIXMSG = "以下のように売上登録しました。";
    private static final String DELETE = "選択された明細行を削除しました。";
    private static final String ERRMSG2 = "明細行を選択して下さい。";

	/**
     *  起動時および登録完了時に呼ばれる。初期画面を表示する。
     * @param form フォームオブジェクト
     * @param model モデルオブジェクト
     * @return Viewとしてinit.jspを指定
     */
    @RequestMapping(value = "/start")
    public String init(SalesForm form, Model model) {

        model.addAttribute("nameList", RecordManager.makeNameList());
        //model.addAttribute("nameList", RecordManager.makeNameList());
        //form.setGoodsName(RecordManager.getFirstPersonName());
        return "init";
    }

    /**
     * 初期画面から呼ばれる。選択した商品の明細追加画面を表示する。
     * @param form フォームオブジェクト
     * @param model モデルオブジェクト
     * @return Viewとしてadd.jspを指定
     */
    @RequestMapping(params = "add")
    public String add(SalesForm form,BindingResult result, Model model,HttpServletRequest request) {
        Item item = RecordManager.findItem(form.getGoodsName());
        RecordManager.setNewItem(new Item(item.getId(), form.getGoodsName(), item.getPrice(), form.getQuantity(), item.getSubtotal()));
        Item newItem = RecordManager.getNewItem();
        int sb = newItem.getPrice() * newItem.getQuantity();
        newItem.setSubtotal(sb);
		model.addAttribute("nameList", RecordManager.makeNameList());
		model.addAttribute("allList", RecordManager.getallList());
		if (result.hasErrors() || 1 > form.getQuantity() || form.getQuantity() > 100) {
			model.addAttribute("total", RecordManager.calTotal(RecordManager.getallList()));
			model.addAttribute("message", ERRMSG);
			return "add";

		  } else {
			RecordManager.addToAllList(newItem);
			model.addAttribute("total", RecordManager.calTotal(RecordManager.getallList()));
			model.addAttribute("message2", ADDMSG);
			return "add";
		}
		//model.removeAttribute("SalesForm");
		//HttpSession session = request.getSession(true);
	}

    /**
     * 細追加画面から呼ばれる。選択した商品を削除し明細追加画面を表示する。
     * @param form フォームオブジェクト
     * @param model モデルオブジェクト
     * @return Viewとしてadd.jspを指定
     */
    @RequestMapping(params = "remove")
    public String remove(SalesForm form, BindingResult result, Model model, HttpServletRequest request) {
    	String num = null;
    	num = request.getParameter("radio");

    	if (num != null){
    		int number = Integer.parseInt(num);
    		RecordManager.removeToAllList(number);
    		model.addAttribute("message4", DELETE);
    		model.addAttribute("total", RecordManager.calTotal(RecordManager.getallList()));
    	}else if(RecordManager.calTotal(RecordManager.getallList()) == 0){
    		model.addAttribute("nameList", RecordManager.makeNameList());
            return "init";

    	} else {
        	model.addAttribute("message5", ERRMSG2);
        }
    		model.addAttribute("total", RecordManager.calTotal(RecordManager.getallList()));
    		model.addAttribute("nameList", RecordManager.makeNameList());
    		model.addAttribute("allList", RecordManager.getallList());
    	return "add";
    }

    /**
     * 確認画面で登録を押した時に呼ばれる。データを登録して初期画面表示へ遷移する。
     * @param form フォームオブジェクト
     * @param model モデルオブジェクト
     * @return initメソッドへのリンクを指定
     */
    @RequestMapping(params = "firm")
    public String firm(SalesForm form, Model model) {
    	long value = System.currentTimeMillis();
        Item item = RecordManager.findItem(form.getGoodsName());
        RecordManager.setNewItem(new Item(item.getId(), form.getGoodsName(), item.getPrice(), form.getQuantity(), item.getSubtotal()));
        Item newItem = RecordManager.getNewItem();
        int sb = newItem.getPrice() * newItem.getQuantity();
        newItem.setSubtotal(sb);
        model.addAttribute("salesid", value);
        model.addAttribute("message3", FIXMSG);
        model.addAttribute("total", RecordManager.calTotal(RecordManager.getallList()));
        model.addAttribute("nameList", RecordManager.makeNameList());
        model.addAttribute("allList", RecordManager.getallList());
        return "fix";
    }
}

add.jsp

<%@ page session="false" language="java"
	contentType="text/html; charset=UTF-8" pageEncoding="UTF-8"%>
<%@page import="java.time.format.DateTimeFormatter"%>
<%@ page import="java.time.LocalDate"%>
<%@ taglib uri="http://java.sun.com/jsp/jstl/core" prefix="c"%>
<%@ taglib uri="http://www.springframework.org/tags/form" prefix="form"%>
<%@ page import="java.util.*"%>
<%@ page import="java.time.*"%>
<!DOCTYPE html>
<html>
<head>
<link rel="stylesheet" type="text/css"
	href="/sales/resources/css/common.css" />
<title>明細追加画面</title>
</head>
<body>
	<form:form modelAttribute="salesForm" action="/sales/system" var="list">
		<div class="header">
			<span class="titleName">オンラインショップ</span>
			<%
			DateTimeFormatter format = DateTimeFormatter.ofPattern("MM月dd日(E)");
			%>
			<span class="date"><%=format.format(LocalDate.now())%></span>
		</div>
		<div class="main"></div>
		<div>明細追加画面</div>
			商品：
			<form:select path="goodsName" items="${nameList}" />
		<div>
			点数：
			<form:input path="quantity" items="${allList}" />
		</div>
		<div>
			<input type="submit" name="add" value="明細追加" />
		</div>
		<div class="message">
			<font color="#ff0000"> <c:out value="${message}" /></font> <font
				color="#1e90ff"> <c:out value="${message2}" />
			</font> <font color="#1e90ff"> <c:out value="${message4}" />
			</font><font color="#1e90ff"> <c:out value="${message5}" />
			</font>
		</div>

		<table border="1">
			<tr>
				<th>削除</th>
				<th>商品ID</th>
				<th>商品名</th>
				<th>単価</th>
				<th>点数</th>
				<th>小計</th>
			</tr>
			<c:forEach items="${allList}" var="item" varStatus="status">
				<tr class="even">
					<form:hidden path="quantity" />
					<td><input type="radio" name="radio" value="${status.index}" /></td>
					<td><c:out value="${item.id}" /></td>
					<td><c:out value="${item.name}" /></td>
					<td><c:out value="${item.price}" /></td>
					<td><c:out value="${item.quantity}" /></td>
					<td><c:out value="${item.subtotal}" /></td>
				</tr>
			</c:forEach>
		</table>
		<div>
			合計：
			<c:out value="${total}" />
			円
		</div>
		<input type="submit" name="firm" value="確定" />
		<input type="submit" name="remove" value="削除" />
	</form:form>
</body>
</html>

init.jsp

<%@ page session="false" language="java"
	contentType="text/html; charset=UTF-8" pageEncoding="UTF-8"%>
<%@page import="java.time.format.DateTimeFormatter"%>
<%@ page import="java.time.LocalDate"%>
<%@ taglib uri="http://java.sun.com/jsp/jstl/core" prefix="c"%>
<%@ taglib uri="http://www.springframework.org/tags/form" prefix="form"%>
<%@ page import="java.util.*"%>
<%@ page import="java.time.*"%>
<!DOCTYPE html>
<html>
<head>
<link rel="stylesheet" type="text/css"
	href="/sales/resources/css/common.css" />
<title>売上システム(初期画面)</title>
</head>
<body>
	<form:form modelAttribute="salesForm" action="/sales/system" var="list">
		<div class="header">
			<span class="titleName">オンラインショップ</span>
			<%
				DateTimeFormatter format = DateTimeFormatter.ofPattern("MM月dd日(E)");
			%>
			<span class="date"><%=format.format(LocalDate.now())%></span>
		</div>
		<div class="main"></div>
		<div>初期画面</div>
			商品：
		<form:select path="goodsName" items="${nameList}" />
		<div>
			点数：
			<form:input path="quantity" items="${allList}" />
		</div>
		<input type="submit" name="add" value="明細追加" />
	</form:form>
</body>
</html>

Answer 1

if (num != null){
  int number = Integer.parseInt(num);
} else if (RecordManager.calTotal(RecordManager.getallList()) == 0){
  model.addAttribute("nameList", RecordManager.makeNameList());
  return "init";
} else {

num が null になっていないと、else if に入りません。私が思うに、このControllerの実装はこうでしょう。

        String num = request.getParameter("radio");
        if (num == null) {
            model.addAttribute("message5", ERRMSG2);
            model.addAttribute("total", RecordManager.calTotal(RecordManager.getallList()));
            model.addAttribute("nameList", RecordManager.makeNameList());
            model.addAttribute("allList", RecordManager.getallList());
            return "add";
        }

        int number = Integer.parseInt(num);
        RecordManager.removeToAllList(number);
        model.addAttribute("message4", DELETE);
        model.addAttribute("total", RecordManager.calTotal(RecordManager.getallList()));
        model.addAttribute("nameList", RecordManager.makeNameList());
        model.addAttribute("allList", RecordManager.getallList());
            
        if(RecordManager.calTotal(RecordManager.getallList()) == 0){
            return "init";
        }

        return "add";

余談ですが、ControllerでHttpServletRequestを受け取っていますが、SpringMVCを利用されているのですから、ラジオボタンの入力もSalesFormに追加するべきでしょう。

Comments

[icecleam]

ありがとうございます。
上記のソースで実装ですることができました。

>SpringMVCを利用されているのですから、ラジオボタンの入力も>SalesFormに追加するべきでしょう。
こちらに関しては、以前もご指摘をいただいて@NotNullを使って、実装を試みたのですが、どうしてもエラーが出てしまい（インポートはしました）、現状はこちらのソースで実装させていただいています。
せっかくご指摘をいただいたのに申し訳ありません。