質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.12%

java.lang.NullPointerException エラー

解決済

回答 4

投稿

  • 評価
  • クリップ 0
  • VIEW 3,317

Yoshi--

score 56

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

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);
      }
    }
 }

イメージ説明

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

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • A-pZ

    2017/05/10 11:44

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

    キャンセル

回答 4

checkベストアンサー

+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となるため、registerUser2 に null が代入される。
  5. registerUser2 を使おうとしたとき、NullPointerException が発生する。
    ……どこでだ?

無駄話はさておき、if文はヨーダ記法で記述するとか、Object.equals() を使えばいいんじゃないかなと思います。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/05/10 10:17

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

    キャンセル

  • 2017/05/10 11:39

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

    キャンセル

  • 2017/05/11 09:21

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

    キャンセル

+1

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

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/05/10 09:21

    ありがとうございます

    キャンセル

0

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

0

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

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

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

あたりですかねえ。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.12%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

同じタグがついた質問を見る