Servlet-JSP での条件分岐

Question

入力された結果に応じて下記のように画面を分岐させたいと思っています。
①入力された文字が全て正しければ、確認画面
②一つでもエラーがあれば、エラー画面

ArrayListを使ってエラー文言を保存（エラー画面で表示する）するので、『list.size()>0』で分岐を試みたのですが文字入力の結果にかかわらずエラー画面に遷移してしまいます。目的と異なりますが、if-elseの遷移ページを逆にすると文字入力の結果に関わらず確認画面に遷移してしまいます。

恐らくは条件分岐のやり方が良くないのかと推測しているのですがご教授ください。
※各メソッドは別クラスで明記

＜使用ソフト＞
eclipse Version: Oxygen.3a Release (4.7.3a)
Apache Tomcat/8.0.50

package ex;

import java.io.IOException;
import java.util.ArrayList;

import javax.servlet.RequestDispatcher;
import javax.servlet.ServletException;
import javax.servlet.http.HttpServlet;
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;
import javax.servlet.http.HttpSession;

/**
 * ValidateServlettクラス
 */
public class ValidateServlet extends HttpServlet {

	protected void doPost(HttpServletRequest request,
			HttpServletResponse response) throws ServletException, IOException {

		// レスポンスの為のコンテンツタイプの設定
		response.setContentType("text/html; charset=utf-8");
		// リクエストのエンコードの設定
		request.setCharacterEncoding("utf-8");

		// セッションの取得
		HttpSession session = request.getSession();
		// セッションからformBeanを取得
		FormBean formBean = (FormBean) session.getAttribute("formBean");

		String code = request.getParameter("code");
		String name = request.getParameter("name");
		String address = request.getParameter("address");
		String number = request.getParameter("number");
		String msg = "";

		// FormBeanクラスに入力データをセット
		formBean.setCode(code);
		formBean.setName(name);
		formBean.setAddress(address);
		formBean.setNumber(number);

		// formBeanをセッションにセット
		session.setAttribute("formBean", formBean);

		//ArrayListをインスタンス生成
		ArrayList<String> list = new ArrayList<String>();

		//従業員コードの文字入力チェック
		if (ValidateUtil.isNull(code)) {
			msg = "従業員コードを入力してください";
			list.add(msg);
		}
		if (ValidateUtil.isHalf(code)) {
			msg = "従業員コードは半角英数字で入力してください";
			list.add(msg);
		}
		if (ValidateUtil.isMaxLength(code, 20)) {
			msg = "従業員コードは20文字以内に入力してください";
			list.add(msg);
		}

		//従業員氏名の文字チェック
		if (ValidateUtil.isNull(name)) {
			msg = "従業員氏名を入力してください";
			list.add(msg);
		}

		if (ValidateUtil.isFull(name)) {
			msg = "従業員氏名は全角文字で入力してください";
			list.add(msg);
		}

		if (ValidateUtil.isMaxLength(name, 30)) {
			msg = "従業員氏名は30文字以内に入力してください";
			list.add(msg);
		}

		//住所の文字チェック
		if (ValidateUtil.isNull(address)) {
			msg = "住所を入力してください";
			list.add(msg);
		}

		if (ValidateUtil.isFull(address)) {
			msg = "住所は全角文字で入力してください";
			list.add(msg);
		}

		if (ValidateUtil.isMaxLength(address, 10)) {
			msg = "住所は30文字以内に入力してください";
			list.add(msg);
		}

		//電話番号の文字チェック
		if (ValidateUtil.isNull(number)) {
			msg = "電話番号を入力してください";
			list.add(msg);
		}

		if (ValidateUtil.isHalfNum(number)) {
			msg = "電話番号は半角数字で入力してください";
			list.add(msg);
		}

		if (ValidateUtil.isMaxLength(number, 10)) {
			msg = "電話番号は30文字以内に入力してください";
			list.add(msg);
		}

		// セッションに値設定
		session.setAttribute("list", list);

		//結果に応じてページ遷移を変える
		if (list.size() > 0) {
			RequestDispatcher RD = request.getRequestDispatcher("/ex/error.jsp");
			// forwardメソッドで、処理を転送
			RD.forward(request, response);
		} else {
		RequestDispatcher rd = request.getRequestDispatcher("/ex/confirm.jsp");
		// forwardメソッドで、処理を転送
		rd.forward(request, response);
		}
	}
}

Comments

[m.ts10806]

デバッグはされたのでしょうか。

[tatakauneko3]

原因が分からないのですがデバッグが作動しません。デバッグを選んでも実行が活性化されず『デバッグ構成』しかありません。

[dodox86]

条件分岐の仕方云々ではなく、入力されている値がすべて適切かまたはValidateUtil.isなんちゃらのメソッドにバグがあるのでは？

[m.ts10806]

デバッグ環境整えてください。
（そもそも「デバッグとは」から理解するべきでは）

[tatakauneko3]

m.ts10806様
おっしゃていることは分かります。ただ、Javaでは出来ていたデバックがJavaEEだと出来なくて調べても分からず、初めて間もない私にはその違いが分からず質問しております。

[tatakauneko3]

dodox86様
ありがとうございます。ValidateUtil.isのメソッド部分はテキスト提供のものを使っているので恐らくはその可能性は低いかなと考えております。

[jimbe]

デバッグ環境が作れないのであれば、入力チェックを一旦全てコメントアウトして確認画面に遷移するのをチェックした後、入力チェックを一つずつ有効にしていってエラーに飛ぶ原因はどれなのかを探す等してみては如何でしょうか。

というか、エラー画面で list に入ったメッセージを表示しているのですよね。それで何が問題なのか分かるのではないでしょうか。
(エラーメッセージとチェックのパラメータが違うような個所もありますが…。)

ValidateUtil の isNull は null の時に true になるようですのに、isFull は全角文字列ではない時に true になるのでしょうか…。

Answer 1

単純に list に必ず何かしらのエラーメッセージが追加されてしまう。
つまり、エラーチェックのどれかに必ず引っかかっているという事は無いでしょうか？
エラーチェック前後でサイズを確認（ログに出す）して、
エラーチェック後にサイズが１以上であれば、
何のメッセージが登録されているかログで表示し確認してみるのはいかがでしょうか？

Comments

[tatakauneko3]

回答頂きありがとうございます。お礼が遅くなり申し訳ございません。