Servetエラー DBの取得結果が画面に表示されない

Question

ここに質問の内容を詳しく書いてください。

初心者です。
Todoリストの一覧をDBから取得して画面に出力するという事を行っています。
home.html画面は表示でき、Todo一覧表示ボタンを押し画面遷移の際にエラーになってしまいます。
DBでselect文できちんとデータが登録されているのは確認済みです。
どのようにすればこのエラーを解決できるのでしょうか？
よろしくお願いいたします。

実現したいこと

Todoリストの一覧をDBから取得して画面に出力する

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

エラーメッセージ
Cannot invoke "java.sql.Connection.prepareStatement(String)" because "connection" is null
サーバーは予期しない条件に遭遇しました。それはリクエストの実行を妨げます。
例外
java.lang.NullPointerException: Cannot invoke "java.sql.Connection.prepareStatement(String)" because "connection" is null
model.dao.TodoListDAO.getTodoList(TodoListDAO.java:36)
servlet.ListServlet.doGet(ListServlet.java:44)
javax.servlet.http.HttpServlet.service(HttpServlet.java:655)
javax.servlet.http.HttpServlet.service(HttpServlet.java:764)
org.apache.tomcat.websocket.server.WsFilter.doFilter(WsFilter.java:53)

該当のソースコード

ソースコード
package model;

import java.sql.Connection;
import java.sql.DriverManager;
import java.sql.SQLException;

public class DBConnection {

	public static Connection getConnection() throws SQLException, ClassNotFoundException {
                // データベースのURL
		final String URL = "jdbc:postgresql://localhost:5432/Tododb";
                // データベースにアクセスするユーザー
		final String USER = "postgres";
                // パスワード
		final String PASSWORD = "postgres";
		Class.forName("org.postgresql.Driver");
                // データベースへ接続する
		Connection connection = DriverManager.getConnection(URL, USER, PASSWORD);
		
		return connection;
	}
}

package model.dao.dto;

import java.sql.Date;

public class TodoDTO {
	/**
	 * TodoのIdを保持する
	 */
	private int id;

	/**
	 * Todoの内容を保持する
	 */
	private String todo;

	/**
	 * Todoの期限を保持する
	 */
	private Date timeLimit;

	/**
	 * 引数なしコンストラクタ
	 * 初期化を行う
	 */
	public TodoDTO () {
		id = 0;
		todo = "";
		timeLimit = new Date(System.currentTimeMillis());
	}

	/**
	 * 受け取った引数によって初期化を行う
	 * @param id
	 * @param todo
	 * @param timeLimit
	 */
	public TodoDTO(int id, String todo, Date timeLimit) {
		this.id = id;
		this.todo = todo;
		this.timeLimit = timeLimit;
	}

	/**
	 * idを取得する
	 * @return id
	 */
	public int getId() {
		return id;
	}

	/**
	 * idを設定する
	 * @param id
	 */
	public void setId(int id) {
		this.id = id;
	}

	/**
	 * Todoの内容を取得する
	 * @return Todoの内容
	 */
	public String getTodo() {
		return todo;
	}

	/**
	 * Todoの内容を設定する
	 * @param todo
	 */
	public void setTodo(String todo) {
		this.todo = todo;
	}

	/**
	 * Todoの期限を取得する
	 * @return
	 */
	public Date getTimeLimit() {
		return timeLimit;
	}

	/**
	 * Todoの期限を設定する
	 * @param timeLimit
	 */
	public void setTimeLimit(Date timeLimit) {
		this.timeLimit = timeLimit;
	}

}

package model.dao;

import java.sql.Connection;
import java.sql.Date;
import java.sql.PreparedStatement;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.util.ArrayList;
import java.util.List;

import model.DBConnection;
import model.dao.dto.TodoDTO;

public class TodoListDAO {

	/**
	 * Todoの一覧を取得する
	 * @return todoの一覧
	 * @throws ClassNotFoundException
	 * @throws SQLException
	 */
	public List<TodoDTO> getTodoList() throws ClassNotFoundException, SQLException {
		// 返却用Listの初期化
		List<TodoDTO> todoList = new ArrayList<>();

		// 実行するSQL
		String sql = "SELECT id, todo, timeLimit from todo";

		// DBに接続し、Todo一覧を取得する
		try (Connection connection = DBConnection.getConnection();
				PreparedStatement pstmt = connection.prepareStatement(sql)) {

			// SQLを実行しResultSetの変数に結果を格納する
			ResultSet res = pstmt.executeQuery();
			// 実行結果を順番に取り出す
			while (res.next()) {
			        //  それぞれid, todo, timeLimitを取得する
				int id = res.getInt("id");
				String todo = res.getString("todo");
				Date timeLimit = res.getDate("timeLimit");

				// 取得したid, todo, timeLimitでTodoDTOを初期化してListに追加
				todoList.add(new TodoDTO(id, todo, timeLimit));
			}
		}

		return todoList;
	}
}

