Trong trường hợp nào tài nguyên này sẽ bị rò rỉ?

Question

Eclipse 4 đưa ra một cảnh báo mà nói stmt có thể có khả năng không được đóng cửa và gây ra rò rỉ tài nguyên:

class Test { 
    public void test() { 
     PreparedStatement stmt = null; 
     try { 
      stmt = HibernateSession.instance().connection().prepareStatement(""); 
     } catch (final SQLException e) { 
      e.printStackTrace(); 
     } finally { 
      if (stmt != null) 
       try { 
        stmt.close(); 
       } catch (final SQLException e) { 
        e.printStackTrace(); 
       } 
     } 
    } 
} 

Theo đó bối cảnh đó sẽ xảy ra?

Answer 1

Tôi đoán kết luận ở đây là: đây là lỗi của Eclipse?

Answer 2

Bạn cần phải sử dụng thử Java 7 với tài nguyên hoặc thử-cuối cùng khối:

try(stmt = HibernateSession.instance().connection().prepareStatement("")) { 


} 

Cảnh báo này sẽ được tạo ra trên các loại mà kế thừa từ AutoCloseable mà không đảm bảo được đóng lại. (Hoặc có thể Closeable, tôi quên cái nào).

Bây giờ tôi thấy những gì bạn đang yêu cầu, chỉ cần viết mã ít phức tạp hơn.

Foo f = null; // don't do this, but it's what you're doing 
f = new Foo(); 

Là những gì bạn đang làm và bạn đã tìm thấy một trong nhiều tình huống mà bạn thực sự phải trả tiền phạt cho công việc không liên quan này.

Ngoài ra, try/finally của bạn phải sạch sẽ. .close() không thể ném, tại sao bạn lại bị bắt?

try { // don't do this 
    stmt.close(); 
} 
catch(SQLException exc) { 

} 

Nên tạo cảnh báo Eclipse cho bạn biết rằng bạn đang bắt thứ gì đó không bị ném. Đó thậm chí có thể là một lỗi biên dịch, không chắc chắn, nhưng có vẻ như bạn sẽ được hưởng lợi từ việc chơi với Eclipse> Preferences> Compiler và xem lại những cảnh báo nào là thông minh. Nếu bạn không hiểu cảnh báo, hãy google và xem nó có hữu ích cho bạn hay không, đừng bỏ qua nó. (giống như bạn đã làm với cái này).

Answer 3

Có thể xảy ra rò rỉ nếu xảy ra trường hợp ngoại lệ khi bạn gọi stmt.close() trong khối finally.

Answer 4

Vấn đề là trong khối finally ngoại lệ của bạn có khả năng xảy ra, điều này có thể ngăn không cho stmt đóng.

Một workaround là bạn có thể thay thế tất cả mọi thứ trong khối finally với:

JDBCUtilities.close(stmt); 

Xem docs cho JDBCUtilities.close. Như bạn có thể thấy, không có ngoại lệ nào được ném bằng phương thức tiện ích này, do đó bạn không cần phải lo lắng về việc rò rỉ tài nguyên. Một lợi ích nữa là phương pháp tiện ích cũng xử lý trường hợp null cho số stmt cũng như vậy chúng tôi không cần tự viết mã.

Thực ra, nó là good practice để sử dụng JDBCUtilities.