スコープに保存した値をJSPで表示したい

Question

QandAシステムを作成中です。サーブレットからスコープに保存した値を、
JSPで表示するとエラーが発生します。
原因をご教授いただけますでしょうか。

package model;

import java.io.Serializable;


public class Question implements Serializable {
 private int question_id;
 private String handle_name;
 private String title;
 private String contents;
 private int urgency;
 private String edit_delete_key;
 private String regist_timestamp;
 private String update_timestamp;

public Question() {}
 public Question(int question_id, String handle_name, String title, String contents, int urgency, String edit_delete_key, String regist_timestamp, String update_timestamp) {
 this.question_id = question_id;
 this.handle_name = handle_name;
 this.title = title;
 this.contents = contents;
 this.urgency = urgency;
 this.edit_delete_key = edit_delete_key;
 this.regist_timestamp = regist_timestamp;
 this.update_timestamp = update_timestamp;
 }

public int getQuestion_id(){return question_id;}
public String getHandle_name(){return handle_name;}
public String getTitle(){return title;}
public String getContents(){return contents;}
public int getUrgency(){return urgency;}
public String getEdit_delete_key(){return edit_delete_key;}
public String getRegist_timestamp(){return regist_timestamp;}
public String getUpdate_timestamp(){return update_timestamp;}
}

package servlet;

import java.io.IOException;
import java.util.List;
//import com.sun.xml.internal.bind.v2.schemagen.xmlschema.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 javax.servlet.http.HttpSession;
import model.GetLogic;
import model.Question;

/**
 * Servlet implementation class Doctest
 */

@WebServlet("/RegistQuestionServlet")
public class ShowQuestionServlet extends HttpServlet {
 private static final long serialVersionUID = 1L;
/**
* @see HttpServlet#doget(HttpServletRequest request, HttpServletResponse response)
*/
protected void doGet(HttpServletRequest request, HttpServletResponse response)
throws ServletException, IOException {

//質問リスト取得
GetLogic getLogic = new GetLogic();	
List<Question> questionList = getLogic.execute();
HttpSession session = request.getSession();
session.setAttribute("questionList", questionList);

//フォワード
RequestDispatcher dispatcher =
 request.getRequestDispatcher("/WEB-INF/jsp/QandAList.jsp");
 dispatcher.forward(request, response);
}
}

package DAO;

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

import model.Question;



public class QuestionDAO {
//  JDBC読み込み
static{
 try{Class.forName("org.postgresql.Driver");
 } catch(ClassNotFoundException e) {
 e.printStackTrace();
 }
}

public List<Question> findAll() {
List<Question> questionList = new ArrayList<>();
// データベースに使用する情報
String jdbc_url ="jdbc:postgresql://localhost:5432/QandA";
String user = "postgres";
String pass = "postgres";
//Timestamp timestamp = new Timestamp(Calendar.getInstance().getTimeInMillis() - 1000*60*60*24);
 // データベース接続
 try(Connection conn = DriverManager.getConnection(jdbc_url, user, pass)){

 //SELECT文の準備
 String sql = "select * from question";
 PreparedStatement pStmt = conn.prepareStatement(sql);

 //SELECT文を実行し結果を取得
ResultSet rs  = pStmt.executeQuery();

//SELECT文の結果を取り出す
while (rs.next()) {
 int question_id = rs.getInt("question_id");
 String handle_name = rs.getString("hanle_name");
 String title = rs.getString("title");
 String contents = rs.getString("contents");
 int urgency = rs.getInt("urgency");
 String edit_delete_key = rs.getString("edit_delete_key");
 String regist_timestamp = rs.getString("regist_timestamp");
 String update_timestamp = rs.getString("update_timestamp");

 // 質問の情報を設定
 Question question = new Question(question_id, handle_name, title, contents, urgency, edit_delete_key, regist_timestamp, update_timestamp);
// List<Question> questionList = getQuestionListLogic.execute();
 questionList.add(question);
}
 } catch (SQLException e){
 e.printStackTrace();
//   return null;
 }
// return questionList;
return questionList;
 }
}

package model;

import java.util.List;

import DAO.QuestionDAO;

public class GetLogic {
 public List<Question> execute(){
 QuestionDAO dao = new QuestionDAO();
 List<Question> questionList = dao.findAll();
 return questionList;
 }
}

