Dlaczego ten kod generuje ostrzeżenie o "potencjalnym wycieku zasobów"?

Question

Eclipse (Juno) daje następujące ostrzeżenie:

Potencjalny przeciek zasobu: 'os' nie mogą być zamknięte

w pierwszej linii ciała try w tym kodzie:

static void saveDetails(byte[] detailsData) { 
    OutputStream os = null; 
    try { 
     os = sContext.openFileOutput(DETAILS_FILE_NAME, Context.MODE_PRIVATE); 
     os.write(detailsData); 
    } catch (IOException e) { 
     Log.w(LOG_TAG, "Unable to save details", e); 
    } finally { 
     if (os != null) { 
      try { 
       os.close(); 
      } catch (IOException ignored) { 
      } 
     } 
    } 
} 

Metoda openFileOutput jest zadeklarowana jako rzut FileNotFoundException.

Czy to jest fałszywy alarm? Wydaje się, że jest to całkiem waniliowa analiza ścieżki wykonania.

Answer 1

Moim zdaniem jest to fałszywy alarm. Twój zasób jest zamknięty w "końcu" bloku, więc nie widzę, co może pójść nie tak.

Jako sidenote, jeśli korzystasz z Java 7, polecam użycie idiomu "try-with-resources".

static void saveDetails(byte[] detailsData) {  
    try (OutputStream os = sContext.openFileOutput(DETAILS_FILE_NAME, Context.MODE_PRIVATE);) { 
     os = sContext.openFileOutput(DETAILS_FILE_NAME, Context.MODE_PRIVATE); 
     os.write(detailsData); 
    } catch (IOException e) { 
     Log.w(LOG_TAG, "Unable to save details", e); 
    } 
} 

Answer 2

Zgaduję, że to dlatego, że przed zamknięciem masz if (os != null). Ponieważ jest to warunkowe, możliwe jest, że OutputStream nie jest zamknięty.

Co się dzieje, jeśli spróbujesz:

static void saveDetails(byte[] detailsData) { 
    OutputStream os = null; 
    try { 
     os = sContext.openFileOutput(DETAILS_FILE_NAME, Context.MODE_PRIVATE); 
     os.write(detailsData); 
    } catch (IOException e) { 
     Log.w(LOG_TAG, "Unable to save details", e); 
    } finally { 
     try { 
      os.close(); 
     } catch (IOException ignored) { 
     } 
    } 
} 

Answer 3

Co jeśli przenieść otwarte i blisko pierwszej klauzuli try? Rzucają ten sam wyjątek. I usuń jeśli os! = Null.