資源泄漏：流從未關閉

Question

在我的真實項目中重現問題的簡短方法。環境：Android SDK 1.16，Eclipse 4.2.0，Windows。在Save1功能fos = new FileOutputStream(file)

private void Save1(boolean externalStorage) 
{ 
    String s = "12345"; 
    File file; 
    FileOutputStream fos = null; 

    if (externalStorage) 
    { 
     try 
     { 
      file = new File(getExternalFilesDir(null), "log"); 
      fos = new FileOutputStream(file);     // Resource leak: 'fos' is never closed 
     } 
     catch(FileNotFoundException e) 
     { 
      return; 
     } 
    } 
    else 
    { 
     try 
     { 
      fos = openFileOutput("log", Context.MODE_PRIVATE); 
     } 
     catch(FileNotFoundException e) 
     { 
      return; 
     } 
    } 

    try 
    { 
     fos.write(s.getBytes()); 
     fos.close(); 
    } 
    catch(IOException e) 
    { 
     return; 
    } 
} 

private void Save2(boolean externalStorage) 
{ 
    String s = "12345"; 
    File file; 
    FileOutputStream fos = null; 

    try 
    { 
     file = new File(getExternalFilesDir(null), "log"); 
     fos = new FileOutputStream(file);     // OK 
    } 
    catch(FileNotFoundException e) 
    { 
     return; 
    } 

    try 
    { 
     fos.write(s.getBytes()); 
     fos.close(); 
    } 
    catch(IOException e) 
    { 
     return; 
    } 
} 

線，警告：創建默認的Android應用程序，將下面的代碼添加到MainActivity.java Resource leak: 'fos' is never closed

的同一線上Save2功能：無需警告。

請不要發送未經測試的答案，問題並不像看起來那麼簡單。將fos.close()添加到該函數的不同部分並沒有幫助。

Answer 1

它也走，如果我添加一個finally塊的try中，如果塊這樣的：

if (externalStorage) { 
      try { 
       fos = new FileOutputStream(new File(getExternalFilesDir(null), 
         "log")); 
      } catch (FileNotFoundException e) { 
       return; 
      } finally { 
       try { 
        fos.close(); 
       } catch (IOException e) { 
        e.printStackTrace(); 
       } 
      } 
     } else { 
      try { 
       fos = openFileOutput("log", Context.MODE_PRIVATE); 
      } catch (FileNotFoundException e) { 
       return; 
      } 
     } 

它越來越有趣......

所以我的猜測會是，所以如果你在try塊中打開一個Stream並且catch塊有一個return語句，那麼應該有一個finally塊關閉這個流。

類似的東西..

一個嘗試同樣的代碼在Eclipse中一個簡單的Java項目，並仍然得到了警告。所以它看起來是不相關的棉絨或Android。它看起來像eclipse編譯器問題。以下是代碼，我必須創建一個虛擬openFileOutput（）方法，因爲它不可用。n java：

private void Save1(boolean externalStorage) { 
    String s = "12345"; 
    FileOutputStream fos = null; 

    if (externalStorage) { 
     try { 
      fos = new FileOutputStream(new File("c://", "log")); 
     } catch (FileNotFoundException e) { 
      return; 
     } 
    } else { 
     try { 
      fos = openFileOutput("log", -1); 
     } catch (FileNotFoundException e) { 
      return; 
     } 
    } 

    try { 
     fos.write(s.getBytes()); 
     fos.close(); 
    } catch (IOException e) { 
     return; 
    } 
} 

/** 
* @param string 
* @param i 
* @return 
*/ 
private FileOutputStream openFileOutput(String string, int i) 
     throws FileNotFoundException { 
     return null; 
} 

Answer 2

如果是Exception，fos將不會被關閉。將finally添加到try-catch將解決此問題。

try 
    { 
     fos = openFileOutput("log", Context.MODE_PRIVATE); 
    } 
    catch(FileNotFoundException e) 
    { 
     return; 
    } 
    //add this: 
    finally { 
     if (fos != null) { 
      fos.close(); 
     } 
    } 

Answer 3

這不是一個答案，但是爲了清晰起見OP和其他讀者而不是評論添加到這裏。

我已經在IDEA 11.2 API 15中使用當前版本的平臺工具鏈（Rev 14 Oct 2012）對此進行了測試，並且沒有lint警告，編譯錯誤或運行時錯誤。我強制該方法通過創建異常並將useExternalStorage設置爲true和false來遍歷每條路徑。

我的猜測是這是你的工具鏈或可能的Eclipse中的一個lint /編譯錯誤（雖然不太可能，Eclipse自己做這樣的檢查？）。

[編輯]

只是一個想法，（我會測試，但我忘了如何使用Eclipse），但的FileInputStream（文件）可能會引發這將被拋出來的地方在你的調用堆棧一個SecurityException 。如果你抓住它會發生什麼？

[編輯]

這是最接近警告我得到的，而不是在所有有關。我相信，這個警告並不在你身上。