Sollen BufferedReader und InputStreamReader explizit geschlossen werden?

Question

Ich möchte den Inhalt eines InputStream in eine String lesen:

private String readToString(InputStream stream) { 
    return new BufferedReader(new InputStreamReader(stream)) 
      .lines().collect(Collectors.joining("\n")); 
} 

Der Strom kommt aus java.lang.Process.

Frage: Muss ich in diesem Fall einen der InputStream, InputStreamReader oder BufferedReader explizit schließen?

Nebenbei bemerkt: die verknüpfte Frage ist kein Duplikat, wie meine Frage nach WIE ist richtig, die Ströme nahe an, nicht wie der Stream in einen String zu lesen!

Answer 1

Sie nur die outer Wrapper schließen müssen, aber tun das nicht explizit so oder so - es gibt Anprobe mit-Ressource, die Ihnen das Leben leichter machen:

public String readToString(InputStream stream) { 

    try (InputStreamReader reader = new InputStreamReader(stream); 
      BufferedReader br = new BufferedReader(reader)) { 

     return br.lines().collect(Collectors.joining("\n")); 

    } catch (IOException e) { 
     e.printStackTrace(); 
     throw new RuntimeException(e); 
    } 
} 

Es gibt auch eine viel einfache Art und Weise, dass und Weise mehr klar zu machen:

Files.readAllLines(YourPath) 

Answer 2

per meinem Kommentar, entweder die BufferedReader oder die Input Schließen wird die Input verursacht geschlossen werden. Ihre readToString Methode sollte den Stream nicht schließen. Dies liegt in der Verantwortung des Anrufers.

Die Begründung: -

Zunächst überlegen, wie Sie den Stream öffnen, bevor Sie readToString nennen. Eine sinnvolle Methode ist:

try (InputStream myStream = getInputStreamSomehow()) { 
    //... 
    String content = readToString(myStream); 
    //... 
} 

Der Stream wird am Ende des Blocks "try-with-resources" geschlossen.

Zweitens, Best Practices und Idiome berücksichtigen. Sehen Sie sich Java-API-Methoden an, die genau wie Ihre Methode den gesamten Inhalt eines Streams lesen. Beispielsweise aus der Java 9: ​​

• readAllBytes()
• transferTo(OutputStream out)

Keines der obigen Verfahren schließt den Strom. Ebenso werden die Benutzer Ihrer readToString-Methode nicht erwarten, dass Sie ihren Stream schließen.