JDBCにてログイン成功させたい

Question

実現したいこと

ログイン成功時に出てくる写真のような実行結果が出てきてほしい
イメージ説明

発生している問題・分からないこと

データベースでpdAdmin4を使用してカラム（id,password,name,comment,login_time)も間違えていないが、index.htmlにて実行しても
「idとパスワードが間違っています」となり、ログイン成功しない。

該当のソースコード

package bean;

/**
 * ・社員情報データ（モデル）
 * 
 */

public class EmployeeBean {

    private String Id;
    private String PassWord;
    private String Name;
    private String Comment;
    private String Login_Time;

    public void setId(String Id) {
        this.Id = Id;
    }

    public String Id() {
        return Id;
    }

    public void setPassWord(String PassWord) {
        this.PassWord = PassWord;
    }

    public String getPassWord() {
        return PassWord;
    }

    public void setName(String Name) {
        this.Name = Name;
    }

    public String getName() {
        return Name;
    }

    public void setLogin_Time(String Login_Time) {
        this.Login_Time = Login_Time;
    }

    public String getLogin_Time() {
        return Login_Time;
    }

    public void setComment(String Comment) {
        this.Comment = Comment;
    }

    public String getComment() {
        return Comment;
    }
}

package controller;

/**
 * 社員情報管理コントローラー
 *
 */

import java.io.IOException;

import javax.servlet.RequestDispatcher;
import javax.servlet.ServletContext;
import javax.servlet.ServletException;
import javax.servlet.http.HttpServlet;
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;

import bean.EmployeeBean;
import service.EmployeeService;

public class EmployeeController extends HttpServlet {
    public void doPost(HttpServletRequest request, HttpServletResponse response)
            throws ServletException, IOException {

        try {
            // 問① index.htmlから送信されたIDとPassWordの値を取得できるように修正してください。
            String id = request.getParameter("id");
            String password = request.getParameter("password");
            
            /*
             *  IDとPassWordと元に、社員情報を検索する関数の呼び出し、結果をJSPに渡す処理
             *  ※ EmployeeBeanとEmployeeServiceをimportするのを忘れないでください。
             */

            // 問② EmployeeServiceクラスをインスタンス化してください。
            EmployeeService employeeservice = new EmployeeService();
            // 問③ EmployeeBeanに、EmployeeServiceよりsearch関数を呼び出し、返り値を格納してください。
            EmployeeBean returndata = employeeservice.search(id,password);
            // 問④ 適切な引数をセットしてください。
            request.setAttribute("EmployeeBean", returndata);

        } catch (Exception e) {
            e.printStackTrace();
        } finally {
            ServletContext context = this.getServletContext();
            RequestDispatcher dispatcher = context.getRequestDispatcher("/index.jsp");
            dispatcher.forward(request, response);
        }
    }
}

package service;

import java.sql.Connection;
import java.sql.DriverManager;
import java.sql.PreparedStatement;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.sql.Statement;
import java.text.SimpleDateFormat;
import java.util.Calendar;

import bean.EmployeeBean;

/**
 * ・社員情報検索サービス
 *
 */

public class EmployeeService {

    // 問① 接続情報を記述してください
    /** ドライバーのクラス名 */
    private static final String POSTGRES_DRIVER = "org.postgresql.Driver";
    /**
     * ・JDMC接続先情報
     *   ポート番号(5432, 5433など)は個人で異なる可能性があります。
     *   念の為、pgAdminにて対象のpostgreSQLバージョンよりメニューを開き、[プロパティ] -> [接続] を確認してください
     */
    private static final String JDBC_CONNECTION = "jdbc:postgresql://localhost:5432/lesson_db";
    /** ・ユーザー名 */
    private static final String USER = "postgres";
    /** ・パスワード */
    private static final String PASS = "postgres";
    /** ・タイムフォーマット */
    private static final String TIME_FORMAT = "yyyy/MM/dd HH:mm:ss";

    // 問② 入力されたIDを条件にログイン時間を更新するUPDATE文
    /** ・SQL UPDATE文 */
    private static final String SQL_UPDATE = "UPDATE employee_table SET  login_time =?  id = ?";

    // 問③ 入力されたIDとPassWordを条件と合致するデータを検索するSELECT文
    /** ・SQL SELECT文 */
    private static final String SQL_SELECT = "SELECT name, comment, login_time FROM employee_table WHERE id = ? AND \"password\" = ?";

    EmployeeBean employeeData = null;

