java でsqldbからデータ取得してログイン処理の作成

Question

前提・実現したいこと

ログイン処理実現のため、入力された文字列をpostgresqlのdbで照合したい。
そのためにtestjavaでsqlに接続・文字取得できているか調べているのですがうまくできておりません。

発生してるエラーはなく、
DAOjavaの取得したい列名（rs.getStringの中身）を適当なものに書き換えると
org.postgresql.util.PSQLException: この ResultSet に列名 ●●●（書きかけた文字） ありません。

とでるのでsqldb取得まではできてると思います。
そこから文字列比較しようとしてるところで失敗に終わるのでどうしてそうなっているのか、アドバイスしていただけると助かります。

該当のソースコード

package dao;

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

import model.Account;
import model.Login;

public class AccountDAO {
  // データベース接続に使用する情報
	private final String url = "jdbc:postgresql://127.0.0.1:5432/test";
	//private final String url ="jdbc:postgresql://127.0.0.1:5432/";
	private final String user = "postgres";
	private final String password = "postgres";

  public Account findByLogin(Login login) {
    Account account = null;

    // データベースへ接続
    try (Connection conn = DriverManager.getConnection(
    		url, user, password)) {

      // SELECT文を準備
      String sql = "SELECT USER_ID, PASS, MAIL, NAME, AGE FROM ACCOUNT WHERE USER_ID = ? AND PASS = ?";
      PreparedStatement pStmt = conn.prepareStatement(sql);
      pStmt.setString(1, login.getUserId());
      pStmt.setString(2, login.getPass());

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

      // 一致したユーザーが存在した場合
      // そのユーザーを表すAccountインスタンスを生成
      if (rs.next()) {
        // 結果表からデータを取得
        String userId = rs.getString("USER_ID");
        String pass = rs.getString("PASS");
        String mail = rs.getString("MAIL");
        String name = rs.getString("NAME");
        int age = rs.getInt("AGE");
        account = new Account(userId, pass, mail, name, age);
      }
    } catch (SQLException e) {
      e.printStackTrace();
      return null;
    }
    // 見つかったユーザーまたはnullを返す
    return null;
  }
}

package model;

public class Account {
  private String userId;
  private String pass;
  private String mail;
  private String name;
  private int age;

  public Account(String userId, String pass, String mail,
      String name, int age) {
    this.userId = userId;
    this.pass = pass;
    this.mail = mail;
    this.name = name;
    this.age = age;
  }

  public String getUserId() {
    return userId;
  }

  public String getPass() {
    return pass;
  }

  public String getMail() {
    return mail;
  }

  public String getName() {
    return name;
  }

  public int getAge() {
    return age;
  }
}

package model;

public class Login {
  private String userId;
  private String pass;

  public Login(String userId, String pass) {
    this.userId = userId;
    this.pass = pass;
  }

  public String getUserId() {
    return userId;
  }

  public String getPass() {
    return pass;
  }
}

package test;

import dao.AccountDAO;
import model.Account;
import model.Login;

public class AccountDAOTest {
  public static void main(String[] args) {
    testFindByLogin1(); // ユーザーが見つかる場合のテスト
    testFindByLogin2(); // ユーザーが見つからない場合のテスト
  }

  public static void testFindByLogin1() {
    Login login = new Login("toumasu", "1212");
    AccountDAO dao = new AccountDAO();
    Account result = dao.findByLogin(login);
    if (result != null &&
        result.getUserId().contentEquals("toumasu") &&
        result.getPass().contentEquals("1212") &&
        result.getMail().contentEquals("[e-mail]") &&
        result.getName().contentEquals("トーマス") &&
        result.getAge() == 19) {
      System.out.println("findByLogin1:成功しました");
    } else {
      System.out.println("findByLogin1:失敗しました");
    }
  }

  public static void testFindByLogin2() {
    Login login = new Login("toumasu", "12345");
    AccountDAO dao = new AccountDAO();
    Account result = dao.findByLogin(login);
    if (result == null) {
      System.out.println("findByLogin2:成功しました");
    } else {
      System.out.println("findByLogin2:失敗しました");
    }
  }
}

補足情報

eclipse　
Version: 2018-12 (4.10.0)ールのバージョンなど）
PostgreSQL PgAdmin4 ver 3.5

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

Comments

[rnosh]

最後にうまくいかないとありますが、どううまくいかなかったのでしょうか。
ページが遷移しないのか？照合の際にエラーが出るのか？エラーが出たならどんなエラーが出ているのか？などなど。
お力添えしたいと思いますが、どこでどう詰まっているのか、もう少し情報をいただけると回答できるかと思います。

Answer 1

大前提ですが、formは入れ子にできません（HTMLの仕様です）

• formタグは入れ子にできない＆その対処法

入れ子にされている場合（私はその大前提知っているのでやったことないですが）
中身の入力情報が正しく認識、送信されない可能性があります。
一回に送信したい情報全てを1つのformで囲いましょう。

「照合処理」のほうですが下記のSQLですと何も照合しようとしてないと思います。

            String sql = "SELECT * FROM ユーザーログイン情報";
            rset = stmt.executeQuery(sql);

