Closing BufferedReader和InputStreamReader

Question

这段代码正在创建内存泄漏问题原因BufferedReader和InputStreamReader，我认为这可能会导致一些例外。我应该如何改变它？

try{ 
    URL url = new URL(sMyUrl); 
    BufferedReader in = new BufferedReader(new InputStreamReader(url.openStream())); 
    while ((str = in.readLine()) != null) { 
     jsonString += str; 
    } 
    in.close(); 
}catch(Exception e){ 

} 

Answer 1

使用try..finally块关闭流会更安全。您也可以使用StringBuilder，因为它设计用于连接字符串。你也应该避免捕捉Exception，并且无所作为。另外，你的代码连接了没有换行符的行。这很可能不是你想要什么，在这种情况下append("\n")当你阅读每一行

这里的这些修改版本：

StringBuilder json = new StringBuilder(); 
try { 
    URL url = new URL(sMyUrl); 
    BufferedReader in = new BufferedReader(new InputStreamReader(url.openStream())); 
    try { 
     String str; 
     while ((str = in.readLine()) != null) { 
      json.append(str).append("\n"); 
     } 
    } finally { 
     in.close(); 
    } 
} catch (Exception e) { 
    throw new RuntimeException("Failed to read JSON from stream", e); 
} 

Answer 2

该代码并不漂亮，但不会造成内存泄漏。我建议你使用内存分析器来确定你的内存在哪里使用。否则，你只是猜测，即使你有10年以上的经验性能调优在Java中;）

更好的选择是使用Java 7

URL url = new URL(sMyUrl); 
try(BufferedReader in = new BufferedReader(new InputStreamReader(url.openStream()))) { 
    while ((str = in.readLine()) != null) { 
    jsonString.append(str).append("\n"); 
    } 
} 

如果你的Java 6或以上就可以使用。

BufferedReader in = new BufferedReader(new InputStreamReader(url.openStream()))) { 
try { 
    while ((str = in.readLine()) != null) { 
    jsonString.append(str).append("\n"); 
    } 
} finally { 
    in.close(); 
}