Zamyka połączenie i zestawienie wreszcie

Question

co jest lepsze dla bloku finally:

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

Lub:

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

Answer 1

lepszy sposób wykorzystania jest 2 jeden, bo jeśli jest wyjątek podczas inicjowania con lub stat, nie zostaną zainicjowane i mogą zostać zainicjowane do null. W takim przypadku użycie pierwszego kodu spowoduje wyświetlenie NullPointerException.

Również, jeśli jesteś już na Java 7, powinieneś rozważyć użycie try-with-resources, który automatycznie zamyka zasoby. Z połączonego samouczka:

Instrukcja try-with-resources zapewnia zamknięcie każdego zasobu na końcu wyciągu. Każdy obiekt implementujący java.lang.AutoCloseable, który zawiera wszystkie obiekty implementujące java.io.Closeable, może być używany jako zasób.

Answer 2

Jeżeli istnieje możliwość, albo jest null, należy sprawdzić, czy. Jeśli taka możliwość nie istnieje, nie ma uzasadnionego powodu, aby to sprawdzić.

Również można dokonać kod nieznacznie lepiej czytelny przez pominięcie niektórych wsporniki pojedynczym stwierdzeniem:

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

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

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

Answer 3

pójdę z drugiej opcji, ale dodanie drugiego zagnieżdżonych finally blok, by upewnić się, że oba con i stat obiekty zaznaczono zbierania śmieci:

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

Żaden z nich są dobre dość. Użyj tego:

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 
      } 
     } 
    } 
} 

i nazwać jak closeQuietly(stat, con);

Albo użyć 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

jako Java 7, nie musisz już więcej używać bloku finallyl aby zamknąć obiekt połączenia lub instrukcji. Zamiast tego możesz skorzystać z nowych funkcji o nazwie "try-with-resources".

Najpierw zadeklarować połączenie i Statament obiektów za pomocą nowej składni dla bloku try-catch następująco:

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

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

przeciwnym razie, nie musisz się martwić, aby zamknąć wyraźnie powiązania z baza danych w bloku finally, ponieważ jvm zrobi to za Ciebie.

mieć ładne kodowanie ....