し、

下記の記述だと固定になっていますがこれは大丈夫ですか？

        if (userid.equals("user01")&&pass.equals("taqusu"))//[23]

確かにuser01、taqusuと入れれば通るかもしれませんが・・・。

通常ですとSQLでuser_idとpasswordをWHERE句に入れて対応します。

SELECT * FROM ユーザーログイン情報 WHERE user_id = '入力ユーザーID' and password = '入力パスワード'

Comments

[S_kura]

返信ありがとうございます。formの件については今現在問題なく動いてるのですが修正しておきます。
教えていただきありがとうございます。

そして記述足りていなくてすみません。
sqlコードのほうは完全に別のものです。
今現在jsp上の画面で入力された文字を3つめのSredirect.javaで決められた文字だけを照合してるだけのソースコードです。
そのコードに
javaで書いたsqlコード組み込んで動作させたいと考えております。

[m.ts10806]

では、ほぼそのまま移管して私が書いたような形でwhereにてid,passを検索するようにしてみては。
DB接続の処理も何も書かれていないのに「うまくいきません」と言われても「そもそも書いてないよね」から始まりますけど・・

[S_kura]

申し訳ございません。この記事を投稿する前に移管しておりましたが、うまくできず別々のソースコードを張ったほうがいいと思い試したものははりませんでした。
ですが試した結果を張っておくべきでした申し訳ございません。

ppackage web1;

import java.io.IOException;
import java.sql.Connection;
import java.sql.DriverManager;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.sql.Statement;

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

@WebServlet("/Sredirect")//[1]
public class Sredirect extends HttpServlet {//[2]

protected void doGet(HttpServletRequest request, HttpServletResponse response)
throws ServletException, IOException {//[3]

response.setContentType("text/html; charset=UTF-8");//[4]

//入力された単語を取得したのを変数に代入
String userid = request.getParameter("word");
String pass = request.getParameter("pass");

//sql
Connection conn = null;
Statement stmt = null;
ResultSet rset = null;

//sqlから取得した文字を代入
String name = null;
String word = null;

//接続文字列
String url = "jdbc:postgresql://127.0.0.1:5432/test";
String user = "postgres";
String password = "postgres";

try{
//PostgreSQLへ接続
conn = DriverManager.getConnection(url, user, password);

//自動コミットOFF
conn.setAutoCommit(false);

//SELECT文の実行
stmt = conn.createStatement();
String sql = "SELECT * FROM ユーザーログイン情報 where ログインid ='"+userid+"' and パスワード = '"+pass+"'";
rset = stmt.executeQuery(sql);

//SELECT結果の受け取り
while(rset.next()) {

name =rset.getString(1);
word =rset.getString(2);

}

}
catch (Exception ex) {
ex.printStackTrace();
}

finally {
try {
if(rset != null)rset.close();
if(stmt != null)stmt.close();
if(conn != null)conn.close();
}
catch (SQLException e){
e.printStackTrace();
}

}

//単語をチェックするメソッドcheckword呼んで判定している
if (!checkWord(name,word)) {

ServletContext context = this.getServletContext();
RequestDispatcher dispatcher = context.getRequestDispatcher("/redirect.jsp");
dispatcher.forward(request, response);

return;

}

ServletContext context = this.getServletContext();
RequestDispatcher dispatcher = context.getRequestDispatcher("/top.jsp");
dispatcher.forward(request, response);

}

// //メゾットチェックワードを定義
boolean checkWord(String name,String word){

if (name == null || word == null)
return false;

if (name.length() > 0 && word.length() > 0)
return true;

return false;

}

}

自分なりにやってみたのですが、name、wordにうまくsqlから取得した文字が代入されずトップ.jspのほうへいかずじまいです。
どのあたりを修正すればうまくいくでしょうか？

[m.ts10806]

コメント欄ではマークダウンが使えません。
質問に追記してください。

[S_kura]

たびたび申し訳ございません。

Answer 2

参照ファイルだけにsql.jarしていたため、java単体では動いていたがweb接続の時はwebにはドライブがないため接続されてなかった。
そのためプロジェクトファイル→プロパティからデブロイトメントアセンブリーにsqlドライブを追加したらwebでも接続ができるようになった

Answer 3

とりあえず、別アカウントの質問でも聞きましたがどこでアカウント情報を返してるか把握してください。

    // 見つかったユーザーまたはnullを返す
    return null;

常にnullです

Answer 4

EclipseやIntelliJ IDEAなどを使って、デバッグモードで

    if (result != null &&
        result.getUserId().contentEquals("toumasu") &&
        result.getPass().contentEquals("1212") &&
        result.getMail().contentEquals("[e-mail]") &&
        result.getName().contentEquals("トーマス") &&
        result.getAge() == 19) {
      System.out.println("findByLogin1:成功しました");
    } else {

の部分でどこで違っているのかをブレークポイントをつけて探すか、
&& で繋がずに、1つ1つ比較してどこでfalseになっているかを調べるのが良いのではないでしょうか。

Comments

[asahina1979]

別アカウントでの質問あたりから指摘入れてるけど、ログイン情報を返してないからノットヌル判定がねー