SQL処理の例外発生時のスタックトレースが表示されない原因を教えてください。

前提・実現したいこと

UPDATE処理でエラーが発生しており、
発生しているエラーの原因を知りたく、スタックトレースを表示させたいです。

使用言語はJavaで、
Servlet＆jspでプログラムを書いています。

原因がわかりません。お力添えいただけますでしょうか。

発生している問題

SQLExceptionを上の階層に伝播しているのですが、
エラーが起きた際に、スタックトレースではなく、まっさらな画面しか返ってきません。

考えられる原因

・SQLExceotionをservletクラスまで届けられていない

・INSERT・UPDATE処理で発生している例外がSQLEXceptionに当てはまらない？

該当のソースコード

jsp
1<%@ page language="java" contentType="text/html; charset=UTF-8"
2    pageEncoding="UTF-8"%>
3<%@ page import="model.AddressBookInfo"%>
4<%
5AddressBookInfo addressInfo = (AddressBookInfo)request.getAttribute("addressInfo");
6%>
7<!DOCTYPE html>
8<html>
9<head>
10<meta charset="UTF-8">
11<title>Insert title here</title>
12</head>
13<body>
14
15	<form action="/FileUploadApp/AddressEditServlet" method="POST">
16		<br>
17		ID　：<input type="text" name="id" value=<%= addressInfo.getId() %> readonly><br>
18		名　前　：<input type="text" name="name" value=<%= addressInfo.getName() %> required><br>
19		住　所　：<input type="text" name="addr" value=<%= addressInfo.getAddr() %> required><br>
20		電話番号：<input type="text" name="telNum" value=<%= addressInfo.getTelNum() %> required><br>
21		<input type="submit" value="update">
22	</form>
23	
24	<a href="/FileUploadApp/CsvFileServlet">戻る</a>
25	
26</body>
27</html>

Java
1import java.io.IOException;
2import java.sql.SQLException;
3
4import javax.servlet.RequestDispatcher;
5import javax.servlet.ServletException;
6import javax.servlet.annotation.WebServlet;
7import javax.servlet.http.HttpServlet;
8import javax.servlet.http.HttpServletRequest;
9import javax.servlet.http.HttpServletResponse;
10
11import logic.CsvEditLogic;
12import logic.CsvSearchLogic;
13import model.AddressBookInfo;
14import model.AddressBookInfoCondDto;
15
16/**
17 * 
18 * 住所録のデータを編集するServletクラス
19 * @author 
20 *
21 */
22@WebServlet("/AddressEditServlet")
23public class AddressEditServlet extends HttpServlet {
24	private static final long serialVersionUID = 1L;
25
26	/**
27	 * データベースに編集を登録
28	 */
29	@Override
30	protected void doPost(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
31
32		try {
33
34			// 文字コード指定
35			response.setContentType("text/html; charset=Shift_JIS");
36
37			// プライマリーキー情報を取得
38			String pk = request.getParameter("id");
39			
40			// 受け取ったIDをInt型に変換
41			int id = Integer.parseInt(pk);
42
43			// formデータを格納した変数？の作成
44			AddressBookInfoCondDto condDto = convertFormToCondDto(request);
45
46			// EditAddressInfoLogicのインスタンス化
47			CsvEditLogic csvEditLogic = new CsvEditLogic();
48
49			// UPDATEをかけるメソッドの呼び出し(引数にformから受け取ったデータ)
50			csvEditLogic.edit(id, condDto);
51
52			// ファイル選択画面にリダイレクト
53			response.sendRedirect("/FileUploadApp/CsvFileServlet");
54
55		} catch (IOException e) {
56			e.printStackTrace();
57		} catch (SQLException e) {
58			e.printStackTrace();
59		} catch (InstantiationException e) {
60			e.printStackTrace();
61		} catch (IllegalAccessException e) {
62			e.printStackTrace();
63		} catch (ClassNotFoundException e) {
64			e.printStackTrace();
65		}
66
67	}
68
69	/**
70	 * formデータパラメータの詰め替え
71	 */
72	private AddressBookInfoCondDto convertFormToCondDto(HttpServletRequest request) {
73
74		// パラメータの詰替変数？の作成
75		AddressBookInfoCondDto condDto = new AddressBookInfoCondDto();
76
77		// パラメータにformデータを詰める
78		condDto.setName(request.getParameter("name"));
79		condDto.setAddr(request.getParameter("addr"));
80		condDto.setTelNum(request.getParameter("telNum"));
81
82		return condDto;
83	}
84
85}
86

Java
1package logic;
2
3import java.sql.SQLException;
4
5import dao.CsvFileDao;
6import model.AddressBookInfoCondDto;
7
8/**
9 * 
10 * 住所録のデータを編集するLogicクラス
11 * @author 
12 *
13 */
14public class CsvEditLogic {
15
16	/**
17	 * 住所録のデータを更新
18	 * @throws InstantiationException 
19	 * @throws IllegalAccessException 
20	 */
21	public void edit(int id, AddressBookInfoCondDto condDto) throws SQLException, InstantiationException, IllegalAccessException, ClassNotFoundException {
22		
23		try {
24			
25			// UploadedCsvFileDaoのインスタンス化
26			CsvFileDao csvFile = new CsvFileDao();
27			
28			// UPDATEをかけるメソッドの呼び出し
29			csvFile.update(id, condDto);
30			
31		} catch (SQLException e) {
32			throw new SQLException();
33		} catch (InstantiationException e) {
34			throw new InstantiationException();
35		} catch (IllegalAccessException e) {
36			throw new IllegalAccessException();
37		} catch (ClassNotFoundException e) {
38			throw new ClassNotFoundException();
39		}
40
41	}
42
43}
44

