java.lang.NullPointerException エラー

Question

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

package servlet;

import java.io.IOException;

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 {

		// フォワード先


	    // 「ユーザー登録」をリクエストされたときの処理
	   // if (action == null) {
	      // フォワード先を設定
	    String forwardPath = "/WEB-INF/jsp/registerForm.jsp";
	    //}

	    // 設定されたフォワード先にフォワード
	    RequestDispatcher dispatcher =
	        request.getRequestDispatcher(forwardPath);
	    dispatcher.forward(request, response);
	  }

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

        // リクエストパラメーターの取得
        request.setCharacterEncoding("UTF-8");

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

        // 登録するユーザー情報を生成
        User registerUser = new User(nickName, eMail, pass, userName);


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

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

	 


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


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

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

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

	      // 登録後のリダイレクト
			response.sendRedirect("WEB-INF/jsp/registerDone.jsp");

	    }else{

        // フォワード
        RequestDispatcher dispatcher =
            request.getRequestDispatcher
                ("/WEB-INF/jsp/registerConfirm.jsp");
        dispatcher.forward(request, response);
      }
	}
 }

この対処を教えてほしいです。エラーの意味と場所はわかっているのですが、どうすればいいのかわかりません

Comments

[A-pZ]

RegisterUser.java の 50行目で出ているとありますので、記載したソースの50行目はどこであるかを明記しないと、ただしい解答は得られないでしょう。

Answer 1

あーこれわかりにくいやつですね。リクエストスコープを使いましょう。

セッションスコープはマルチスレッドセーフではありません。
同じWebブラウザから別画面を開くと、それら二つの画面ではひとつのセッションを共有します。それぞれの画面からサーバにほぼ同時にアクセスすると、その処理は確率的に「同時並行で実行される」可能性があります。

それらのリクエストを仮に「リクエストA」「リクエストB」と呼ぶことにして、このとき、

1. リクエストAの処理が、session.setAttribute("registerUser", registerUser); を実行する。
2. リクエストAの処理が、session2.getAttribute("registerUser");を実行して登録ユーザを取り出そうとする。
3. 2.が実行される直前に、同時並行で動いているリクエストBの処理がsession.removeAttribute("registerUser"); を実行する。
4. 3.が実行された結果、2.の返り値が[nullとなる](http://docs.oracle.com/javaee/5/api/javax/servlet/http/HttpSession.html#getAttribute(java.lang.String)ため、registerUser2 に null が代入される。
5. registerUser2 を使おうとしたとき、NullPointerException が発生する。

……どこでだ?

---

無駄話はさておき、if文はヨーダ記法で記述するとか、[Object.equals()](http://docs.oracle.com/javase/jp/7/api/java/util/Objects.html#equals(java.lang.Object, java.lang.Object)) を使えばいいんじゃないかなと思います。

Comments

[tacsheaven]

50行目だと
> User registerUser = new User(nickName, eMail, pass, userName);
ですね。これで NullPointerException ならば、User のコンストラクタの中ですかね。

[matobaa]

えっえっ
あれれ。Userのコンストラクタの中でNPEが発生してもスタックトレースに出ないんでしたっけ。

[tacsheaven]

もしかするとこれが関係しているかもしれないですね。
http://qiita.com/kei2100/items/d5bbdfe10f7cae30f893

[matobaa]

興味深い。JIT最適化の結果ですかー。そんなことがあるんですね。面白い、勉強になります。

[Yoshi--]

みなさま返答ありがとうございます。色々試してみます。

Answer 2

エラーの意味と場所はわかっているのですが、どうすればいいのかわかりません

きっとわかってはいないので、デバッガでステップ実行するとよいです。

Comments

[Yoshi--]

ありがとうございます

Answer 3

エラーメッセージに、doPost の中（RegisterUser.java の 50行目）とエラーの起こった場所が特定されていますね。

doPost ということはフォームから何かデータを受け取って処理（名前からしてユーザー登録でしょうか）しようとしているので、可能性として一番考えられるのはフォームから受け取るはずのデータが受け取れていないでしょうか。

• フォームでの名前と、RegisterUser の中で見ているPOSTデータの名前とが合っていない
• 空で来ることを想定したガードをしていない
• データベースへアクセスして結果が該当なしだった時を想定していない

あたりですかねえ。

Answer 4

java - HTTPステータス 500 エラーの調査及び解決方法を教えてください - スタック・オーバーフロー

NullPointerException (Java Platform SE 7 )