潜在的空指针访问。变量在此位置可能为空。Java语言lang.NullPointerException[重复]

Question

我知道关于这个错误有很多类似的话题，但我尝试了一些建议，我的问题仍然没有解决。我遵循本教程：https://www.youtube.com/watch?v=B3gEbC37DAM

这是我的代码：

public class JdbcDaoImpl {    
  public Circle getCircle(final int circleId) {
    Connection conn = null;

    try {
        String driver = "org.apache.derby.jdbc.ClientDriver";
        Class.forName(driver).newInstance();
        conn = DriverManager.getConnection("jdbc:derby//localhost:1527//db");
        PreparedStatement ps = conn.prepareStatement("SELECT * FROM circle where id= ?");
        ps.setInt(1, circleId);

        Circle circle = null;
        ResultSet rs = ps.executeQuery();
        if (rs.next()) {
            circle = new Circle(circleId, rs.getString("name"));
        }
        rs.close();
        ps.close();
        return circle;
    } catch (Exception e) {
        throw new RuntimeException(e);

    } finally {
        try {
            conn.close();

        } catch (SQLException e) {
        }

    }
  }
}

关于“conn.close（）；”它说：潜在的空指针访问。变量在此位置可能为空。当我运行程序时，我有一个错误：线程“main”java中的异常。lang.NullPointerException

我尝试了我在类似主题上看到的解决方案，例如：

if(conn!=null){
    conn.close();
}

但我仍然有错误。

提前感谢您的帮助！

Answer 1

连接可以为空的原因很简单。如果在连接之前执行的代码被分配，即在该代码成功运行之前：

String driver = "org.apache.derby.jdbc.ClientDriver";
Class.forName(driver).newInstance();
conn = DriverManager.getConnection("jdbc:derby//localhost:1527//db");

此代码必须成功运行才能使conn不为null，因此在对其调用close（）之前需要进行非null检查。

Answer 2

您不需要显式加载驱动程序（Class.forName），假设您正在使用Java

您可以使用try-with-resources来管理自动关闭的对象（Connection、PreparedStatement和ResultSet），因此您的代码可以写成：

public Circle getCircle(final int circleId) {
    try (Connection conn = DriverManager.getConnection("jdbc:derby//localhost:1527//db");
         PreparedStatement ps = conn.prepareStatement("SELECT * FROM circle where id= ?")){
      ps.setInt(1, circleId);

      Circle circle = null;
      try (ResultSet rs = ps.executeQuery()) {
        if (rs.next()) {
          circle = new Circle(circleId, rs.getString("name"));
        }
      }
      return circle;
    } catch (Exception e) {
      throw new RuntimeException(e);
    }
  }

这也保证了所有资源都被正确关闭，从而解决了许多因资源泄漏而可能出现的问题。

问题是getConnection可能会引发异常，将conn保留为null，这将导致在稍后尝试在finally块中关闭它时出现null指针。

Answer 3

最后，块总是执行，而你正在使用try catch，你可以得到一个潜在的NullPointerException是当conn=DriverManager.get连接（"jdbc： derby//localhost:1527//db"）；抛出NullPointerException，这是由你的捕获（Exception e），然后最后块开始执行，你做conn.close（），但你的conn对象是NULL再次抛出NullPointerException，这是不被捕获的，因为你正在捕获<>cat（SQLException）