loginUserがnullかだけでなくその要素のNameとPassのどちらもないとログインできないようにしたいです。

java
1このコードをこの下のコードをその下のコードに変更したところ、NullPointerExceptionが出ます。
2<%if(loginUser!=null) %>
3
4<% if(loginUser.getName()!=null & loginUser.getPass()!=null){ %>
5

jspのコードの全体です。

java
1<%@ page language="java" contentType="text/html; charset=UTF-8"
2    pageEncoding="UTF-8"%>
3    <%@ page import="model.User" %>
4    <%
5    //セッションスコープからユーザー情報を取得
6    User loginUser=(User)session.getAttribute("loginUser");
7    %>
8<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd">
9<html>
10<head>
11<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
12<title>どこつぶログイン</title>
13</head>
14<body>
15<h1>どこ粒ログイン</h1>
16/*
17
18このコードをこの下のコードをその下のコードに変更したところ、NullPointerExceptionが出ます。
19<%if(loginUser!=null) %>
20
21*/
22<% if(loginUser.getName()!=null & loginUser.getPass()!=null){ %>
23
24<p>ログインに成功しました。</p>
25<p>ようこそ<%=loginUser.getName() %>さん</p>
26<a href="/example11/Main">つぶやき投稿.閲覧へ</a>
27<%}else{ %>
28<p>ログインに失敗しましt。</p>
29<a href="/example11/">TOPへ</a>
30<%} %>
31</body>
32</html>

行動規範の内容に同意します

回答2件

java
1//セッションスコープからユーザー情報を取得
2User loginUser=(User)session.getAttribute("loginUser");

この時点でまだセッションに「loginUser」がセットされていなければ「loginUser == null」になってしまいます。
なので、

java
1if(loginUser.getName()!=null & loginUser.getPass()!=null){

この先頭部分は

java
1if(null.getName()!=null

とやってるのと同じことになるのでNullPointerExceptionが出るのです。
ちゃんと判定するなら

java
1if (loginUser != null && loginUser.getName() != null && loginUser.getPass() != null) {

のようにしないとまずいでしょ。

投稿2016/04/15 01:43

tkturbo

総合スコア5572

edoooooo

2016/04/15 04:15

ありがとうございます。jspで名前も、パスワードも入力しなかった時に限って、エラーになっていました。 if文では、抜け道のないように()内を記述したいと思います。

行動規範の内容に同意します

ベストアンサー

loginUserのnullチェックが消えたため、loginUserがnullの場合に落ちます。
loginUserのnullチェックも別途行う必要があります。

Java
1// loginUserのnullチェックと中身のnullチェックを同時に
2if(loginUser != null && loginUser.getName()!= null && loginUser.getPass()!= null){
3    // nullでなければ処理
4}

または、

Java
1// 先にloginUserのnullチェック
2if(loginUser != null){
3    // loginUserがnullでなければ、中身のnullチェック
4    if(loginUser.getName()!=null && loginUser.getPass()!=null){
5        // 中身もnullでなければ処理
6    }
7}