Где закрыть результирующий набор

Question

Я получаю набор результатов уже закрытым, когда я передаю результаты в другой метод. Где я должен закрыть набор результатов.

public void mainTest(){ 

ResultSet rs= pstmt.executeQuery(query); 

List list = populateRS(rs); 

if(rs!=null)rs.close(); 
} 

public List populateRS(ResultSet rs){ 

//work with result set 
if(rs!=null)rs.close(); 
} 

Answer 1

Вы должны, вероятно, использовать try-finally блок, чтобы закрыть ResultSet даже если populateRS (или что-то другое) бросает исключение:

ResultSet rs; 
try { 
    rs = pstmt.executeQuery(query); 
    List list = populateRS(rs); 
} finally { 
    if (rs != null) { 
     rs.close(); 
    } 
} 

Answer 2

Закройте предметы рядом с тем местом, где вы их открываете. В этом случае это будет в методе mainTest после вызова populateRS. Если метод ничего не открывает, он не должен закрывать его.

Answer 3

Вы должны закрыть ResultSet в методе mainTest отделить участие populateRS метод в жизненном цикле ResultSet.

У вас должен быть блок finally, в котором вы должны закрыть ResultSet. Эта практика дает вам гарантию, что ResultSet закрыт, даже если выбрано исключение.

Answer 4

Закрыть в том же методе вы открываете, если это вообще возможно. Последовательное выполнение этого упрощает, чтобы разработчики и разработчики кода легко сортировали ресурсы (явно освобожденные, явно проблематичные, и нуждались в большем внимании).

Несколько другие ноты:

1. Использование try (...) или сделать закрытие в finally поэтому ресурс закрыт, даже если код, используя его не удается с исключением.
2. Используйте аннотации @WillClose and @WillNotClose, если необходимо, чтобы IDE и инструменты, такие как findbugs, могли указывать на проблемы.

---

public void mainTest(){ 
    List<?> list; 
    try (ResultSet rs = pstmt.executeQuery(query)) { 
    list = populateRS(rs); 
    } 
    // work with list 
} 

public List<?> populateRS(@WillNotClose ResultSet rs){ 
    //work with result set 
} 

или если вы застряли со старыми Java:

public void mainTest(){ 
    List<?> list; 
    ResultSet rs = pstmt.executeQuery(query); 
    try { 
    list = populateRS(rs); 
    } finally { 
    if(rs!=null)rs.close(); 
    } 
    // work with list 
} 

Answer 5

Это хорошо, чтобы закрыть, где вы открываете.
Это хорошая практика программирования, чтобы закрыть все ресы в наконец блок

 public void mainTest() 
     { 
     ResultSet rs = null; 
     try{ 
       rs= pstmt.executeQuery(query); 
      List list = populateRS(rs); 
      }finally{ 
       try { 
       rs.close(); 
       } catch (SQLException ex) { 

       } 
      } 
     } 

    public List populateRS(ResultSet rs){ 

     //work with result set 

    } 

в соответствии с Явы docs

Ввод кода очистки в окончательно блоке всегда хорошая практика, даже если никаких исключений не ожидается.

Answer 6

Используйте новый try-with-resources statement, который будет автоматически закрывать ResultSet происходит ли исключение или нет, потому что он реализует AutoCloseable.

Оператор try-with-resources - это оператор try, который объявляет один или несколько ресурсов.Ресурс - это объект, который должен быть закрыт после завершения программы. Оператор try-with-resources гарантирует, что каждый ресурс будет закрыт в конце инструкции.

public void mainTest() 
{ 
    try (ResultSet rs = pstmt.executeQuery(query)) { 
    List list = populateRS(rs); 
    } catch (SQLException ex) { 

    } 
} 

public List populateRS(ResultSet rs){ 
    // work with result set 
}