java.lang.NullPointerException ... 初始化 JDBC MYSQL 连接

Question

请不要建议我使用 InternalFrame 或 Dialogs。我无法从头开始项目。 主题：我正在构建一个 GUI 程序来显示标记表。我参加了 3 个 JFrames 和 1 个简单的课程...

Frame1.java

它有 1 个 JTextField 可以输入 roll_no。 & 2 个按钮在 DB & showResult 中提供数据。 feedData 按钮调用 Frame2 和 showResult 按钮调用 Frame3。

Frame2.java

为了提供数据，有几个 JTextFields & Buttons 可以将内容传输到 mySQL DB。

Frame3.java

是一个从数据库中获取内容的结果窗口。

Support.java

包含静态变量和 getter-setter 方法

 .....
 .....//contains in Support.java
     public boolean add() {
    query = "Insert into table1 (enroll,Sname,Fname,sub1,sub2,sub3,sub4,sub5 )values(?,?,?,?,?)";
    try {
        PreparedStatement psmt = conn.prepareStatement(query);
        psmt.setString(1, enroll);
        psmt.setString(2, Sname);
        psmt.setString(3, Fname);
        psmt.setInt(4, sub1);
        psmt.setInt(5, sub2);
        psmt.setInt(6, sub3);
        psmt.setInt(7, sub4);
        psmt.setInt(8, sub5);
        int y = 0;
        y = psmt.executeUpdate();
        if (y == 0) {
            return false;
        }
    } catch (Exception e) {
        e.printStackTrace();
        return false;
    }
    return true;
}

add() 在 Frame2.java 中按下保存按钮时调用。 . .如果 catch 块正在执行，为什么 println(query) 打印 NULL

Answer 1

根据您在 cmets 中对其他答案的一些问题标签和响应以及问题本身，我假设您打算在代码中的某个地方调用

Class.forName("com.mysql.jdbc.Driver").newInstance();
conn = DriverManager.getConnection(url, username, password);

在调用 add() 方法之前不会发生这种情况。为了修复它，我建议这样做（从 Vivek bhatnagar 的答案中借用的大量代码）：

try {
    conn = DriverManager.getConnection(url, username, password);
    PreparedStatement pstmt = conn.prepareStatement("INSERT INTO `table`
        (pid,tid,rid,tspend,description) VALUE 
        (?,?,?,?,?)");
    pstmt.setString(1, pid );
    pstmt.setString(2, tid);
    pstmt.setString(3, rid);
    pstmt.setInt(4, tspent);
    pstmt.setString(5,des );
    pstmt.executeUpdate();
} catch (Exception e) {
    // whatever you want to do to handle the exception
} finally {
    // close your connection
}

如果您使用的是 Java 7，请按如下方式设置：

try (Connection conn = DriverManager.getConnection(url, username, password)) {
    try (PreparedStatement pstmt = conn.prepareStatement(/*sql here*/)) {
        // Your code here
    } catch (SQLException sqle) {
        // handle exceptions from the statement
    }
} catch (SQLException outerSqlEx) {
    // handle exceptions from connecting
}

我怎么知道你的问题是什么（NullPointerException 的一般帮助）？

NullPointerException 仅在您尝试调用 null 变量上的方法时抛出（以及在其他几个特定时间，如 API 文档中所述）。找到NullPointerException 的简单方法是查找堆栈跟踪指示的行，然后查找行上的点。 try 块中只有两行可以抛出 NullPointerException。

Statement stmt = conn.createStatement();
// could be here ----^ 

和

y = stmt.executeUpdate(query);
// or --^

让我们看看这些点。当conn 是null 时，第一个将抛出。当stmt 是null 时，第二个将抛出。在您现在已针对其他答案进行编辑的原始代码中，您在调用conn.createStatement(); 后设置了query 的值。由于query 在你的catch 块中仍然为空，我们知道它尚未设置，因此它必须是第一个，所以conn 在程序中的那个点是null。

此外，由于API Documentation for createStatement 意味着它要么返回一个有效的Connection 对象，要么抛出一个SQLException，我们可以确定stmt 在调用executeUpdate 时永远不会是null。

Answer 2

在您的 try 块中，您在设置相关变量之前调用可能引发异常的方法：

    Statement stmt = conn.createStatement();
    query = "Insert into table1 (enroll,Sname,Fname,sub1,sub2,sub3,sub4,sub5 )values('" + getEnroll() + "','" + getSname() + "','"+getFname()+"',"+getSub1()+","+getSub2()+","+getSub3()+","+getSub4()+","+getSub5()+")";

因此，如果您的代码在 conn.createStatement() 行上失败，它将进入 catch 块，而不会初始化查询变量。

您可以简单地通过切换语句的顺序来解决此问题，或者将查询行放在 try/catch 块之外和之前。

Answer 3

补充@Southpaw 的回答：

你也可以使用这样的东西：

PreparedStatement pstmt = con.prepareStatement("INSERT INTO `table`
        (pid,tid,rid,tspend,description) VALUE 
        (?,?,?,?,?)");
pstmt.setString(1, pid );
pstmt.setString(2, tid);
pstmt.setString(3, rid);
pstmt.setInt(4, tspent);
pstmt.setString(5,des );
pstmt.executeUpdate();

请注意它的好处：

1."Query is rewritten and compiled by the database server"

If you don't use a prepared statement, the database server will have to parse, and compute an execution plan for the statement each time you run it. If you find that you'll run the same statement multiple times (with different parameters) then its worth preparing the statement once and reusing that prepared statement. If you are querying the database adhoc then there is probably little benefit to this.

2."Protected against SQL injection"

This is an advantage you almost always want hence a good reason to use a PreparedStatement everytime. Its a consequence of having to parameterize the query but it does make running it a lot safer. The only time I can think of that this would not be useful is if you were allowing adhoc database queries; You might simply use the Statement object if you were prototyping the application and its quicker for you, or if the query contains no parameters.