eclipseで「java.lang.NullPointerException」の解決方法

Question

前提・実現したいこと

eclipseを使ってIDや氏名、メールアドレス等を新規登録し、IDに基づいて各々の行動記録を表記、閲覧できるサイトを作ろうとしています。
そこで、アカウントを新規登録する際に「java.lang.NullPointerException」が出てきて作業が進まなくなってしまったので、解決方法をご教授願いたいです。

個人的には、恐らく後述している「InputChecker.java」の文字数チェックをするcheckLongInputメソッド内のdata関数がnullなのでエラーが出ている気がするのですが、かと言ってIdやnameの文字数チェックをするためにこのdata関数をどう記述し直したらいいのかもわからず……。（もし見当違いなことをいっていたらごめんなさい）

初心者なのでこの処理だけで5時間以上かかってて自分ではもう解決方法がわかりません。
有識者の方々、お力添えください…！

発生している問題・エラーメッセージ

HTTPステータス 500 – Internal Server Error
タイプ 例外報告

説明 サーバーは予期しない条件に遭遇しました。それはリクエストの実行を妨げます。

例外

java.lang.NullPointerException
	model.InputChecker.checkLongInput(InputChecker.java:8)
	servlet.AddUser.doPost(AddUser.java:90)
	javax.servlet.http.HttpServlet.service(HttpServlet.java:660)
	javax.servlet.http.HttpServlet.service(HttpServlet.java:741)
	org.apache.tomcat.websocket.server.WsFilter.doFilter(WsFilter.java:53)
注意 原因のすべてのスタックトレースは、サーバのログに記録されています

Apache Tomcat/9.0.34

該当のソースコード

(ファイル名：InputChecker.java)
package model;

import java.util.regex.*;

public class InputChecker {
	public static String checkLongInput(String data) throws InputCheckException{
		//256文字以下であればOK
		if (data.length() <= 256) {
			return data;
		}
		//ダメなので、例外をスロー
		throw new InputCheckException();
	}
	
	public static String checkPhoneNumber(String tel) throws InputCheckException{
		//メールアドレスの正規表現
		String regularExpression = "^[0-9]{2,4}-[0-9]{2,4}-[0-9]{3,4}$";
		Pattern pattern = Pattern.compile(regularExpression);
		Matcher matcher = pattern.matcher(tel);
		//パターンがマッチしたら引数をそのまま返す。
		if (matcher.find()) {
			return tel;
		}
		//マッチしなかったので、例外をスロー
		throw new InputCheckException();
	}
	
	public static String checkMailAddress(String mail) throws InputCheckException{
		//メールアドレスの正規表現
		String regularExpression 
				= "^[a-zA-Z0-9.!#$%&'*+/=?^_`{|}~-]+@[a-zA-Z0-9-]+(?:\.[a-zA-Z0-9-]+)*$";
		Pattern pattern = Pattern.compile(regularExpression);
		Matcher matcher = pattern.matcher(mail);
		//パターンがマッチしたら引数をそのまま返す。
		if (matcher.find()) {
			return mail;
		}
		//マッチしなかったので、例外をスロー
		throw new InputCheckException();
	}
}

(ファイル名：Adduser.java)

～中略～
protected void doPost(HttpServletRequest req, HttpServletResponse resp) throws ServletException, IOException {
		req.setCharacterEncoding("UTF-8");
		HttpSession session = req.getSession();

		// フォームから送られた確認キーが保存したものと一致するか確認
		ValidationKey validationKey = (ValidationKey) session.getAttribute("validationKey");
		if (!req.getParameter("vKey").equals(validationKey.getValue())) {
			// 一致しなかったので、セッションスコープに保存したキーを破棄し、エラーページに
			session.removeAttribute("validationKey");
			//表示データを用意する
			ErrorViewData errorData = new ErrorViewData("問題が発生しました。",
													"トップに戻る","/ActionLogger/");
			req.setAttribute("errorData", errorData);
			//エラー表示にフォワード
			RequestDispatcher dispatcher = req.getRequestDispatcher("/WEB-INF/jsp/errorView.jsp");
			dispatcher.forward(req, resp);
			return;
		}

		User user = new User();
		try {
//			if(req.getParameter("Id") == null) {
//				System.out.println ("nullです");
//			}
			user.setId( checkLongInput(req.getParameter("Id")) );
			user.setName(checkLongInput(req.getParameter("name")) );
			user.setAddress( checkLongInput(req.getParameter("address")) );
			user.setTel( checkLongInput(req.getParameter("tel")) );
			user.setMail( checkLongInput(req.getParameter("mail")) );

			// パスワードのハッシュ化
			String rawPassword = req.getParameter("password");
			MessageDigest digest = MessageDigest.getInstance("SHA-256");
			digest.reset();
			digest.update(rawPassword.getBytes("utf8"));
			String passwordHash = String.format("%064x", new BigInteger(1, digest.digest()));

			user.setPassword(passwordHash); // ハッシュ値をオブジェクトに設定

			// userオブジェクトをセッションスコープに一旦保存（DBに入れるのはConfirmの後）
			session.setAttribute("userToAdd", user);

			// 確認画面にリダイレクト
			RequestDispatcher dispatcher = req.getRequestDispatcher("/WEB-INF/jsp/addUserConfirm.jsp");
			dispatcher.forward(req, resp);

		} catch (NoSuchAlgorithmException e) {
			e.printStackTrace();
		} catch (UnsupportedEncodingException e) {
			e.printStackTrace();
		} catch (InputCheckException e) {
			//表示データを用意する
			ErrorViewData errorData = new ErrorViewData("フォームに入力された内容に問題がありました。",
													"入力画面に戻る","/ActionLogger/adduser");
			req.setAttribute("errorData", errorData);
			//エラー表示にフォワード
			RequestDispatcher dispatcher = req.getRequestDispatcher("/WEB-INF/jsp/errorView.jsp");
			dispatcher.forward(req, resp);
		}
	}

補足情報（FW/ツールのバージョンなど）

エラーメッセージが出ている行で該当するファイルの行です。

・model.InputChecker.checkLongInput(InputChecker.java:8)
→if (data.length() <= 256) {

・servlet.AddUser.doPost(AddUser.java:90)
→user.setId( checkLongInput(req.getParameter("Id")) );

Comments

[m.ts10806]

行数が狂うのでコード以外の情報はマークダウンのcode内に含まないようにしてください。
（あと変に省略しすぎないでほしい）

[dodox86]

> req.getParameter("Id"))
フォームのhtml(jsp?)の提示がないので推測ですが、リクエストのフォームのパラメータは、頭が小文字で、"id"なのではないですか。他のものは"name"とか"address"になっています。で、y_waiwaiさんの回答に繋がります。

[Ren-IT]

dodox86様の言っていた通り、"Id"と"id"を間違えていたみたいです…。そこを直すとエラーがでなくなりました。完全に自分の凡ミスです…ご指摘いただきありがとうございました…！

今回、teratailで質問をするのが初めてだったので、コードの書き方等に不備があってごめんなさい。
以後は気を付けます…本当にありがとうございました！

Answer 1

NullPointerException

nullの変数をどーこーしようとしたときに出る例外です

 if (data.length() <= 256) {

dataがnullになってるんじゃないでしょうか。
そこらへんチェックしましょう