Why do we need to close a FileInputStream (and streams in general) in any case before we leave the program? What would happen otherwise? If the program stops before the input stream is closed explicitly in the program, doesn't the stream also close automatically?
- 3Lots of comments sound like "why you won't close the stream" but I think your question was not about "why" but rather "why explicity".Ustaman Sangat– Ustaman Sangat2012-03-01 20:44:02 +00:00Commented Mar 1, 2012 at 20:44
Add a comment |
6 Answers
File handles are scarce, finite resources. You can run out of them if you don't clean them up properly, just like database connections.
If you've written a small program with just one user you can get away with being sloppy and not closing in a finally block.
But if you end up using that idiom in an application that has many users and file handles you might have a problem.
"First we make our habits, then they make us." I try to apply best practices even when they aren't necessary.
3 Comments
markdsievers
I'm definitely going to recycle the habit wisdom. That's a quotable quote right there.
Dan R.
That quotable quote ("First we make our habits, then they make us.") is from Stephen Covey.
duffymo
11 years late and wrong. Attributed to John Dryden, a poet who died in 1700. Covey isn't that smart. quoteinvestigator.com/2016/12/19/habits-make
Yes, when the process terminates the unmanaged resources will be released. For InputStreams this is fine. For OutputStreams, you could lose an buffered data, so you should at least flush the stream before exiting the program.
6 Comments
Zippo
And what about when the stream object is going out of scope (and garbage-collected)? Will it be closed?
Jon Skeet
@Zippo: Just because a variable goes out of scope doesn't mean the object will be garbage collected. Garbage collection is non-deterministic. Some streams may have a finalizer which will flush them - but it would be a very bad idea to rely on that, IMO.
Jon Skeet
@Zippo: Thread.stop() is deprecated anyway, as it's generally a bad idea to stop a thread rather than ask it to stop itself in a controlled way.
Ustaman Sangat
Java 8 will have some features where you can have streams automatically closed upon exiting a scope, comparable to how monitor locks are realease upon exiting a synchronized block.
Jon Skeet
@UstamanSangat: It's in Java 7, actually: docs.oracle.com/javase/tutorial/essential/exceptions/… (Assuming that's the feature you mean...)
|
Dude. If you don't close your stream, your unit test will fail. Or at least, it should. So, that's why you need to close it. ;)
And while the OS will almost certainly clean up if you just exit, they'll generally get freed up faster if you explicitly close them. Furthermore, what if your code ends up in a long-running program sometime down the road? Then they'll have problems and curse you. :(
So, it's like washing your hands after using the bathroom. Eventually someone will pay the price if you don't do it. You can get away with it for a while, but it's still a good practice.
4 Comments
Srikanth
Just curious, why is this voted down? Seems like a good answer to me although funny :)
Don Branson
Naturally, I'd to know, too. :) But maybe the commenter just didn't appreciate that the answer is informative AND humorous.
user42155
I don't know either. For me it was helpful.
Personally, because the answer has virtually no technical content -- it would have been a better comment. It doesn't have much more to say than "it's a good practice", and while we often accept such explanations for expediency in our projects, it's not a good enough answer for Stack Overflow. No personal offence meant to you, Don; I'm sure you know what you're talking about, but I prefer answers with more meat.
In addition to Jon's answer, it is generally a good idea to close any resource.
Think of a database connection. Your database cannot have infinite connections opened, in this case when you don't need it, it's better you close it as soon as you're done with it.
It is also good to make this a habit. "finally" block is your friend. In case of C++, you can also use RAII to manage this automatically.
2 Comments
Ustaman Sangat
And Java 8 is supposed reduce the boiler plate involved with finally.
Ustaman Sangat
Java 7 actually, the java.io.AutoCloseable
Sort of. The stream is backed by a "real" operating system file descriptor, and when the process terminates the OS will clean up any open file descriptors.
Anyway, it's good practice to always close your resources when you're done, so close the streams :)
Comments
If you don't close streams, you may have problems opening them back up again. This is especially true if they're hanging off the end of sockets.
Closing a stream also makes sure that data is flushed through the stream if there is any data left to send.