package servlet;

import java.io.IOException;
import java.sql.SQLException;
import java.util.ArrayList;
import java.util.List;

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 model.dao.TodoListDAO;
import model.dao.dto.TodoDTO;

/**
 * Servlet implementation class ListServlet
 */
@WebServlet("/list-servlet")
public class ListServlet extends HttpServlet {
	private static final long serialVersionUID = 1L;
       
    /**
     * @see HttpServlet#HttpServlet()
     */
    public ListServlet() {
        super();
        // TODO Auto-generated constructor stub
    }

	/**
	 * @see HttpServlet#doGet(HttpServletRequest request, HttpServletResponse response)
	 */
	protected void doGet(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
		// todoの一覧を保持する変数を宣言
				List<TodoDTO> todoList = new ArrayList<>();

				// DAOを生成し、Todo一覧を取得する
				TodoListDAO dao = new TodoListDAO();
				try {
					// todo一覧を取得する
					todoList = dao.getTodoList();
					System.out.println(todoList);
				}catch(SQLException | ClassNotFoundException e) {
					e.printStackTrace();
				}

				// todo一覧をリクエストスコープに設定する
				request.setAttribute("todoList", todoList);
				// todo一覧画面に遷移する
				RequestDispatcher rd = request.getRequestDispatcher("list.jsp");
				rd.forward(request, response);
	}

	/**
	 * @see HttpServlet#doPost(HttpServletRequest request, HttpServletResponse response)
	 */
	protected void doPost(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
		// TODO Auto-generated method stub
		doGet(request, response);
	}

}

<!DOCTYPE html>
<html>
<head>
<meta charset="UTF-8">
<title>Home</title>
</head>
<body>
	<h1>Todo 管理Webアプリケーション</h1>
	<form action="list-servlet" method="get">
		<input type="submit" value="Todo一覧を表示">
	</form>

</body>
</html>

<%@ page language="java" contentType="text/html; charset=UTF-8"
    pageEncoding="UTF-8" import="java.util.List, java.util.ArrayList, model.dao.dto.TodoDTO"%>
<!DOCTYPE html>
<html>
<head>
<meta charset="UTF-8">
<title>Todo List</title>
</head>
<body>
	<%
		List<TodoDTO> todoList = (List) request.getAttribute("todoList");
	%>

	<% for(TodoDTO todo: todoList){  %>
		<%=todo.getTodo() %> 期限：<%=todo.getTimeLimit() %><br>
	<% } %>


</body>
</html>

### 試したこと
インターネットで調べる
\pleiades\workspace\Todo\src\main\webapp\WEB-INF\libフォルダにjdbcドライバを格納
ここに問題に対して試したことを記載してください。

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

ここにより詳細な情報を記載してください。

Answer 1

getConnection()内でcloseしてしまってますから、接続成功した場合、finallyで必ずcloseされます。
closeはその接続を使い終わってからです。

Comments

[退会済みユーザー]

ご指摘ありがとうございます。
connectionにnullが渡されてしまっているのかと考えて、
一旦DBConnectionクラスの例外処理を消しました。（参考サイトがこの形式だったので）
結果、エラーはなくなったのですが、画面遷移時に何も表示されない空白になってしまいます。
http://localhost:8080/Todo/list-servlet?というURLになります。
上記の全てのソースコードは参考サイトを模写しています。
何か原因は分かりますでしょうか？
また、closeが使い終わってからならばListServletクラスにfinallyをつけるということでしょうか？
よろしくお願いいたします。

[m.ts10806]

参考サイトの提示を。
あと先に指摘すればよかったですが、コードブロック（マークダウンのcode）はファイル毎にしてください。
かなり見づらいのと、閉める場所が悪いのか、以降のコードと関係ない項目まで含まれています。

>また、closeが使い終わってからならばListServletクラスにfinallyをつけるということでしょうか？

connection を取得しているtry-catchのfinallyでconnection.close()を実行すれば良いと思います。
今のコードからすると、TodoListDAO.getTodoList()内ですね。

＞画面遷移時に何も表示されない空白

これはリクエストの結果何も出力してないか、
続行不可能なエラーが起きてそこで処理が終了しているかのどちらかだと思います。
エラー出てませんか？