----------追加----------------

いただいた回答より編集致しましたが
List<Question> list = (List<Question>) session.getAttribute("questionList");の行で以下エラーが出ます。ご確認いただけますでしょうか。
エラー：この行で見つかった複数の注釈：　-Listを型に解決できません

<%@ page language="java" contentType="text/html; charset=UTF-8"
   pageEncoding="UTF-8" %>
<%@ page import="model.Question"%>
<%
  @SuppressWarnings("unchecked")
  List<Question> list = (List<Question>) session.getAttribute("questionList");
%>

<!DOCTYPE html>
<html lang="ja">
<head>
  <meta charset="utf-8">
  <meta name="viewport" content=“width=device-width,initial-scale=1.0”>
  <meta name="viewport" content="width=device-width, initial-scale=1.0, maximum-scale=1.0, minimum-scale=1.0">
  <title>課題1</title>
  <link rel="stylesheet" href="css/QandAList.css">
</head>
<body>
<header>
  <div class="container">
    <p>質問一覧</p>
    <a href="/QandASystem/RegistQuestionServlet">質問する</a>
  </div>
</header>
<div class="main">
  <div class="container">
      <table border="1">
        <thead>
          <tr>
            <th class="no" width="7%">No</th>
            <th width="15%">緊急度</th>
            <th width="40%">タイトル</th>
            <th>登録者</th>
            <th>登録日</th>
            <th>更新日</th>
          </tr>
        </thead>
        <tbody>
          <% for (int i=0; i<list.size(); i++) { %>
          <tr>
            <td><%=i+1%></td>
            <td><%=list.get(i).getUrgency()%></td>
            <td><%=list.get(i).getTitle()%></td>
            <td><%=list.get(i).getHandle_name()%></td>
            <td><%=list.get(i).getRegist_timestamp()%></td>
            <td><%=list.get(i).getUpdate_timestamp()%></td>
          </tr>
          <tr>
            <td><%=i+1%></td>
            <td><%=list.get(i).getUrgency()%></td>
            <td><%=list.get(i).getTitle()%></td>
            <td><%=list.get(i).getHandle_name() %></td>
            <td><%=list.get(i).getRegist_timestamp() %></td>
            <td><%=list.get(i).getUpdate_timestamp() %></td>
          </tr>
          <tr>
            <td><%=i+1%></td>
            <td><%=list.get(i).getUrgency()%></td>
            <td><%=list.get(i).getTitle()%></td>
            <td><%=list.get(i).getHandle_name() %></td>
            <td><%=list.get(i).getRegist_timestamp() %></td>
            <td><%=list.get(i).getUpdate_timestamp() %></td>
          </tr>
          <tr>
            <td><%=i+1%></td>
            <td><%=list.get(i).getUrgency()%></td>
            <td><%=list.get(i).getTitle()%></td>
            <td><%=list.get(i).getHandle_name() %></td>
            <td><%=list.get(i).getRegist_timestamp() %></td>
            <td><%=list.get(i).getUpdate_timestamp() %></td>
          </tr>
          <tr>
            <td>1</td>
            <td><%=list.get(i).getUrgency()%></td>
            <td><%=list.get(i).getTitle()%></td>
            <td><%=list.get(i).getHandle_name() %></td>
            <td><%=list.get(i).getRegist_timestamp() %></td>
            <td><%=list.get(i).getUpdate_timestamp() %></td>
          </tr>
          <% } %>
        </tbody>
      </table>
  </div>
</div>
<script src="js/main.js"></script>
</body>
</html>

Answer 1

原因は、サーブレット側でList<Question> questionListをセッションにセットしているのに、

//質問リスト取得
GetLogic getLogic = new GetLogic();    
List<Question> questionList = getLogic.execute();
HttpSession session = request.getSession();
session.setAttribute("questionList", questionList);

JSP側で以下のようにキャストしていることだと思います。List<Question>にキャストするように変えてみてください。

Question q = (Question) session.getAttribute("questionList");

また、その後に続く以下の記述もListに対応するように書き直す必要があると思います。

<tr>
    <td>5</td>
    <td><%=q.getUrgency()%></td>
    <td><%=q.getTitle()%></td>
    <td><%=q.getHandle_name() %></td>
    <td><%=q.getRegist_timestamp() %></td>
    <td><%=q.getUpdate_timestamp() %></td>
