Do I have to close FileOutputStream which is wrapped by PrintStream?

前端 未结 5 1724
醉话见心
醉话见心 2020-12-05 07:43

I\'m using FileOutputStream with PrintStream like this:

class PrintStreamDemo {  
    public static void main(String args[]) { 
            


        
相关标签:
5条回答
  • 2020-12-05 08:12

    No you dont need to. PrintStream.close method automatically closes the underlining output stream.

    Check the API.

    http://download.oracle.com/javase/6/docs/api/java/io/PrintStream.html#close%28%29

    0 讨论(0)
  • 2020-12-05 08:23

    No, you only need to close the outermost stream. It will delegate all the way to the wrapped streams.

    However, your code contains one conceptual failure, the close should happen in finally, otherwise it's never closed when the code throws an exception between opening and closing.

    E.g.

    public static void main(String args[]) throws IOException { 
        PrintStream ps = null;
    
        try {
            ps = new PrintStream(new FileOutputStream("myfile.txt"));
            ps.println("This data is written to a file:");
            System.out.println("Write successfully");
        } catch (IOException e) {
            System.err.println("Error in writing to file");
            throw e;
        } finally {
            if (ps != null) ps.close();
        }
    }
    

    (note that I changed the code to throw the exception so that you understand the reason of the problem, the exception namely contains detailed information about the cause of the problem)

    Or, when you're already on Java 7, then you can also make use of ARM (Automatic Resource Management; also known as try-with-resources) so that you don't need to close anything yourself:

    public static void main(String args[]) throws IOException { 
        try (PrintStream ps = new PrintStream(new FileOutputStream("myfile.txt"))) {
            ps.println("This data is written to a file:");
            System.out.println("Write successfully");
        } catch (IOException e) {
            System.err.println("Error in writing to file");
            throw e;
        }
    }
    
    0 讨论(0)
  • 2020-12-05 08:30

    No, according to the javadoc, the close method will close the underlying stream for you.

    0 讨论(0)
  • 2020-12-05 08:32

    No. It is not require to close other components. when you close stream it automatically close other related component.

    0 讨论(0)
  • 2020-12-05 08:33

    No , here is implementation of PrintStream's close() method:

    public void close() {
        synchronized (this) {
            if (! closing) {
            closing = true;
            try {
                textOut.close();
                out.close();
            }
            catch (IOException x) {
                trouble = true;
            }
            textOut = null;
            charOut = null;
            out = null;
            }
        }
    

    You can see out.close(); which closes output stream.

    0 讨论(0)
提交回复
热议问题