Do I need to close the input stream manually after using IOUtils.toString(input) of commons-io?

Question

Commons-IO has an IOUtils.toString(inputStream) method, which can read all content from an input stream:

InputStream input = getInputStream();
String content = IOUtils.toString(input);

My question is shall I close the input stream manually after using it?

I thought IOUtils may close it since it has read all the content, but I can't find that in the source code.

If you can't find that in the source code, isn't it a strong indication that it isn't there? — JB Nizet, Dec 07 '13 at 15:46
For files, use FileUtils.readFileToString(), it uses IOUtils, but also closes the stream. — Daniel Hári, Mar 31 '17 at 07:45

score 37 · Accepted Answer · edited May 24 '16 at 16:40

37

The javadoc says:

Wherever possible, the methods in this class do not flush or close the stream. This is to avoid making non-portable assumptions about the streams' origin and further use. Thus the caller is still responsible for closing streams after use.

edited May 24 '16 at 16:40

Anders Rabo Thorbeck

816
2
13
22

answered Dec 07 '13 at 15:47

JB Nizet

633,450
80
1,108
1,174

This is the conventional pattern for stream based libraries. The client should handle it. – Sotirios Delimanolis Dec 07 '13 at 15:49
2

Thank you, but I found I never need to do other operations on the input stream when I reading the content by `IOUtils.toString`, except closing it ... – Freewind Dec 07 '13 at 16:45
3

for files you can use FileUtils.readFileToString() that closes the stream – Daniel Hári Mar 31 '17 at 07:46

Do I need to close the input stream manually after using IOUtils.toString(input) of commons-io?

1 Answers1

Linked

Related