HTMLからJSPに問題なく遷移はするけどInsertが反映されない

Question

前提・実現したいこと

データベースに名前と年齢を登録する・データベースに登録されているデータを一覧で表示する、というプログラムを練習しています。

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

HTMLからServlet,Daoクラス,getter,setter,jspと特にプログラムにエラーは出ないのですが、テーブルには情報が入っておらず又、一覧も表示されず真っ白な画面が映るばかりです。
何が原因か調べてもいまいちわからず、行き詰っています。
知恵をお貸しいただけないでしょうか。

該当のソースコード

import java.io.IOException;
import java.util.ArrayList;

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 shcooList.ListDAO;
import shcooList.ListGS;

@WebServlet("/ListServlet")
public class ListServlet extends HttpServlet {
	private static final long serialVersionUID = 1L;

	protected void doPost(HttpServletRequest request, HttpServletResponse response)
			throws ServletException, IOException {
		//名前と年齢の値をString型変数に代入
		String name = request.getParameter("name");
		String age = request.getParameter("age");

		//DAOクラスをインスタンス化
		ListDAO listdao = new ListDAO();
		//DAOクラスのsetListメソッドに引数を与え処理させる
		listdao.setList(name, age);

		RequestDispatcher rd = request.getRequestDispatcher("ListJSP.jsp");
		rd.forward(request, response);
	}

	protected void doGet(HttpServletRequest request, HttpServletResponse response)
			throws ServletException, IOException {
		ListDAO listdao = new ListDAO();
		ArrayList<ListGS> list = listdao.getList();
		request.setAttribute("list", list);

		RequestDispatcher rd = request.getRequestDispatcher("ListOut.jsp");
		rd.forward(request, response);

	}

}

package shcooList;

import java.sql.Connection;
import java.sql.DriverManager;
import java.sql.PreparedStatement;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.util.ArrayList;

public class ListDAO {
	//接続設定
	private Connection con = null;
	private PreparedStatement ps = null;
	private ResultSet rs = null;

	//Servletで得た引数をもとに処理するメソッド
	public void setList(String name, String age) {
		try {
			//データベースに接続
			Class.forName("com.mysql.cj.jdbc.Driver");

			String url = "jdbc:mysql://localhost/shcoowork";
			String user = "root";
			String password = "password";

			con = DriverManager.getConnection(url, user, password);

			//SQL文を変数sqlに入れる
			String sql = "INSERT INTO worklist(name,age) VALUES(?,?)";

			//SQL文の解析
			ps = con.prepareStatement(sql);

			//プレースホルダの値を入れる
			ps.setString(1, name);
			ps.setInt(2, Integer.parseInt(age));
			//↑を反映させる
			ps.executeUpdate();

		} catch (ClassNotFoundException e) {
			e.printStackTrace();
		} catch (SQLException e) {
			e.printStackTrace();
		} finally {
			try {
				if (con != null) {
					con.close();
				}
				if (ps != null) {
					ps.close();
				}
				if (rs != null) {
					rs.close();
				}
			} catch (Exception e) {
				e.printStackTrace();
			}

		}
	}

	public ArrayList<ListGS> getList() {

		ArrayList<ListGS> list = new ArrayList<ListGS>();
		try {
			Class.forName("com.mysql.cj.jdbc.Driver");

			String url = "jdbc:mysql://localhost/shcoowork";
			String user = "root";
			String password = "password";

			con = DriverManager.getConnection(url, user, password);

			String sql = "SELECT * FROM worklist";
			ps = con.prepareStatement(sql);
			rs = ps.executeQuery();

			while(rs.next()) {
				String name = rs.getString("name");
				int age = rs.getInt("age");
				ListGS List = new ListGS(name,age);
				list.add(List);
			}

		} catch (ClassNotFoundException e) {
			e.printStackTrace();
		} catch (SQLException e) {
			e.printStackTrace();
		} finally {
			try {
				if (con != null) {
					con.close();
				}
				if (ps != null) {
					ps.close();
				}
				if (rs != null) {
					rs.close();
				}
			} catch (Exception e) {
				e.printStackTrace();
			}
		}
		return list;
	}
}

package shcooList;

public class ListGS {
	//変数を宣言
	private String name;
	private int age;

	//コンストラクタ
	public ListGS(String name,int age) {
		this.name = name;
		this.age = age;
	}

	//対応するゲッターとセッター
	public String getName() {
		return name;
	}
	public void setName(String name) {
		this.name = name;
	}
	public int getAge() {
		return age;
	}
	public void setAge(int age) {
		this.age = age;
	}
}

<!DOCTYPE html>
<html>
<head>
<meta charset="UTF-8">
<title>input</title>
</head>
<body>