</tr>

追記

例えば、listという変数に代入した場合

<%
  @SuppressWarnings("unchecked")
  List<Question> list = (List<Question>) session.getAttribute("questionList");
%>

Listに対応した書き方は以下のようにJSPの中でfor文を書きます。

<tbody>
  <% for (int i=0; i<list.size(); i++) { %>
  <tr>
    <td><%=i+1%></td>
    <td><%=list.get(i).getUrgency()%></td>
    <td><%=list.get(i).getTitle()%></td>
    <td><%=list.get(i).getHandle_name()%></td>
    <td><%=list.get(i).getRegist_timestamp()%></td>
    <td><%=list.get(i).getUpdate_timestamp()%></td>
  </tr>
  <% } %>
</tbody>

Comments

[nagahi]

回答ありがとうございます！
キャストの部分でのエラーは消えました。
しかしJSPの値を取得するところでエラーがきえません。
Listに対応する記述の方法ご教授いただけないでしょうか。無知ですみません。

[rubytomato]

追記しましたのでご確認ください。

[nagahi]

ご回答ありがとうございます。
JSPにご教授いただいた修正を加えたのですが、エラーが発生しました。
お手数ですが、私の記載方法に誤りがないかご確認いただけますでしょうか。
※質問に修正後のコードを追記しております

[rubytomato]

> キャストの部分でのエラーは消えました。
> しかしJSPの値を取得するところでエラーがきえません。

Listへのキャストのエラーが解決できたということだったので、回答には記載していませんでしたが
Listを使うには

> <%@ page import="model.Question"%>

と同じようにインポートする必要があります。

<%@ page import="java.util.List"%>

それと、forループ処理の中で<tr> ... </tr>の部分を5回書いていますが、forループでデータ件数分繰り返すのでこの部分は1回で済みます。

[nagahi]

ご回答ありがとうございます！
おかげさまでエラーが改善いたしました。

可能であればお答えいただきたいのですが
表示結果がcolumnの項目のみで、値が表示されないのですが
これはデータが登録されてないということでしょうか。
SQLの全く知識がなく、登録されてないだけであれば
独学で入れて試してみます。

[rubytomato]

> 表示結果がcolumnの項目のみで、値が表示されないのですが
> これはデータが登録されてないということでしょうか。

questionテーブルにデータが登録されているかどうかはデータベースにログインしてselect文を実行してみればわかると思います。
データを登録していないのであれば登録してください。

データが登録されているのにページに表示されないということであれば、データ検索周りの処理に問題があるのではないでしょうか？

サーブレットの検索処理の後に以下のSystem.out.println文を追加して、検索件数をコンソールに出力して確認してください。

List<Question> questionList = getLogic.execute();
// 追加
System.out.println(questionList.size());

出力された値が0であれば検索できていないということになります。その場合は別途質問を立ててください。

[nagahi]

ご回答ありがとうございます！

ご記載いただいた通り、サーブレットにSystem.out.println文を追加し
実行すると、0と表示されました。。

ターミナルでselect * from questionのコマンドをたたいても
何も表示されないのですが、これはテーブルにデータがないのか
データ検索周りの処理に問題があるのか、どちらかわかりますでしょうか。

不足の情報等ありましたらお知らせください。
すみませんが、よろしくお願いします。

[rubytomato]

> ターミナルでselect * from questionのコマンドをたたいても
> 何も表示されないのですが、これはテーブルにデータがないのか

データベースにログインした後にselect文を実行したのでしょうか？
それとこのselect文の結果をそのまま質問内容に追記してください。データは表示されなくてもなにかしらのメッセージは表示されていませんか？

> データ検索周りの処理に問題があるのか、どちらかわかりますでしょうか。

この情報だけから判断するならデータが登録されていないのではないでしょうか？
データを登録してもなおページに表示されないのなら検索処理に問題があるのだと思います。

とりあえずデータを登録してもう一度確認してください。

[nagahi]

ご対応ありがとうございます。

データベースの登録につきまして、
登録し、その後select文にて値を確認できました。

その後、サーブレット実行しても0と表示されましたので
検索処理に問題があるようです。

