EingabeStream ohne einen try/catch-Block schließen?

Question

Ich weiß, dass ein InputStream geschlossen sein sollte. Aber ich habe Zweifel, wo und wie ich das mache.

Gemäß der Dokumentation auf IOUtils.closeQuietly:

Unconditionally einen Inputstream schließen. Entspricht InputStream.close(), außer Ausnahmen werden ignoriert. Dies ist typischerweise in Endblöcke.

Ich brauche keinen try/catch Block in meinem Code, so habe ich keinen finally Block. Ist es in Ordnung, den InputStream vor der Rückgabe in meiner Methode zu schließen, oder sollte ich etwas anderes tun? Diese Methode wird von mehreren Diensten verwendet, um eine InputStream aus einer Datei zu laden.

public InputStream read(String filename) { 
    InputStream inputStream = Thread.currentThread().getContextClassLoader().getResourceAsStream(filename); 

    if (inputStream == null) { 
     // Throw some exception 
    } 

    IOUtils.closeQuietly(inputStream); 

    return inputStream; 
} 

Answer 1

Sie sollten nicht IOUtils.closeQuietly(inputStream); überhaupt in dieser Methode anrufen - es sehr wenig Sinn, einen geschlossenen Strom in der Rückkehr.

jedoch diese Methode sollte in einem try/finally-Block aufgerufen werden:

InputStream is = null; 
try { 
    is = read(filename); 
    // Do whatever with is. 
} finally { 
    IOUtils.closeQuietly(is); 
} 

oder mit Try-mit-Ressourcen (in Anbetracht der Kommentar here, dass „Try-mit-Ressourcen Aussage beseitigen die meisten muss für die Verwendung von IOUtils.closeQuietly „):

try (InputStream is = read(filename)) { 
    // Do whatever with is. 
} 

Answer 2

try/finally-Block:

InputStream ist = null; versuchen { InputStream ist = ist = lesen (Dateiname); // Tu was auch immer ist. } schließlich { is.close(); }

Hinweis: alle I/O-Ressourcen in die endgültig geschlossen werden müssen, blockieren, da es in der try-catch-Block initialisiert wurde. Es ist auch hinzuzufügen geraten:

} catch(IOException e){ 
    e.printstacktrace(); 
} 

... für die Ausnahme