<p>新規登録</p>
<form action = "ListServlet" method ="post">
<p>氏名：<input type ="text" name ="name"/></p>
<p>年齢：<input type ="text" name ="age"/></p>
<input type ="submit" value ="登録"/><br>
</form>

<p>登録一覧</p>
<form action = "ListServlet">
<input type ="submit" value ="表示"/>
</form>
</body>
</html>

<%@ page import= "shcooList.ListGS" %>
<%@ page language="java" contentType="text/html; charset=UTF-8"
    pageEncoding="UTF-8"%>
<!DOCTYPE html>
<html>
<head>
<meta charset="UTF-8">
<title>list</title>
</head>
<body>
一件処理しました
</body>
</html>

<%@page import="java.util.ArrayList" %>
<%@page import="shcooList.ListGS" %>

<%@ page language="java" contentType="text/html; charset=UTF-8"
    pageEncoding="UTF-8"%>
<!DOCTYPE html>
<html>
<head>
<meta charset="UTF-8">
<title>Insert title here</title>
</head>
<body>
<%ArrayList<ListGS> list = (ArrayList<ListGS>)request.getAttribute("list"); %>
<%for(int i = 0; i < list.size(); i++){
	ListGS listgs = list.get(i);%>
	<%= (i + 1) %>人目<br>
	<%= listgs.getName() %>
	<%= listgs.getAge() %>
	<%} %>
</body>
</html>

試したこと

データベース側の問題かと思い、drop database、create databaseをしてテーブルまで新しく作り直しています。が変わりませんでした。
動画を見て細かいミスを直してもみましたが変わりません。

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

windows10 64bit
eclipse IDE
tomcat8.5
mySQL
jdk-14.0.1
を使用しています。

Answer 1

ご提示のコードを見る限り、特にAUTOCOMMITが設定されていないので、INSERTやUPDATEが成功した後にはCOMMITする必要があります。更新処理をした後はConnection#commit()を実行しましょう。失敗した場合はConnection#rollback()です。JDBC関連のサンプルコードなどを別途ご参照ください。

...DAOのコード部分を引用（適当に省略）
...
con = DriverManager.getConnection(url, user, password);

//SQL文を変数sqlに入れる
String sql = "INSERT INTO worklist(name,age) VALUES(?,?)";

//SQL文の解析
ps = con.prepareStatement(sql);

//プレースホルダの値を入れる
ps.setString(1, name);
ps.setInt(2, Integer.parseInt(age));
//↑を反映させる
ps.executeUpdate();

// !!!更新(INSERT)できたのでCOMMIT
con.commit();

Comments

[sad]

ご回答ありがとうございます。
con.commit();を入れてみたのですが結果が変わらず、、、
デバッグをしてみたところ（やり方を知らなかった）、Class.forNameからcatchまで飛んでおりそもそも繋がっていないということがわかりました。
しかし登録からなにから調べたとおりにやっているつもりで、原因がわかりません。
お手数ですが、なにか原因と思われるお心当たりはございませんか、、、？
自分に聞くべき質問のような気もしますが、、。

[dodox86]

> Class.forNameからcatchまで飛んでおり
接続できていないと言いますか、そもそもJDBCドライバーが使えていないのだと思います。CLASSPATHに登録されていますか？catch節で例外の内容をe.printStackTrace(); で出力していますよね。何の例外で、どのような出力になっていますか？e.printStackTrace()で情報が多すぎれば、e.getMessage()でもある程度の情報が得られるはずですので確認してください。

[dodox86]

サーブレットコンテナであるtomcatでJDBCドライバーが使えない（＝接続できない）問題については当サイトでも結構質問回答が挙がっているようです。（関連するもので私も回答したことがあります）
「tomcat JDBC」,「サーブレット JDBC」などで検索してみてください。MySQL用であることにご注意ください。

[sad]

新しく作ったプロジェクトでクラスパスを指定していませんでした、、、申し訳ございません。無事繋がりました。
と、今度はexecuteUpdate()とexecuteQuery()でSQLExeptionが発生しています。自身の勉強不足を痛感しております、、何度もご回答、ありがとうございます。

[dodox86]

SQLは合っているはずなのにエラー、そんな場合ありがちなのが、データベースシステムにはログインできているけど、対象としているデータベース(JDBC接続文字列でいう、"shcoowork"の部分)が間違っている、テーブルが無い、ですね。

[dodox86]

Exceptionの内容を良く見れば分かるはずなので確認してみてください。

[sad]

SQL文で指定する場所をinsert into <テーブル名>ではなく
<データベース名>.<テーブル名>とすることでうまくいきました！ご指摘の”テーブルが無い”でそんなはずは、、と閃きました。ありがとうございます！あとは適当に作ったJSPの編集だけなので何とかなりそうです。助かりました(また何かありましたら助言お願いいたします)！

[dodox86]

解決、進展できてよかったです。引き続きがんばってください。