Đóng kết nối và tuyên bố cuối cùng

Question

Đó là tốt hơn cho khối finally:

finally { 
     try { 
      con.close(); 
      stat.close(); 
     } catch (SQLException sqlee) { 
      sqlee.printStackTrace(); 
     } 
    } 

Hoặc:

finally { 
     try { 
      if (con != null) { 
       con.close(); 
      } 
      if (stat != null) { 
       stat.close(); 
      } 
     } catch (SQLException sqlee) { 
      sqlee.printStackTrace(); 
     } 
    } 

Answer 1

Better cách để sử dụng là một trong 2, bởi vì nếu một ngoại lệ được ném khi khởi tạo con hoặc stat, chúng sẽ không được khởi tạo và có thể được khởi tạo thành null. Trong trường hợp đó, sử dụng mã số 1 sẽ ném NullPointerException.

Ngoài ra, nếu bạn đã ở trên Java 7, bạn nên cân nhắc sử dụng try-with-resources, thao tác này sẽ tự động đóng tài nguyên. Từ hướng dẫn được liên kết:

Tuyên bố try-with-resources đảm bảo rằng mỗi tài nguyên được đóng ở cuối báo cáo. Bất kỳ đối tượng nào triển khai thực hiện java.lang.AutoCloseable, bao gồm tất cả các đối tượng thực hiện java.io.Closeable, có thể được sử dụng như một tài nguyên.

Answer 2

Nếu có một khả năng hoặc là null, bạn phải kiểm tra xem. Nếu khả năng không tồn tại, không có lý do hợp lệ để kiểm tra nó.

Ngoài ra, bạn có thể làm cho mã của bạn hơi có thể đọc được tốt hơn bằng cách bỏ qua một số dấu ngoặc đơn tuyên bố:

finally { 
    try { 
     if (con != null) 
      con.close(); 

     if (stat != null) 
      stat.close(); 

    } catch (SQLException sqlee) { 
     sqlee.printStackTrace(); 
    } 
} 

Answer 3

tôi sẽ đi với tùy chọn thứ hai, nhưng thêm một giây lồng nhau finally khối, chỉ để chắc chắn rằng cả hai con và stat đối tượng được đánh dấu để thu gom rác thải:

finally { 
    try { 
     if(con != null) 
      con.close(); 
     if(stat != null) 
      stat.close(); 
    } catch(SQLException sqlee) { 
     sqlee.printStackTrace(); 
    } finally { // Just to make sure that both con and stat are "garbage collected" 
     con = null; 
     stat = null; 
    } 
} 

Answer 4

Không ai trong số họ là tốt đủ. Sử dụng điều này:

public static void closeQuietly(AutoCloseable ... closeables) { 
    for (AutoCloseable c : closeables) { 
     if (c != null) { 
      try { 
       c.close(); 
      } catch (Exception e) { 
       // log or ignore, we can't do anything about it really 
      } 
     } 
    } 
} 

Và gọi nó như closeQuietly(stat, con);

Hoặc sử dụng java 7 try-with-resource:

List<String> results = new ArrayList<>(); 
    try (Statement statement = conn.createStatement(); 
     ResultSet rs = statement.executeQuery(query)) { 

     int numberOfColumns = getColumnCount(rs); 
     while (rs.next()) { 
      int i = 1; 
      while (i <= numberOfColumns) { 
       results.add(rs.getString(i++)); 
      } 
     } 
    } 

Answer 5

Tính đến Java 7, bạn không cần nữa sử dụng khối finallyl để đóng một đối tượng Connection hoặc Statement. Thay vào đó, bạn có thể sử dụng các tính năng mới được gọi là 'try-with-resources'.

Trước tiên, bạn khai báo một kết nối và Statament đối tượng bằng cách sử dụng cú pháp mới cho một khối try-catch như sau:

try(Connection con = DriverManager.getConnection(database-url, user, password); Statement st = conn.createStatement()) { 

//your stuffs here 
} catch (SQLException e) { 
    e.printStackTrace(); 
}  

Làm như vậy, bạn sẽ không cần phải lo lắng để đóng một cách rõ ràng mối liên hệ với cơ sở dữ liệu trong một khối cuối cùng vì jvm sẽ làm điều đó cho bạn.

Có đẹp mã hóa ....