入力ユーザーチェック

Question

//ユーザー登録に関するリクエストを処理するコントローラ

package servlet;

import java.io.IOException;
import java.util.HashMap;
import java.util.Map;

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

import model.RegisterUserLogic;
import model.User;

@WebServlet("/RegisterUser")
public class RegisterUser extends HttpServlet {
	private static final long serialVersionUID = 1L;

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

		// サーブレットクラスの動作を決定する「action」の値をリクエストパラメータから取得
		String action = request.getParameter("action");

		if(!"done".equals(action)){
			// 「ユーザー登録」をリクエストされたときの処理
			RequestDispatcher dispatcher = request.getRequestDispatcher("/WEB-INF/jsp/registerForm.jsp");
			dispatcher.forward(request, response);
		} else {
			// 「登録」をリクエストされたときの処理
			RequestDispatcher dispatcher = request.getRequestDispatcher("/WEB-INF/jsp/registerDone.jsp");
			dispatcher.forward(request, response);
		}
	}

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

		// リクエストの文字コードを指定
		request.setCharacterEncoding("UTF-8");

		// サーブレットクラスの動作を決定する「action」の値をリクエストパラメータから取得
		String action = request.getParameter("action");

		if (!"done".equals(action)) {
			//「確認」をリクエストされたときの処理

			String nickName = request.getParameter("nickName");
			String eMail = request.getParameter("eMail");
			String pass = request.getParameter("pass");
			String userName = request.getParameter("userName");

			//入力チェック
			Map<String, String>errors = new HashMap<String, String>();

		
			if( (nickName == null || nickName.trim().length() == 0) &&
				(eMail == null || eMail.trim().length() == 0) &&
				(pass == null || pass.trim().length() == 0) &&
				(userName == null || userName.trim().length() == 0)){
				errors.put("nickName", " * 氏名を入力してください。");
				errors.put("eMail", " * メールアドレスを入力してください。");
				errors.put("pass", " * パスワードを入力してください。");
				errors.put("userName", " * ユーザー名を入力してください。");
			}
// 登録するユーザー情報を生成
			User registerUser = new User(nickName, eMail, pass, userName);

			// セッションスコープに登録ユーザーを保存
			HttpSession session = request.getSession();
			session.setAttribute("registerUser", registerUser);

			if(!errors.isEmpty()){   //errorsが空でない時（入力されていないのでエラーメッセージがある時）
				//入力エラーがある場合は入力画面に戻す
				request.setAttribute("errors", errors);
				RequestDispatcher dispatcher = request.getRequestDispatcher("/WEB-INF/jsp/registerForm.jsp");
				dispatcher.forward(request, response);

			}else{
				//エラーがない時、確認画面にフォワードする
			RequestDispatcher dispatcher = request.getRequestDispatcher("/WEB-INF/jsp/registerConfirm.jsp");
			dispatcher.forward(request, response);
			}

		} else {
			// 登録確認画面から「登録」をリクエストされたときの処理

			// セッションスコープに保存された登録ユーザーを取得
			HttpSession session = request.getSession();
			User registerUser = (User) session.getAttribute("registerUser");

			// 登録処理の呼び出し
			RegisterUserLogic logic = new RegisterUserLogic();
			logic.execute(registerUser);

			// 不要となったセッションスコープ内のインスタンスを削除
			session.removeAttribute("registerUser");

			// 登録後のリダイレクト
			response.sendRedirect(request.getServletContext().getContextPath() + "/RegisterUser?action=done");
		}
	}
}

この場合、氏名、メール、パスワード、ユーザー名が全て、入力されていない時にメッセージが出ると思うのですが、

例えば、メールだけとか

氏名とメールだけが空の場合の

書き方は同じように
if文で書いていくのでしょうか？

if((nickName == null || nickName.trim().length() == 0) && (eMail == null || eMail.trim().length() == 0)){
errors.put("nickName", " * 氏名を入力してください。");
errors.put("eMail", " * メールアドレスを入力してください。");
}

パターンの種類が多くてもifをたくさん使うやり方が望ましいのでしょうか？？

Answer 1

どのやり方もケースバイケースではありますが、この場合、私なら氏名、メール、パスワード、ユーザー名について一つずつif文による判定を入れ、エラーになった分のエラー文言を配列に格納します。

その後、配列に一つでも値が入っていれば、その値を繰り返し処理で全て表示します。

これで、一つ一つのエラーチェックに対して、該当するエラーのみを表示することが出来ます。

■処理概要
１．エラー文言配列「error」を宣言
２．氏名が入力されていない → errorに「氏名を入力して下さい」を格納
３．メールが入力されていない → errorに「メールを入力して下さい」を格納
４．パスワードが入力されていない → errorに「パスワードを入力して下さい」を格納
５．ユーザーが入力されていない → errorに「ユーザーを入力して下さい」を格納
６．errorの配列の長さが0でなければ、errorに格納されている値を全て表示する

※追記

例えばこんな感じでしょうか。デバッグしていないので動くかどうかは分かりません。参考程度としてください。

ArrayList<String> errors = new ArrayList<String>();

if(nickName == null){
  errors.add(" * 氏名を入力してください。");
}

if(eMail == null){
  errors.add("  * メールアドレスを入力してください。");
}

if(pass == null){
  errors.add(" * パスワードを入力してください。");
}

if(userName == null){
  errors.add(" * ユーザー名を入力してください。");
}

if(errors.isEmpty == false){
  //エラー処理
  for(int i=0; i < errors.size(); i++) {
    System.out.println(errors.get(i));
  }
}

Comments

[Yoshi--]

ありがとうございます。

まだ経験が浅い為、、
例えばでいいのですが、どーいったソースコードになるのでしょうか？？
お手数ですが、よろしくお願いいたします

[akabee]

夜になっても良いのであれば添削してみますが、その前に他の方から回答があるかもしれませんね。
ちなみに、どのあたりのコーディングのイメージが掴めませんでしょうか？配列にはArrayListを用いるのが良いかと思いますよ。

[Yoshi--]

ありがとうございます！！

[akabee]

回答編集しておきました。

[Yoshi--]

なるほど！どうもありがとうございました。

[akabee]

コード間違えていましたので修正しておきました。errors.isEmpty == true、ではなくerrors.isEmpty == false、です。