    // 送信されたIDとPassWordを元に社員情報を検索するためのメソッド
    public EmployeeBean search(String id, String password) {

        Connection connection = null;
        Statement statement = null;
        ResultSet resultSet = null;
        PreparedStatement preparedStatement = null;

        try {
            // データベースに接続
            Class.forName(POSTGRES_DRIVER);
            connection = DriverManager.getConnection(JDBC_CONNECTION, USER, PASS);
            statement = connection.createStatement();

            // 処理が流れた時間をフォーマットに合わせて生成
            Calendar cal = Calendar.getInstance();
            SimpleDateFormat sdFormat = new SimpleDateFormat(TIME_FORMAT);

            // PreparedStatementで使用するため、String型に変換
            String login_time = sdFormat.format(cal.getTime());

            /*
             *  任意のユーザーのログインタイムを更新できるように、プリペアドステートメントを記述。
             */
            

            // preparedStatementに実行したいSQLを格納
            preparedStatement = connection.prepareStatement(SQL_UPDATE);
            // 問④ preparedStatementを使って、第一引数に現在時刻を、第二引数にIDをセットしてください。
            preparedStatement.setString(1,login_time);
            preparedStatement.setString(2,id);

            // 問⑤ UPDATEを実行する文を記述してください。
            preparedStatement.executeUpdate();

            /*
             *  UPDATEが成功したものを即座に表示
             *  任意のユーザーを検索できるように、プリペアドステートメントを記述。
             */
            preparedStatement = connection.prepareStatement(SQL_SELECT);
            // 問⑥ preparedStatementを使って、第一引数にIDを、第二引数にPASSWORDをセットしてください。
            preparedStatement.setString(1,id);
            preparedStatement.setString(2,password);

            // SQLを実行。実行した結果をresultSetに格納。
            resultSet = preparedStatement.executeQuery();

            while (resultSet.next()) {
                // 問⑦ tmpName,tmpComment,tmpLoginTimeに適当なデータを格納してください。
                String tmpName = resultSet.getString("name");
                String tmpComment = resultSet.getString("comment");
                String tmpLoginTime = resultSet.getString("login_time");

                // 問⑧ EmployeeBeanに⑦で取得したデータを格納してください。
                employeeData = new EmployeeBean();
                employeeData.setName(tmpName);
                employeeData.setComment(tmpComment);
                employeeData.setLogin_Time(tmpLoginTime);
            }

            // forName()で例外発生
        } catch (ClassNotFoundException e) {
            e.printStackTrace();

            // getConnection()、createStatement()、executeQuery()で例外発生
        } catch (SQLException e) {
            e.printStackTrace();

        } finally {
            try {

                if (resultSet != null) {
                    resultSet.close();
                }
                if (statement != null) {
                    statement.close();
                }
                if (connection != null) {
                    connection.close();
                }

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

<%@ page import="bean.EmployeeBean"%>

<%
    // 問① getAttributeに適切な引数をセットして、EmployeeControllerから渡されたEmployeeBeanを取得してください。
    EmployeeBean employeeBean = (EmployeeBean) request.getAttribute("employeeBean");
%>
<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd">
<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
<title>検索結果</title>
</head>
<body>
    <div align="center">
        <!-- 検索が成功した場合の表示 -->
        <%
            if (employeeBean != null) {
        %>
        <table border="1">
            <tr>
                <th>社員名</th>
                <td><%=employeeBean.getName()%></td>
            </tr>
            <tr>
                <th>コメント</th>
                <td><%=employeeBean.getComment()%></td>
            </tr>
            <tr>
                <th>ログインタイム</th>
                <td><%=employeeBean.getLogin_Time()%></td>
            </tr>
        </table>

        <!-- 問②  それ以外の表示（エラーの場合）-->
        <% } else { %>
           <p>IDもしくはパスワードが間違ってます</p>
        <% } %>
    </div>

</body>
</html>

試したこと・調べたこと

• teratailやGoogle等で検索した
• ソースコードを自分なりに変更した
• 知人に聞いた
• その他

上記の詳細・結果

同じような現象の方をこちらで見つけ、同じようにコードを打ち直しましたが、
idとパスワードが間違っています。と同じ実行結果となった。

補足

pgAdmin4 のポート番号等が違う可能性があるのですが、
プログラミングが初めてでどこが違うのかわかりませんでした。

Comments

[jimbe]

「idとパスワードが間違っています」というのはどこに出ているのでしょうか。
コードにもスクリーンショットにも見当たらないようなのですが。

[hand45major56]

すみません。画像添付していたのを修正しました。
実行結果のみしか添付できませんでした。
実行前は、eclipseにて、index.htmlを実行。
IDに「09090」、パスワードに「admin」といれましたが
実行結果に「idとパスワードが間違っています。」となってしまいます。

[jimbe]

修正ありがとうございます。

EmployeeController は最終的には index.jsp を出力するようです。
index.jsp もご提示頂けますか？

[hand45major56]

こちらこそご連絡ありがとうございます。
こちらi質問内容の関係上、index/html→index.jspに変更して修正しました。
よろしければご確認をよろしくお願いします

[jimbe]

編集有難う御座います。
質問の 10000 文字制限に引っ掛かってしまいましたか(><

Answer 1

※コードを実行して確認はしていません。

ご存知とと思いますが、 Servlet → jsp のフォワードでのデータ受け渡しは HttpServletRequest オブジェクトの setAttribute と getAttribute の組み合わせですが、データの区別は第一引数の文字列で行われます。
その為、対応する set/get では同じ文字列にしなくてはなりません。

ご提示の EmployeeController.java と index.jsp はそれぞれ
request.setAttribute("EmployeeBean", returndata);
と
EmployeeBean employeeBean = (EmployeeBean) request.getAttribute("employeeBean");
となっており、名前(の大文字小文字)が同じではありません。

また jdbc での接続を疑っておられるようですが、原因に複数の可能性がある場合は、それらを切り分けられるような方法で確認テストを行うことをお勧めします。
例えば EmployeeService クラス(の search メソッド) はサーブレットから呼ばなくても

public class EmployeeServiceTest {
  public static void main(String args[]) {
    EmployeeService es = new EmployeeService();
    System.out.println("bean=" + es.search("YamadaTaro", "javaman"));
  }
}

のようなテストアプリを作ってコマンドで実行してみれば動作確認は出来るはずです。
データベースとの接続で問題があるならアプリでもエラーが出るはずですし、エラーにならなければ サーブレットに問題があるとなり、postgress の設定に問題があるのかと目視確認に時間を取られずに済みます。

また Servlet のほうは一時的に EmployeeService を使う部分をコメント化し直接 Bean を設定することで、index.jsp への動作を確認することが出来ます。

 :
  // 問② EmployeeServiceクラスをインスタンス化してください。
  //EmployeeService employeeservice = new EmployeeService();
  // 問③ EmployeeBeanに、EmployeeServiceよりsearch関数を呼び出し、返り値を格納してください。
  //EmployeeBean returndata = employeeservice.search(id,password);
  EmployeeBean returndata = new EmployeeBean();
  returndata.setName("YamadaTaro");
  :

コード内のコメントから何かの課題のようですが、提出するまではどのように弄っても大丈夫なのではないでしょうか。
プログラムは途中幾ら弄っても最終的に不要部分を消してしまえば他人には途中の試行錯誤等は分かりません。一部を取り出して動かして見るとか逆に一部を省略・置き換えて動かして見るとかはプロでも普通にやっていることですので、じゃんじゃん動かしてみてください。

Comments

[hand45major56]

すみません…色々ありがとうございます！
ご指摘あったところを大文字で修正して実行したのですが
次は「HTTPのGETメソッドはこのURLではサポートされていません。」となりました。
受け渡しも間違えていないと思ったのですが…

[hand45major56]

たびたび申し訳ありません…
pgadmin 4側のデータの受け渡しを勝手に拒否されてるみたいなものってありますかね？
eclipseも前までNortonを入れていたことで実行出来なかったりしたのでその可能性もあるかなと…

[jimbe]

>HTTPのGETメソッドはこのURLではサポートされていません。

EmployeeController には doPost しか実装していませんから、 HTTP の GET ではそのような表示になります。
実行した時の index.html の form タグの method は post になっているでしょうか。
もしくは、もし直接プラウザに EmployeeController の URL を指定して実行していた場合は HTTP の GET になります。 index.html から実行してください。

[jimbe]

>pgadmin 4側のデータの受け渡しを勝手に拒否されてるみたいなものって

まず、 pgadmin というのは postgress の設定用のプログラムであって、 お作りになっているシステム自体とは関係ありません。ですから pgadmin が何か悪さをすることは無いと思います。

質問本文で発生していた「IDもしくはパスワードが間違ってます」という表示は、 index.jsp が動作することで表示されていたものであり、従って index.html → EmployeeController → index.jsp と動いていたはずです。(直接 index.jsp を実行したので無ければ・・・。)
それが今同じく index.html から操作して「HTTPのGETメソッドは…」と出たのであれば、 EmployeeController 自体の実行で失敗していることになり postgress のアクセスまで到達していません。
ですので postgress の問題で発生しているとは思えません。