0

準備されたステートメントが null を返す理由を教えてください。データベースにクエリを実行するコードは次のとおりです。

package beans;
import java.sql.*;
public class ConnectToDB {
private Connection connect;

private final String url = "jdbc:mysql://localhost/";
private final String DBuser = "root";
private final String DBpass = "root";
private final String DBname = "reservation";

private final String Driver = "com.mysql.jdbc.Driver";
public ConnectToDB(){
    try{
        Class.forName(this.Driver);
        this.connect = DriverManager.getConnection(this.url+this.DBname, this.DBuser, this.DBpass);
    }catch(  ClassNotFoundException | SQLException e){ e.printStackTrace(); }
}
public Connection getConnection(){
    return this.connect;
}

private String get_user_info(String username,int index){
    /*
     * user_info[0] = user_id;
     * user_info[1] = username;
     * user_info[2] = password;
     * user_info[3] = firstname;
     * user_info[4] = middle_name;
     * user_info[5] = lastname;
     * user_info[6] = client_rights;
     * 
     */
    String user_info[] = new String[7];
    PreparedStatement pstmt = null;
    ResultSet rset = null;
    String query = null;
    try{
        query = "Select * from user where username = ?";
        pstmt = this.connect.prepareStatement(query);
        pstmt.setString(1, username);
        rset = pstmt.executeQuery();
        while(rset.next()){
            int user_id = rset.getInt("user_id");
            user_info[0] = String.valueOf(user_id);
            user_info[1] = rset.getString("username");
            user_info[2] = rset.getString("password");
            user_info[3] = rset.getString("firstname");
            user_info[4] = rset.getString("middle_name");
            user_info[5] = rset.getString("lastname");
            user_info[6] = rset.getString("user_rights");
        }
    }catch(SQLException e){ e.printStackTrace(); }
    finally{
        try{
            pstmt.close();
            rset.close();
        }catch(SQLException e){ e.printStackTrace(); }
    }
    return user_info[index];
}

public int getUserId(String username){
    String user_id_from_db = get_user_info(username, 0);
    int user_id = Integer.parseInt(user_id_from_db);
    return user_id;
}

public String getUsername(String username){ return get_user_info(username, 1); }
public String getPassword(String username){ return  get_user_info(username, 2); }
public String getFirstname(String username){ return get_user_info(username, 3); }
public String getMiddlename(String username){ return get_user_info(username, 4); }
public String getLastname(String username){ return  get_user_info(username, 5); }
public String getUserRights(String username){ return get_user_info(username, 6); }

public boolean userExists(String username){
    boolean queryStatus = false;
    if(username.equalsIgnoreCase(getUsername(username)))
        queryStatus = true;
    else
        queryStatus = false;
    return queryStatus;
}   
}

次に、クエリを呼び出すコードを次に示します。

<jsp:useBean id="user" class="beans.ConnectToDB" scope="session" />
<jsp:useBean id="aes" class="beans.AES" scope="session" />

String getUsername = request.getParameter("username");
        String getPassword = request.getParameter("password");  

        final String passphrase = "#asdf@1234#";    
        byte[] password_byte = getPassword.getBytes();  
        byte[] passphrase_byte = passphrase.getBytes();
        byte[] encrypt_password = aes.encrypt(password_byte, passphrase_byte);      

        if((getUsername != null && !getUsername.isEmpty()) || (getPassword != null && !getPassword.isEmpty())){
            String username_from_db = user.getUsername(getUsername);
            String password_from_db = user.getPassword(getUsername);

            byte[] pass_db_byte = password_from_db.getBytes();
            byte[] encrypted_pass_db = aes.encrypt(pass_db_byte, passphrase_byte);

            if(getUsername.equalsIgnoreCase(username_from_db) && encrypt_password.equals(encrypted_pass_db)){
                response.sendRedirect("home_page.jsp");
            }
        }
        else{ response.sendRedirect("index.jsp"); }

メソッドを呼び出すgetUsername(String username)と、null が返されます。スローされる例外は次のとおりです。

org.apache.jasper.JasperException: An exception occurred processing JSP page /authenticate_user.jsp at line 29
26:             byte[] encrypt_password = aes.encrypt(password_byte, passphrase_byte);      
27:             
28:             if((getUsername != null && !getUsername.isEmpty()) || (getPassword != null && !getPassword.isEmpty())){
29:                 String username_from_db = user.getUsername(getUsername);
30:                 String password_from_db = user.getPassword(getUsername);
31: 
32:                 byte[] pass_db_byte = password_from_db.getBytes();


Stacktrace:
org.apache.jasper.servlet.JspServletWrapper.handleJspException(JspServletWrapper.java:568)
org.apache.jasper.servlet.JspServletWrapper.service(JspServletWrapper.java:470)
org.apache.jasper.servlet.JspServlet.serviceJspFile(JspServlet.java:390)
org.apache.jasper.servlet.JspServlet.service(JspServlet.java:334)
javax.servlet.http.HttpServlet.service(HttpServlet.java:728)

root cause

java.lang.NullPointerException
beans.ConnectToDB.get_user_info(ConnectToDB.java:61)
beans.ConnectToDB.getUsername(ConnectToDB.java:72)
org.apache.jsp.authenticate_005fuser_jsp._jspService(authenticate_005fuser_jsp.java:110)
org.apache.jasper.runtime.HttpJspBase.service(HttpJspBase.java:70)
javax.servlet.http.HttpServlet.service(HttpServlet.java:728)
org.apache.jasper.servlet.JspServletWrapper.service(JspServletWrapper.java:432)
org.apache.jasper.servlet.JspServlet.serviceJspFile(JspServlet.java:390)
org.apache.jasper.servlet.JspServlet.service(JspServlet.java:334)
javax.servlet.http.HttpServlet.service(HttpServlet.java:728)
4

2 に答える 2

1

あなたはあまりにも多くのものを投稿しました。

必要なのはこれだけです：

beans.ConnectToDB.get_user_info(ConnectToDB.java:61)

ConnectToDB.java をテキスト エディターで開き、61 行目に移動して、その行のすべてのオブジェクト参照を調べます。そのうちの 1 つは null です。これは、適切に初期化しなかったか、取得したときに常に null 以外になると想定していたためです。どれを見つけて、適切に初期化してください。問題が解決しました。

これは良い解決策ではないと思います。JSP がある場合は、サーブレット/JSP エンジンがあり、JNDI データベース接続プールとネーム サービスが必要です。それを設定し、データベース接続パラメーターを外部化する必要があります。それらはあなたのコードには属していません。プールは、接続をより適切に管理します。

リソースも適切に閉じていません。それらは、作成とは逆の順序で、finally ブロックで閉じ、個々の try/catch ブロックでラップする必要があります。それを行うために呼び出すことができる静的ユーティリティメソッドを作成します。

package persistence;

public class DatabaseUtils {
    private DatabaseUtils() {}

    // Similar for ResultSet and Connection
    public static void close(Statement st) {
        try {
            if (st != null) {
                st.close();
            }
        } catch (Exception e) {
            // Log the exception
        }
    }
}
于 2013-05-31T12:17:27.200 に答える