日付変換でNullPointerExceptionが発生する

Question

前提・実現したいこと

社員管理システムを学習の一環で作っています。
社員情報の更新を行う際にエラーが発生しております。

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

社員情報の更新ページでは「退職日」という項目が存在します。
これは必要に応じて入力するものになりますが、入力せずに更新を行うと
以下NullPointerExceptionのエラーが発生します。

java.lang.NullPointerException
	at java.text.SimpleDateFormat.parse(SimpleDateFormat.java:1439)
	at java.text.DateFormat.parse(DateFormat.java:364)
	at convert.Convert.formatDateSlash(Convert.java:35)
	at dao.EmployeeInfoDao.findDetail(EmployeeInfoDao.java:144)
	at controller.DetailController.doPost(DetailController.java:134)
	at javax.servlet.http.HttpServlet.service(HttpServlet.java:650)
	at javax.servlet.http.HttpServlet.service(HttpServlet.java:731)
	at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:303)
	at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:208)
	at org.apache.tomcat.websocket.server.WsFilter.doFilter(WsFilter.java:52)
	at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:241)
	at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:208)
	at org.apache.catalina.core.StandardWrapperValve.invoke(StandardWrapperValve.java:219)
	at org.apache.catalina.core.StandardContextValve.invoke(StandardContextValve.java:110)
	at org.apache.catalina.core.StandardHostValve.invoke(StandardHostValve.java:169)
	at org.apache.catalina.valves.ErrorReportValve.invoke(ErrorReportValve.java:103)
	at org.apache.catalina.valves.AccessLogValve.invoke(AccessLogValve.java:962)
	at org.apache.catalina.core.StandardEngineValve.invoke(StandardEngineValve.java:116)
	at org.apache.catalina.connector.CoyoteAdapter.service(CoyoteAdapter.java:445)
	at org.apache.coyote.http11.AbstractHttp11Processor.process(AbstractHttp11Processor.java:1115)
	at org.apache.coyote.AbstractProtocol$AbstractConnectionHandler.process(AbstractProtocol.java:637)
	at org.apache.tomcat.util.net.JIoEndpoint$SocketProcessor.run(JIoEndpoint.java:318)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
	at org.apache.tomcat.util.threads.TaskThread$WrappingRunnable.run(TaskThread.java:61)
	at java.lang.Thread.run(Thread.java:748)

該当のソースコード

public String formatDateSlash(String formatDate) {

		Constants strConst = new Constants();

		try {
			//フォーマットパターンを'yyyy-MM-dd'に指定します。
			SimpleDateFormat format1 = new SimpleDateFormat(strConst.AD_DATE_DASH, Locale.ENGLISH);
			Date date = format1.parse(formatDate);
			//フォーマットパターンを'yyyy/MM/dd'に変更します。
			SimpleDateFormat format2 = new SimpleDateFormat(strConst.AD_DATE_SLASH);
			formatDate = format2.format(date);

			return formatDate;

		} catch (ParseException e) {

			e.printStackTrace();
			return null;

		} catch (NullPointerException e) {

			e.printStackTrace();
			return null;

		} finally {

		}
	}

試したこと

try-catchでNullPointerExceptionを例外処理するようにしましたが、
変わらずエラーが発生します。

どうかご教授いただければ幸いです。

###追記
呼び出し元のソースコードです。