もう一度コード再確認してみますが
もしコードの修正箇所等見つかりましたら
教えていただければ幸いです。。

[rubytomato]

> もしコードの修正箇所等見つかりましたら
> 教えていただければ幸いです。。

例えばcreate table文など、questionテーブルのすべてのカラム名が分かる情報を追記してください。

それと、QuestionDAOクラスの以下の箇所に"throw new RuntimeException(e);" を追加してください。
2カ所あるので2カ所とも同様に対処してください。

} catch (SQLException e){
e.printStackTrace();
// 追加
throw new RuntimeException(e);
}

追加したら動作確認してエラーが起きないか確認してください。ブラウザで確認する他にコンソールにもエラーが出力されていないか確認してください。

[nagahi]

ありがとうございます。記載いただいたことを実施致しましたので
下記に結果を記載いたします。

> 例えばcreate table文など、questionテーブルのすべてのカラム名が分かる情報を追記してください。
→以下にcreate table文を記載します。
　CREATE TABLE question
(
-- 質問ID
question_id int NOT NULL,
-- ハンドルネーム
handle_name text NOT NULL,
-- タイトル
title text NOT NULL,
-- コンテンツ
contents text NOT NULL,
-- 緊急度
-- 1：急いでます
-- 2：困ってます
-- 3：いつでも
urgency int NOT NULL,
-- 編集・削除キー
edit_delete_key text,
-- 登録日時
regist_timestamp timestamp with time zone NOT NULL,
-- 更新日時
update_timestamp timestamp with time zone,
PRIMARY KEY (question_id)
) WITHOUT OIDS;

> 追加したら動作確認してエラーが起きないか確認してください。ブラウザで確認する他にコンソールにもエラーが出力されていないか確認してください。
→追加後、サーブレット実行でHTTPステータス 500のエラーが発生しました。
　説明：The server encountered an internal error that prevented
　　　　it from fulfilling this request.
　例外：javax.servlet.ServletException: サーブレットの実行により
　　　　例外を投げました
　　　　org.apache.tomcat.websocket.server.WsFilter.
　　　　doFilter(WsFilter.java:52)

　原因：java.lang.NoClassDefFoundError: Could not initialize
　　　　class DAO.QuestionDAO
　model.GetLogic.execute(GetLogic.java:9
　　　・・・・・

[rubytomato]

動作確認でエラーが起きたということは、QuestionDAOクラスの実装に問題がありそうです。
とりあえず、追加して頂いた2つの "throw new RuntimeException(e);" はコメントアウトしてください。

その代わりに、ShowQuestionServletを以下のように修正してください。
追加するコードはダミーのデータをリストにセットします。

//質問リスト取得
GetLogic getLogic = new GetLogic();
// ↓コメントアウト
// List<Question> questionList = getLogic.execute();
// ↓ 追加
List<Question> questionList = new ArrayList<>();
Question q1 = new Question(1, "test_handle1", "test_title1", "test_contents1", 1, "test_key2", "2020-04-12 23:59:59", "2020-04-12 23:59:59");
Question q2 = new Question(2, "test_handle2", "test_title2", "test_contents2", 2, "test_key1", "2020-04-12 23:59:59", "2020-04-12 23:59:59");
questionList.add(q1);
questionList.add(q2);
// ↑ 追加
HttpSession session = request.getSession();
session.setAttribute("questionList", questionList);

これで、動作確認してダミーデータがページに表示されるか確認してください。

[nagahi]

ありがとうございます！
記載いただた内容で実行すると、JSPのレスポンスにダミーデータが表示
されました！

やはりQuestionDAOクラスに問題があるということでしょうか。

[rubytomato]

> 記載いただた内容で実行すると、JSPのレスポンスにダミーデータが表示されました！
> やはりQuestionDAOクラスに問題があるということでしょうか。

そういうことだと思います。

データが表示できたのであれば、この質問の主旨である『スコープに保存した値をJSPで表示したい』という問題は
解決できたということだと思いますので、この質問は解決済みとしてクローズしてください。

■記載コードの2つの問題点

この質問の記載コードには大きく分けて2つの問題があり、1つはデータ検索処理の部分、1つはデータ表示の部分です。

データ表示部分はこれまでのやり取りでコードの不具合は解決できましたが、
データ検索部分にも複数の不具合があるようなので解決するにはまだまだ確認・調査が必要です。
このため、この質問でこれ以上やり取りをするのは不適切なのでデータ検索部分については別途質問を立ててください。

と言っても、この続きで質問を立てても丸投げという指摘を受ける可能性がありますので、以下の点についてご自身でもう少しお調べ頂いた方がいいと思います。
（あくまでも私がそう思っているだけなので、このまますぐに新しいご質問を立てても、この質問で引き続き回答を待つのもご自由です。）

■ご確認頂きたい点

1) JDBCドライバーの有無

