Schließen BufferedReader und InputStreamReader

Dieses Stück code ist das erstellen der memory-leak-Probleme Ursache von BufferedReader und InputStreamReader was ich denke, geschehen könnte Ursache einige Ausnahmen. Wie sollte ich es ändern? 

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){

}
  • Möglicherweise bewegen Sie die close () - Logik finally-block? Sind Sie sicher, dass keine Ausnahmen geschieht während des Lesens (oder) die Verbindung zu schließen?
  • Habe versucht, schriftlich e.printStackTrace() in Ihrer catch-Klausel, um zu sehen, ob Ausnahmen geworfen?
InformationsquelleAutor Pit Digger | 2012-08-30
  1. 12

    Es wäre sicherer, schließen Sie Ihren stream mit einem try..finally block. Sie könnte auch ein StringBuilder wie es ist entworfen, für die Verkettung von strings. Sie sollten auch vermeiden, fangen Exception und nichts zu tun mit es. Auch der code ist die Verkettung der Zeilen ohne Zeilenumbrüche. Dies kann auch nicht sein, was du willst, in dem Fall append("\n") wenn Sie Lesen, jede Zeile.

    Hier eine version mit den änderungen:

    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);
}
    • Brauche ich, um in der Nähe InputStreamReader ? Oder Ihr geschlossen als Puffer reader wird ein neues Objekt erstellen?
    • Keine Notwendigkeit, explizit schließen Sie das InputStreamReader wie es wird automatisch geschlossen, wenn Sie in der Nähe der BufferedReader.
    • danke. Jedenfalls brauche ich ein gutes Java-streams tutorial.
    • Ich habe in einigen tutorials, die Ressource geprüft wird, für null-ness vor dem schließen. Aber wenn ich versuche, dass es mich bat, legte Sie in einem anderen try-catch-block. warum ist das so? und ist es wichtig, für null-ness vor dem schließen einer Ressource
    • Kasun, schlage ich vor, Sie buchen eine eigene Frage mit einem Beispiel.
    • Diese Antwort ist falsch, da es gar nicht kompilieren (zumindest in java7). Variable BufferedReader in erklärt wird, in den Geltungsbereich des try block und ist nicht sichtbar im Rahmen finally block. Die variable kann auch nicht vergeben werden, im Falle MalformedURLException geworfen wird aus der URL-Konstruktor der Klasse.
    • Nein, das ist eine Fehlinterpretation. Es gibt zwei try Blöcke. Die variable in ist nicht definiert innerhalb der try block, schließt es.
    • Du hast Recht. Muss etwas falsch sein mit mir dann. Sorry für die Verwirrung.

    InformationsquelleAutor ᴇʟᴇvᴀтᴇ
  2. 9

    Der code ist nicht hübsch, aber wird nicht erstellen Sie einen Speicherverlust. Ich schlage vor, Sie verwenden einen memory-profiler, um festzustellen, wo der Speicher verwendet wird. Ansonsten sind Sie nur raten, auch wenn Sie zehn + Jahre Erfahrung in performance tuning in Java 😉

    Eine bessere alternative ist die Verwendung von 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");
  }
}

    Wenn Sie Java 6 oder älter, die Sie verwenden können.

    BufferedReader in = new BufferedReader(new InputStreamReader(url.openStream()))) {
try {
  while ((str = in.readLine()) != null) {
     jsonString.append(str).append("\n");
  }
} finally {
  in.close();
}
    • Es gibt absolut keinen Vorteil bei der Einstellung in = null und dann null in der finally block. Legen Sie einfach die try sofort, nachdem Sie konstruieren die BufferedReader.
    • Danke ! Brauche ich, um in der Nähe InputStreamReader ?
    • Guter Punkt. Die Fixierung.
    • Wenn Sie in der Nähe der BufferedReader schließt es InputStreamreader schließt den InputStream ein, die Sie von openStream() etc.
    InformationsquelleAutor Peter Lawrey
Schreibe einen Kommentar