なぜデータベースから取得しているname,passと同じく入力しても、ログインができないのでしょうか？

Question

前もってデータベースにcreateとinsertをしておいて、灘,1234と、name,passを指定しました。jspで灘,1234と入力しても、ログインできません。コードの要点を短くまとめ、説明しています。宜しくお願いします。

前もってデータベースに CREATE TABLE MUTTER(NAME VARCHAR(100) NOT NULL,PASS VARCHAR(100) NOT NULL);と入力しました。
続いて、INSERT INTO MUTTER(NAME,PASS)VALUES('灘','1234');
そして、INSERT INTO MUTTER(NAME,PASS)VALUES('綾部','2345');
と入力しました。

index.jspでは、ユーザー名の欄に 灘
パスワードの欄に 1234
を入力しています。

入力した値を
index.jsp

<form action="/example24/Login" method="post">
ユーザー名:<input type="text" name="name"><br>
パスワード:<input type="password" name="pass"><br>
<input type="submit" value="ログイン">

これで送信して、
Login.java

String name=request.getParameter("name");
		String pass=request.getParameter("pass");
		
		User user=new User(name,pass);
	
		LoginLogic loginLogic=new LoginLogic();
		List<Mutter>mutterList=loginLogic.execute(user);
				request.setAttribute("mutterList", mutterList);
		

受け取り、userを作りLoginLogicをnewして、executeメソッド(executeはMutter.DAOのfindAll()を呼ぶ)を呼び出す。
LoginLogic

public  List<Mutter>execute(User user){
		MutterDAO dao=new MutterDAO();
			List<Mutter>mutterList=dao.findAll(user);
			return mutterList;

さらにdao.findAllを呼び出し、データベースからuserのnameとpassに一致しているNAMEとPASSを取得、そして、userName,userPassに代入
Mutter.DAO

public List<Mutter>findAll(User user){
		
		String sql="SELECT NAME,PASS FROM MUTTER WHERE NAME=(?)AND PASS=(?)";
		PreparedStatement pStmt=conn.prepareStatement(sql);
		pStmt.setString(1, user.getName());
		pStmt.setString(2, user.getPass());
		
		ResultSet rs=pStmt.executeQuery();
		
		while(rs.next()){
			
	String userName=rs.getString("NAME");
			String userPass=rs.getString("PASS");
			
		Mutter mutter=new Mutter(userName,userPass);
			mutterList.add(mutter);

そしてmutterListをreturnする(returnされたmutterListはLogin.javaにいきrequest.setAttribute("mutterList", mutterList); このようにリクエストスコープに保存

loginResult.jsp

   <%
    //セッションスコープからユーザー情報を取得
    User loginUser=(User)session.getAttribute("loginUser");
 
    //リクエストスコープに保存されたつぶやきリストを取得
    List<Mutter>mutterList=(List<Mutter>)request.getAttribute("mutterList");
   
    %>
<%if(mutterList!=null){ %>
<% for(Mutter mutter:mutterList){ %>
<%if(mutter.getUserName().length()!=0 && mutter.getUserPass().length()!=0){ %>
<p>ログインに成功しました。</p>
<p>ようこそ<%=loginUser.getName() %>さん</p>
<%} %>
<%}}else{ %>
<p>ログインに失敗しました。</p>
<a href="/example24/">TOPへ</a>
<%}%>

if文で、データベースからリクエストパラメータの入力画面でユーザーが入力した値と同じnameとpassをSELECTしているはずなので、elseとならずにログイン成功

となるはずが、elseとなります。
おかしいところをどうかお教えください。
どうぞよろしくお願いします。

Answer 1

文字コードが相違しているとか？

Comments

[edoooooo]

nameとpassのリクエストパラメータを取得するLogin.java(servlet)のpostメソッドでは、//リクエストパラメータの取得
request.setCharacterEncoding("UTF-8");
このようにUTF-8で指定しています。
　最後にデータを表示するloginResutでは、HTMLの最初で、<%@ page language="java" contentType="text/html; charset=UTF-8"
pageEncoding="UTF-8"%>このように指定しています。
　他にも指定するべき場所があるのでしょうか？
　見当はずれでしたら、どうかお教えください。宜しくお願いします。

[edoooooo]

昨日は、何度実行しても、動かすことができなく、今日になり、eclipseを再起動していないのに、動かすことが、できました。
　これからいろいろとコードを改良していく中で、文字コードにも、気おつけていきたいと思います。ありがとうございました。

Answer 2

ResultSetがどうなっているかを確認してみましたか？

Comments

[edoooooo]

ResultSetクラスのrsにpStmt.executeQuery()を代入して、next()でget()で取得しているのですが、そういうことではないですよね、どうなっているか？とは、どのようなことなのでしょうか？どうぞよろしくお願いします。

[dojikko]

ResultSetにちゃんと正しい(意図する)値が返ってきているかという意味です

> nameとpassをSELECTしているはずなので

と書かれていますが、本当にそうなっていますかという意味です
違うものが返ってきていていたり、比較しようと思っているものが違っていたら、それらを比較したらFalseになりますよね

[edoooooo]

MutterDAOの中で、System.out.println(userName);
System.out.println(userPass);と表示したところ、灘 1234 と入力した通りの、値がコンソールに出力されました。
昨日には、何度実行しても、if elseのどちらの表示も、されなかったのが、今日になり、eclipseを再起動していないにもかかわらず、なぜか、動かすことが、できました。
　回答をいただいているのに、理由がわかりません、申し訳ありません。
ありがとうございました。
しかし、データベースに登録されていない、名前と、passを入力したところ、elseとなるはずが、elseの実行(<a href="/example24/>トップへ</a>が表示されません。
if else文のコードが、たとえifとならなくても、elseにたどり着かないと、思ったのですが、教えていただけないでしょうか？

[edoooooo]

<%if(mutterList!=null){ %>
<% for(Mutter mutter:mutterList){ %>
<%if(mutter.getUserName().length()!=0 && mutter.getUserPass().length()!=0){ %>
<p>ログインに成功しました。</p>
<p>ようこそ<%=loginUser.getName() %>さん</p>
<%} %>
<%}}else{ %>
<p>ログインに失敗しました。</p>
<a href="/example24/">TOPへ</a>
<%}%>

[kaputaros]

横から失礼します。

おそらく、mutterListがnullではないのでは？
0長のリストが返ってきている可能性があるので、nullまたはlength=0で判定したほうが良いかと思います。

ログ出しする際は、中身だけでなく、すべての変数を出来る限りログ出しした方が、原因をつかみやすくなりますよ。
また、変数を出力する際、ただ出すのではなく、変数を[]などでくくって出力されるようにしておくと、スペースが入ってたりすることに気づきやすいです。

[edoooooo]

ありがとうございます。

変数を[ ]でくくって、出力して確認することも、していきます。　ありがとうございます。
無事解決ました。

Answer 3

考えられる点としては dojikko様、HiroshiWatanabe様の点だと思われますが、
String name=request.getParameter("name") で想定通りの値が入っているかをデバッグモードやSystem.outなどで確認してください。

また、SQLの宣言にも気になる記述があります。

SELECT NAME,PASS FROM MUTTER WHERE NAME=(?) AND PASS=(?)

カッコは不要です

Comments

[edoooooo]

System.out.println()で出力したところ、しっかりと、入力した値が、コンソールに表示されました。カッコは、昨日直していたのですが、ダメだったのですが、今日、実行してみたら、なぜが、ログインすることが、できました。ありがとうございました。