"throw new RuntimeException(e);"というコードを追加して頂いたときに発生した以下のエラーから

> java.lang.NoClassDefFoundError: Could not initialize
> class DAO.QuestionDAO

JDBCドライバーが正常に読み込まれていない可能性があります。
ビルドパスにPostgreSQLのJDBCドライバーがあるか確認してください。

2) Questionクラスのフィールドの型

データベースのquestionテーブルのカラムのデータ型とJavaのQuestionクラスのフィールドの型は適切にマッピングされていないといけません。
特に問題になっているのはquestionテーブルのtimestamp型をJavaのString型にマッピングしている点は問題かと思います。

CREATE TABLE question (
question_id int NOT NULL,
handle_name text NOT NULL,
title text NOT NULL,
contents text NOT NULL,
urgency int NOT NULL,
edit_delete_key text,
regist_timestamp timestamp with time zone NOT NULL,
update_timestamp timestamp with time zone,
PRIMARY KEY (question_id)
)
WITHOUT OIDS;

3) テーブルのカラム名

ResultSetから検索結果を取得する際にテーブルのカラム名を記述していますが、カラム名に書き間違いがないか確認してください。

//SELECT文の結果を取り出す
while (rs.next()) {
int question_id = rs.getInt("question_id");
String handle_name = rs.getString("hanle_name");
String title = rs.getString("title");
String contents = rs.getString("contents");
int urgency = rs.getInt("urgency");
String edit_delete_key = rs.getString("edit_delete_key");
String regist_timestamp = rs.getString("regist_timestamp");
String update_timestamp = rs.getString("update_timestamp");
}

以上3点が現状であやしいと思うところです。他にもあるかもしれませんが動作確認しながら調べてみてください。

■改めてご質問をされる場合

上記の点を確認したうえで、どうしても解決できない場合は改めてご質問される方がよいと思います。
その場合は以下の点をご質問内容に反映して頂くと回答が付きやすいと思います。

1) 記載するソースコードはコードブロックで囲み、且つ適切なインデント（段落）を付ける

2) 利用するデータベースの種類、バージョンを記載する

3) 利用するテーブルの情報（create table文）を記載する

4) 開発ツールの種類、バージョンを記載する

5) Javaのバージョンを記載する

6) JDBC ドライバーのバージョンを記載する

7) エラーメッセージは可能な限り全文記載する。字数制限があるので全文が無理な場合は先頭から何行か抜き出して記載し、あとは全文がわかるスクリーンショットを撮ってそれを添付するといいと思います。この場合誤解が生まれないように『字数制限のため』という断り書きがあるといいと思います。

最後に、先ほど追加して頂いたダミーデータを追加する処理は不要なので削除して、コメントアウトした部分はもとに戻しておいてください。
また、2カ所に追加して頂いた"throw new RuntimeException(e);"を以下のように復活させてください。
こうすることでエラーが起きたときにすぐに気づけると思います。（もともとの例外処理の中で e.printStackTrace(); だけを書くのは悪手です。）

1カ所目

try{
Class.forName("org.postgresql.Driver");

} catch(ClassNotFoundException e) {
e.printStackTrace();
throw new RuntimeException("class load error", e);
}

2カ所目

try (Connection conn = DriverManager.getConnection(jdbc_url, user, pass)) {

} catch (SQLException e) {
e.printStackTrace();
throw new RuntimeException("sql execute error", e);
}

以上よろしくお願いします。

[nagahi]

ご回答ありがとうございます！

最初の質問の部分は解決致しましたので、本件はクローズさせて
いただきます。
ご記載いただいた点を確認し、解決できなければ
再度別で質問をたてるように致します。

長い間ご対応いただき、ありがとうございました。
大変助かりました。