Ocurrio un error:java.io.IOException: Stream closed

[Marcel Chasiguasin]

Dear all,

why is the following code returning this exception Ocurrio un error:java.io.IOException: Stream closed???

package com.cec.ejemplos;

import java.io.*;

public class PruebasFiles_003_001 {

	static public void main(String[] args) {
		File file = new File("PruebasFiles_003.html");
		try {
			FileWriter fw = new FileWriter(file);
			BufferedWriter bf = new BufferedWriter(fw);
			PrintWriter pw = new PrintWriter(bf);
			pw.println(" < html > < body > < h1 > ");
			pw.println("Hola maestro");
			pw.printf("int: %-15d \r\n", 123);
			pw.format("int: %+012d \r\n", 123);
			pw.println(" < /h1 > < /body > < /html > ");
			pw.close();
			bf.flush();
			bf.close();
		} catch (IOException ex) {
			System.out.println("Ocurrio un error:" + ex);
		}
	}
}

[Tomasz Sochanski]

PrintWriter is closed when you call BufferedWriter.flush(), first run bf.flush(), then pw.close().

[Joanne Neal]

And because your BufferedWriter is wrapped by the PrintWriter, you don't need to call bf.close. It will be done automatically when you call pw.close
Edit: Which is basically what Tomasz said now that I reread it.

[Marcel Chasiguasin]

Joanne Neal wrote:And because your BufferedWriter is wrapped by the PrintWriter, you don't need to call bf.close. It will be done automatically when you call pw.close
Edit: Which is basically what Tomasz said now that I reread it.

Dear Joanne,

for free memory and close the connection. do I only need execute this code
bf.flush();
bf.close();

or this
pw.flush();
pw.close();
???

[Tomasz Sochanski]

my bet is:

bf.flush();
pw.close();

Thanks Joanne for your notice regarding unnecessary call to bf.close().

[Joanne Neal]

Marcel Chasiguasin wrote:for free memory and close the connection. do I only need execute this code
bf.flush();
bf.close();

or this
pw.flush();
pw.close();
???

You only need to call close on the outermost wrapper object. So, as you have a FileWriter wrapped by a BufferedWriter wrapped by a PrintWriter, calling close on your PrintWriter will also close the FileWriter and BufferedWriter.
I think on most writers calling close will automatically do a flush, but it won't do any harm to call it explicitly.

[Javin Paul]

On related note Java 7 has added automatic resource Management which will automatically closed any resource opened inside try block, no need to call close() explicitly.