資源泄漏：'in'永遠不會關閉，儘管它已關閉

Question

我知道這裏有幾個類似的有問題的問題，但其中大多數人已經忘記在其流上放置close()指令。這裏有所不同。

可以說我有以下的小例子：

public void test() throws IOException 
{ 
    InputStream in; 
    if(file.exists()) 
    { 
     in = new FileInputStream(file); 
    } 
    else 
    { 
     in = new URL("some url").openStream(); 
    } 
    in.close(); 
} 

這給我在Eclipse（朱諾SR1）一Resource leak: 'in' is never closed警告。 但是當我移動到in.close()的條件塊，警告消失：

public void test() throws IOException 
{ 
    InputStream in; 
    if(file.exists()) 
    { 
     in = new GZIPInputStream(new FileInputStream(file)); 
     in.close(); 
    } 
    else 
    { 
     in = new URL("some URL").openStream(); 
    } 
} 

這到底是怎麼回事？

Answer 1

以下是我會寫：

public void test() throws IOException 
{ 
    InputStream in = null; 
    try { 
     if(file.exists()) { 
      in = new FileInputStream(file); 
     } else { 
      in = new URL("some url").openStream(); 
     } 
     // Do something useful with the stream. 
    } finally { 
     close(in); 
    } 
} 

public static void close(InputStream is) { 
    try { 
     if (is != null) { 
      is.close(); 
     } 
    } catch (IOException e) { 
     e.printStackTrace(); 
    } 
} 

Answer 2

我懷疑警告是不正確的。它可能會檢查您是否在相同範圍內關閉了流。在第二種情況下，您沒有關閉第二個流。

Answer 3

因爲IO異常，則可能會遇到資源泄漏（poentially）

試着做以下幾點：

public void test() throws IOException 
{ 
    InputStream in= null; 
    try { 
     if(file.exists()) 
     { 
      // In this case, if the FileInputStream call does not 
      // throw a FileNotFoundException (descendant of IOException) 
      // it will create the input stream which you are wrapping 
      // in a GZIPInputStream (no IO exception on construction) 
      in = new GZIPInputStream(new FileInputStream(file)); 
     } 
     else 
     { 
      // Here however, if you are able to create the URL 
      // object, "some url" is a valid URL, when you call 
      // openStream() you have the potential of creating 
      // the input stream. new URL(String spec) will throw 
      // a MalformedURLException which is also a descendant of 
      // IOException. 
      in = new URL("some url").openStream(); 
     } 

     // Do work on the 'in' here 
    } finally { 
     if(null != in) { 
      try 
      { 
       in.close(); 
      } catch(IOException ex) { 
       // log or fail if you like 
      } 
     } 
    } 
} 

做上述將確保你已經關閉了流或至少盡最大努力這樣做。

在您的原始代碼中，您聲明瞭InputStream但從未初始化過。首先這是一種糟糕的形式。如上所示，將其初始化爲null。我的感覺是，我目前沒有運行Juno，因爲它看到InputStream'in'，可能會讓它穿過所有的障礙和障礙，以達到您要使用它的地步。不幸的是，正如有人指出的，你的代碼對於一個例子來說有點不妥。這樣做，因爲我已經詳細以及@duffymo你會擺脫警告。

Answer 4

如果文件不存在並且您嘗試關閉不存在的文件，那麼您的流中可能無法初始化。

你的第二個例子也需要一個關閉語句以避免泄漏。

Answer 5

這同一個Eclipse的報告可能發生，當你明確地拋出一個異常，你已經打開了你的資源之後，如：

public void method() throws IOException { 
    BufferedReader br = new BufferedReader(new FileReader("myfile.txt")); 
    while (br.ready()) { 
     String line = br.readLine(): 
     if (line.length() > 255) { 
     throw new IOException("I am some random IOException"); 
     } 
    } 
    br.close(); 
} 

這是一些做作用於演示目的的代碼，所以不要太難。

如果有人註釋掉該行，警告消失。當然，你反而希望確保資源被正確關閉。你可以這樣做：

if (line.length() > 255) { 
    br.close(); 
    throw new IOException("I am some random IOException"); 
} 

雖然在這種情況下不要依賴Eclipse警告。養成使用try/finally方法的習慣，確保資源正確並始終關閉。

Answer 6

我有類似：

InputStream content = httpResponse.getEntity()==null?null:httpResponse.getEntity().getContent(); 

，讓同一warrning。但如果我離開它就像這樣：

InputStream content =httpResponse.getEntity().getContent(); 

我沒有收到任何warrnings。不奇怪或者是什麼？

- 我希望我的信息是增加知識的原始問題。謝謝！