Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Why is Files.lines (and similar Streams) not automatically closed?

The javadoc for Stream states:

Streams have a BaseStream.close() method and implement AutoCloseable, but nearly all stream instances do not actually need to be closed after use. Generally, only streams whose source is an IO channel (such as those returned by Files.lines(Path, Charset)) will require closing. Most streams are backed by collections, arrays, or generating functions, which require no special resource management. (If a stream does require closing, it can be declared as a resource in a try-with-resources statement.)

Therefore, the vast majority of the time one can use Streams in a one-liner, like collection.stream().forEach(System.out::println); but for Files.lines and other resource-backed streams, one must use a try-with-resources statement or else leak resources.

This strikes me as error-prone and unnecessary. As Streams can only be iterated once, it seems to me that there is no a situation where the output of Files.lines should not be closed as soon as it has been iterated, and therefore the implementation should simply call close implicitly at the end of any terminal operation. Am I mistaken?

like image 968
MikeFHay Avatar asked Dec 03 '15 17:12

MikeFHay


People also ask

Why we need to close the stream objects always after usage?

You should always close a stream in order to free open resources on your OS. Opening a stream always returns an identifier which you can use to close it wherever you are in your code (as long as the identifier is valid), whether their from another method or class.

What will happen if you forget to close the stream of the i/o file?

Bad things that can happen when you don't close your streams: you can run out of file handles. data that you think is written to disk may still be in the buffer (only) files might still be locked for other processes (depends on the platform)

What happens if you dont close a stream?

What is the actual consequence to leaving a stream, channel, or connection open? System you connected is still handling this connection, even if your process is not using it. That means that extra memmory, cpu, sockets and possible other resources are still allocated.

Why streams are closed?

Streams have a BaseStream. close() method and implement AutoCloseable , but nearly all stream instances do not actually need to be closed after use. Generally, only streams whose source is an IO channel (such as those returned by Files. lines(Path, Charset) ) will require closing.


1 Answers

Yes, this was a deliberate decision. We considered both alternatives.

The operating design principle here is "whoever acquires the resource should release the resource". Files don't auto-close when you read to EOF; we expect files to be closed explicitly by whoever opened them. Streams that are backed by IO resources are the same.

Fortunately, the language provides a mechanism for automating this for you: try-with-resources. Because Stream implements AutoCloseable, you can do:

try (Stream<String> s = Files.lines(...)) {     s.forEach(...); } 

The argument that "it would be really convenient to auto-close so I could write it as a one-liner" is nice, but would mostly be the tail wagging the dog. If you opened a file or other resource, you should also be prepared to close it. Effective and consistent resource management trumps "I want to write this in one line", and we chose not to distort the design just to preserve the one-line-ness.

like image 97
Brian Goetz Avatar answered Oct 02 '22 20:10

Brian Goetz