if文でnullの場合にしているelseの内容の通りに動きません、どうかお教えください。

Question

なぜかindex.jspでUserクラスのユーザー名を空欄にしたのに、
loginResult.jspでexample11/Mainに画面が移ってしまいます。
<%if(loginUser!=null && loginUser.getName()!=null && loginUser.getPass()!=null){
のように指定しているのになぜなのでしょうか？

Login.java(servlet)

//Userインスタンス(ユーザー情報)の生成
		User user=new User(name,pass);
		
		//ログイン処理
		LoginLogic loginLogic =new LoginLogic();
		boolean isLogin=loginLogic.execute(user);
		
		//ログイン成功時の処理 
		if(isLogin){
			//ユーザー情報をセッションスコープに保存
			HttpSession session=request.getSession();
			session.setAttribute("loginUser", user);
	

loginResult.jsp(WEB-INF/jsp)

<h1>どこ粒ログイン</h1>
<%if(loginUser!=null && loginUser.getName()!=null && loginUser.getPass()!=null){
	
	%>
<p>ログインに成功しました。</p>
<p>ようこそ<%=loginUser.getName() %>さん</p>
<a href="/example11/Main">つぶやき投稿.閲覧へ</a>
<%}else{ %>
<p>ログインに失敗しましt。</p>
<a href="/example11/">TOPへ</a>
<%} %>

User

package model;

import java.io.Serializable;
public class User implements Serializable{
	private String name;//ユーザー名
	private String pass;//パスワード
public User(){}
public User(String name,String pass){
	this.name=name;
	this.pass=pass;
}
public String getName(){return name;}
public String getPass(){return pass;}

}

index.jsp

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

Answer 1

getNameは、 UserクラスのString name; の値を返しますから、nullが返される事はありません。
nameに何も代入されていないのであれば、空文字列("")が返されます。

<%if(loginUser!=null && loginUser.getName()!=null && loginUser.getPass()!=null){ 

の null を "" に書き換えれば、動作するのではないかと思います。

Comments

[kaputaros]

横から失礼します。

Userクラスには引数なしコンストラクタもあり、そちらでインスタンスを作成してすぐにgetName()等した場合は、参照型フィールドの初期値であるnullが返ると思うので、その言い方だと語弊を招くのでは？？

この場合引数なしコンストラクタは private にするか宣言しないでおくべきかとは思いますが･･･^^;

[edoooooo]

ありがとうございました。length>0 以外でも""でいいのですね
勉強になりました。
　ありがとうございます。

Answer 2

こんにちは。
nullじゃなくて空文字""なのではないですかね。
各項目をログ出しもしくはそのまま出力してして確認してみましたか？

カラッポかどうかを確認する際は、共通クラスなどに
「null または 空文字 かどうかを判定するメソッド」を用意して使ったりします。

(例)
public static isNotBlank(String target){
　boolean flag = true;
  if(target == null || "".equals(target)){
    flag = false;
  }
  return flag;
}

Comments

[edoooooo]

if文の中だけでなくメソッドでの使用法もあるのですね
条件が多くなったり、再利用ができないというデメリットが改善されそうです。
　いろんなコードを書いて勉強しているのですが、このタイプも作ってみたいと思います。
ありがとうございました。

[kaputaros]

if文の中でそのメソッドを呼び出すんですけどね。
共通便利クラスを１つ定義しておくと、何度も同じ処理を書かなくてすみますよ。
staticなのもミソです。

Answer 3

index.jspで

ユーザー名:<input type="text" name="name"><br>
パスワード:<input type="password"name="pass"><br>

このまま入力せずにsubmitした場合、

• nameの値 : 長さ0の文字列≠null
• passの値 : 長さ0の文字列≠null

こうなります。

if文で

loginUser.name.length() > 0 && logoinUser.pass.length() > 0

も条件に追加してみてはどうでしょう。

Comments

[edoooooo]

　ありがとうございます。loginUserはnullでチェックできても、nameやpassはlengthで制限をかけるのですね！
　loginUser.name.length() > 0 && loginUser.pass.length() > 0
このようにしてみたのですが、The field User.name is not
visible(そのフィールドのUser.nameは目に見えない)となり
<%if(loginUser!=null && loginUser.getName().length() > 0 && loginUser.getPass().length() > 0){
%>
と下のコードと同じように、したところ、しっかりとelseのactionをさせることができました。ありがとうございました。

[tkturbo]

おおう、getName()とgetPass()を使うのでしたね。。。

[edoooooo]

少し入り組んだところにNameとPassを置いていたので、
すみません、ありがとうございました。