if elseをうまく作れないです。 どうぞよろしくお願いします。

Question

doPostメソッドで、新規登録で入力した値がnullでないか?
またはその値が、すでにデータベースにないか?
そのどちらでもない場合に、新規登録を実行して、データベースに登録したいです。
どのように書けば、if else がしっかり組み立てられるのでしょうか？

package servlet;

import java.io.IOException;

import java.util.List;
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.Mutter;
import model.User;
import model.SetLogic;
import model.LoginLogic;

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

	/**
	 * @see HttpServlet#doGet(HttpServletRequest request, HttpServletResponse response)
	 */
	protected void doGet(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
		
		
		RequestDispatcher dispatcher=request.getRequestDispatcher("/WEB-INF/jsp/set.jsp");
		dispatcher.forward(request, response);
		
	}

	/**
	 * @see HttpServlet#doPost(HttpServletRequest request, HttpServletResponse response)
	 */
	protected void doPost(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
		
		request.setCharacterEncoding("UTF-8");
		String name=request.getParameter("name");
		String pass=request.getParameter("pass");
		
		
		User user=new User(name,pass);
		
		if(name.length()==0 || pass.length()==0){
			request.setAttribute("newerrorMsg", "未入力があります。");
			
			RequestDispatcher dispatcher=request.getRequestDispatcher("/WEB-INF/jsp/set.jsp");
			dispatcher.forward(request, response);
			
		}
			
		//LoginLogicを使ってmutterListを取得
		LoginLogic loginLogic=new LoginLogic();
		
		List<Mutter>mutterList=loginLogic.execute(user);
			
			
		if(mutterList.size()!=0){
				request.setAttribute("setErrorMsg","すでに登録されています。");
			
				RequestDispatcher dispatcher=request.getRequestDispatcher("/WEB-INF/jsp/set.jsp");
				dispatcher.forward(request, response);
			}
			//mutter1とする
		for(Mutter mutter1:mutterList){
		 if(mutter1.getUserName().length()!=0||mutter1.getUserPass().length()!=0){
				 request.setAttribute("setErrorMsg", "すでに登録されています。");
				 
				RequestDispatcher dispatcher=request.getRequestDispatcher("/WEB-INF/jsp/set.jsp");
				dispatcher.forward(request, response);
			
		 }else{
	            HttpSession session=request.getSession();
	            session.setAttribute("loginUser",user);
		
		 User loginUser=(User)session.getAttribute("loginUser");
		
		 Mutter mutter=new Mutter(loginUser.getName(),loginUser.getPass());
		 SetLogic setLogic=new SetLogic();
			
			try{
					setLogic.execute(mutter);
			}catch(ClassNotFoundException e){
					e.printStackTrace();
				}
		
		  RequestDispatcher dispatcher=request.getRequestDispatcher("/WEB-INF/jsp/setResult.jsp");
		  dispatcher.forward(request, response);
				}
		}
	}
}

Comments

[A-pZ]

if文やfor文など、処理が分岐または繰り返しするコードに関してはインデントをそろえたほうがデバッグしやすいですし、他の方も見やすいのでできればインデントは整えたほうがよいでしょう。データを取得していると思われる loginLogic.execute(user) の LoginLogic.execute(user)で行われている処理とその結果の内容がわからないですが、おそらく mutter1 の処理は不要かと思われます。

[edoooooo]

ありがとうございます。修正してみます。

Answer 1

List<Mutter>mutterList=loginLogic.execute(user);

if(mutterList.size()!=0){
    request.setAttribute("setErrorMsg","すでに登録されています。");
    RequestDispatcher dispatcher=request.getRequestDispatcher("/WEB-INF/jsp/set.jsp");
    dispatcher.forward(request, response);
}
//mutter1とする
for(Mutter mutter1:mutterList){
    if(mutter1.getUserName().length()!=0||mutter1.getUserPass().length()!=0){
        request.setAttribute("setErrorMsg", "すでに登録されています。");
        RequestDispatcher dispatcher=request.getRequestDispatcher("/WEB-INF/jsp/set.jsp");
        dispatcher.forward(request, response);
    }

内容は細かくわからないのですが、おそらくここは入力された情報でデータベースを照合して、データが有ればエラーメッセージを出す、という仕様ですよね？
であれば、照合結果のリストが空でない(mutterList.size()!=0)時点で終わってますよね？
リストが空ならその次のfor文に入れませんし、空でないならデータベースにその情報が存在したということでその次のループに入る意味がないように思います。

Comments

[edoooooo]

おっしゃられている通りの仕様で考えています。
リストがからの場合は、set.jspにrequestして、エラーメッセージを表示して、
からでない場合は、本当にからでないかgetUserNameで、確認しているのですが、確認は、一つ目だけでいいということでしょうか？

[edoooooo]

無事作成できました。
ありがとうございました。

Answer 2

新規登録できないということが判明したら例外を投げちゃえばいいのではないでしょうか。そして、例外を捕捉した場合は（いろんな原因で例外が飛んでくるわけですが）まとめて処理すればすっきりしそうです。

Comments

[edoooooo]

例外を投げて、例外を受け取った先で、処理ということができるのでしょうか？
redirectで、if elseの処理によって飛ばして、その先でまたif elseするということでしょうか？これは、例外を投げるということではないですか？

[edoooooo]

コードの２つ目のif elseを削除したところ、動かすことができました。
今後複雑になった時には、例外を使えるようにしていきたいと思います。
ありがとうございました。

[unau]

例外処理の要諦は、「正常系と異常系の分離」「大域脱出」にあると考えています。
この辺り、先日別の質問に回答したので、よろしければ見てみてください。
https://teratail.com/questions/33005

Java は特に、メソッドのインタフェースとして「引数の数と型」「戻り値の型」「例外の型」を厳格に要求する、など、例外機構が言語仕様としてきちんと作られています。ですので、複雑な場合でなくても、積極的に例外を採り入れ、正常系と異常系を分離した書き方に慣れていた方が速くステップアップできると思います。