Java
1package dao;
2
3import java.sql.Connection;
4import java.sql.DriverManager;
5import java.sql.PreparedStatement;
6import java.sql.ResultSet;
7import java.sql.SQLException;
8import java.util.ArrayList;
9import java.util.List;
10
11import model.AddressBookInfo;
12import model.AddressBookInfoCondDto;
13
14/**
15 * 
16 * DB関連の処理クラス
17 * @author 
18 *
19 */
20public class CsvFileDao {
21
22	// DB接続に使用する情報(Enumに変更)
23	private final String url = "jdbc:mariadb://localhost/upload_db";
24	private final String user = "****";
25	private final String password = "*****";
26
27	/**
28	 * レコードをUPDATE
29	 */
30	public void update(int id, AddressBookInfoCondDto condDto) throws SQLException, InstantiationException, IllegalAccessException, ClassNotFoundException {
31
32		Connection conn = null;
33		PreparedStatement ps = null;
34
35		try {
36
37			// JDBCドライバをロード
38			Class.forName("org.mariadb.jdbc.Driver").newInstance();
39
40			// DB接続
41			conn = DriverManager.getConnection(url, user, password);
42
43			// UPDATE文の準備
44			String updateSql = "UPDATE `address_book` SET `name` = '?', `addr` = '?', `telNum` = '?' WHERE (`id` = ?);"; 
45			ps = conn.prepareStatement(updateSql);
46
47			// INSERT実行
48			ps.setString(1, condDto.getName());
49			ps.setString(2, condDto.getAddr());
50			ps.setString(3, condDto.getTelNum());
51			ps.setInt(4, id);
52			ps.executeUpdate();
53
54			// DB更新と接続の解除
55			conn.commit();
56
57		} catch (SQLException e) {
58			throw new SQLException();
59		} catch (InstantiationException e) {
60			throw new InstantiationException();
61		} catch (IllegalAccessException e) {
62			throw new IllegalAccessException();
63		} catch (ClassNotFoundException e) {
64			throw new ClassNotFoundException();
65		}finally {
66			if (ps != null) { ps.close(); }
67			if (conn != null) { conn.close(); }
68		}
69
70	}
71
72}
73

m.ts10806

2021/02/03 04:35

throwsしてるならそのメソッド内でのcatchする必要ないのでは何を見て書いたのでしょうか。「エラーの原因」は「prepareStatementに渡すSQLの末尾に`;`をつけてるから」でしょうけど。

kjrk

2021/02/03 05:29

きちんと勉強していませんでした。例外処理の勉強の方を参考書を読むなどして体系的に学習します。こちらを、 "UPDATE `address_book` SET `name` = '?', `addr` = '?', `telNum` = '?' WHERE (`id` = ?);"; 下記のように変更しましたが、エラーの解決に至りませんでした。 "UPDATE `address_book` SET `name` = '?', `addr` = '?', `telNum` = '?' WHERE (`id` = ?)";

m.ts10806

2021/02/03 05:42

バインドする変数にクォートつけたらどうなると思いますか？

kjrk

2021/02/03 05:45

?こちらをシングルクォート付けてみましたが、動作の方確認できていません。

m.ts10806

2021/02/03 05:51

いや、もともとついてるじゃないですか。 >SET `name` = '?', `addr` = '?', `telNum` = '?' なんのためにパラメータにして型を指定してバインドするのかを考えたら明白です。おまじないではありません。全てのことには意味がありますし、意味がわからないまま使っていては一生自分のものにはなりません。「なぜこう書くのだろう」興味わきませんか？

m.ts10806

2021/02/03 05:52

「体系的な学習」をしたいのでしたら、やることは全体の流れや繋がりを掴むことからです

kjrk

2021/02/03 05:56

かしこまりました。ご丁寧な指摘、ありがとうございます。全てのものに意味があることをきちんと考えながら書くように心がけます。

m.ts10806

2021/02/03 06:16

過去質問いくつか見た感じ、とりあえずとりあえずなんとなくなんとなくでやってる感がスゴイので、早めにやり方改めたほうがいいですよ。質問せずに解決できたらそれが一番早いわけですから。他者に頼ると分かったつもりに陥りますし、自分以外の時間も使うことになるわけですから。厚意の場とは言え、「前も同じこと言われてなかったっけ？」はマズイです。

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

回答1件

ベストアンサー

Java
1        } catch (SQLException e) {
2            throw new SQLException();

オリジナルのExceptionを握り潰して、(メッセージの入っていない)新たなインスタンスを生成して上位階層に渡してるからです。

そもそもm.ts10806さんが指摘されているように、上位階層に伝播させるだけならcatchする必要はありません。

投稿2021/02/03 05:34

javahack

総合スコア1088

kjrk

2021/02/03 05:38

ありがとうございます。理解を深めるために体系的に学習もしてみます。

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

あなたの回答

tips

プレビュー

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

質問の解決につながる回答をしましょう。サンプルコードなど、より具体的な説明があると質問者の理解の助けになります。また、読む側のことを考えた、分かりやすい文章を心がけましょう。

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

ただいまの回答率
85.36%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

『🎄teratailクリスマスプレゼントキャンペーン2024🎄』開催中！

＼teratail特別グッズやAmazonギフトカード最大2,000円分が当たる！／

SQL処理の例外発生時のスタックトレースが表示されない原因を教えてください。

前提・実現したいこと

発生している問題

考えられる原因

該当のソースコード

関連した質問