public EmployeeInfoDto findDetail(String employee_id) {

		EmployeeInfoDto employeeInfoDto = null;
		Connection conn = null;
		PreparedStatement pStmt = null;
		ResultSet rs = null;
		int employeeId;

		try {

			employeeId = Integer.parseInt(employee_id);

			Convert cvrt = new Convert();
			SqlConstants sqlConst = new SqlConstants();
			Constants strConst = new Constants();

			Class.forName(sqlConst.JDBC_DRIVER);
			conn = DriverManager.getConnection(sqlConst.DB_CONNECT_URI, sqlConst.USER_NAME, sqlConst.PASSWORD);
			pStmt = conn.prepareStatement(sqlConst.SEARCH_EMPLOYEE_DETAIL);
			pStmt.setInt(1, employeeId);
			rs = pStmt.executeQuery();

			while (rs.next()) {

				employeeInfoDto = new EmployeeInfoDto();
				employeeInfoDto.setEmployeeId(rs.getString(strConst.EMPLOYEE_ID));
				employeeInfoDto.setName(rs.getString(strConst.NAME));
				employeeInfoDto.setNameHiragana(rs.getString(strConst.NAME_HIRAGANA));
				employeeInfoDto.setBirthday(cvrt.formatDateSlash(rs.getString(strConst.BIRTHDAY)));
				employeeInfoDto.setSex(rs.getString(strConst.SEX));
				employeeInfoDto.setMailAddress(rs.getString(strConst.MAIL_ADDRESS));
				employeeInfoDto.setTelephoneNumber(cvrt.addDash(rs.getString(strConst.TELEPHONE_NUMBER)));
				employeeInfoDto.setCompanyInfoId(rs.getString(strConst.COMPANY_INFO_ID));
				employeeInfoDto.setBusinessManager(rs.getString(strConst.BUSINESS_MANAGER));
				employeeInfoDto.setDepartment(rs.getString(strConst.DEPARTMENT));
				employeeInfoDto.setCommissioningStatus(rs.getString(strConst.COMMISSIONING_STATUS));
				employeeInfoDto.setEnterDate(cvrt.formatDateSlash(rs.getString(strConst.ENTER_DATE)));
				employeeInfoDto.setRetireDate(cvrt.formatDateSlash(rs.getString(strConst.RETIRE_DATE)));
				employeeInfoDto.setStatus(rs.getString(strConst.STATUS));

				return employeeInfoDto;
			}

		} catch (SQLException e) {
			e.printStackTrace();

			return null;

		} catch (ClassNotFoundException e) {
			e.printStackTrace();

			return null;

		} catch (NumberFormatException e) {
			e.printStackTrace();

			return null;

		} finally {

			if (conn != null) {

				try {

					conn.close();

				} catch (SQLException e) {
					e.printStackTrace();

					return null;

				}
			}
		}
		return employeeInfoDto;
	}

Comments

[sazi]

formatDateSlashの呼び出し元はNullを受けても大丈夫な作りですか？
もしくは、NullだったらformatDateSlashを利用しないようになっていますか？

[nyan_engineer]

呼び出し元のソースコードを追記しました。
Nullである場合にformatDateSlashを利用しない作りにはまだできておりません。。

Answer 1

普通にnullチェックすればいい話なのでは？
というか、例外に頼らず本来はそうすべき。

public String formatDateSlash(String formatDate) {
    if(formatDate == null) return null;
    // 以下同じ、ただし、NullPointerの例外処理は不要

Comments

[LouiS0616]

代入式になっちゃってます。

[swordone]

おうふ。ご指摘ありがとうございます。

[nyan_engineer]

回答ありがとうございます。
例外処理をする必要は無いんですね。。
どの場面で例外処理を行うのかなど、まだまだ勉強不足でした、、

[maisumakun]

NullPointerExceptionをcatchすべき場面は稀（通常の場合はコードでnullチェックすべき）かと思います。

https://www.jpcert.or.jp/java-rules/err08-j.html

[nyan_engineer]

参考になります、、！ありがとうございます。

Answer 2

e.printStackTrace();でエラーの内容が表示されているだけでは。
処理がそこで止まってしまうのであれば問題ですが・・・

Comments

[nyan_engineer]

処理は問題なくおこなわれます。
エラー内容が表示されていても特に問題はないですか？
毎回エラーメッセージが表示されるので、、

[tetutetu]

横からですみません。
e.printStackTrace();
はエラーをコンソールに出力するための記述なので
コンソールにエラーが出力されるのは当然です。
不要なのであれば、NullPointerExceptionをキャッチした際の処理を
//do nothing（コメントアウト。何もしない　の意）等記述しておくとよいと思います
何も記述していない場合、第三者がコードを見た際に（例外処理忘れてるのかな？）とあらぬ誤解を生む原因になります。

追記：同じくfinallyの中身もコメントアウトで何もしない旨を書いた方が適切だとは思われます
理由は上に同じです。

[LouiS0616]

@tetutetu さん
安易に握り潰しを勧めるのはまずいでしょう。相手が例外に不慣れなようならば尚更。
そもそもNullPointerExceptionはキャッチして捌くべき類のものでは無いですし...

[nyan_engineer]

勉強不足で申し訳ありません。。
そもそも、入力値がnullである場合にこのメソッドを使わないようにするべきでしょうか？

[退会済みユーザー]

使うべきかどうかというより、formatDateSlash() にnullが突っ込まれてもいいようにしておくのがいいと思います。

[nyan_engineer]

yukke_さん
ありがとうございます。別回答のように、例外処理ではなくnullチェックにて対応いたします。

[swordone]

finallyで何もしないなら、そもそもfinallyブロック自体いらないのだが…