在Java中关闭输入流

Closing inputstreams in Java

我在try / catch块中有以下代码

1
2
3

InputStream inputstream = conn.getInputStream();
InputStreamReader inputstreamreader = new InputStreamReader(inputstream);
BufferedReader bufferedreader = new BufferedReader(inputstreamreader);

我的问题是，当我必须在finally块中关闭这些流时，是否必须关闭所有3个流，或者仅关闭befferedreader会关闭所有其他流？

相关讨论

按照约定，包装器流(用于包装现有流)在关闭时会关闭基础流，因此在您的示例中只需关闭bufferedreader。而且，关闭已经关闭的流通常是无害的，因此关闭所有3个流不会受到伤害。

相关讨论

通常，只关闭最外部的流是可以的，因为按照惯例，它必须触发基础流的关闭。

因此，通常代码如下所示：

1
2
3
4
5
6
7
8
9
10

BufferedReader in = null;

try {
in = new BufferedReader(new InputStreamReader(conn.getInputStream()));
...
in.close(); // when you care about Exception-Handling in case when closing fails
}
finally {
IOUtils.closeQuietly(in); // ensure closing; Apache Commons IO
}

但是，在少数情况下，基础流构造函数会在流已打开的情况下引发异常。在那种情况下，上面的代码将不会关闭基础流，因为从未调用外部构造函数并且in为null。因此，finally块不会关闭任何内容，从而使基础流保持打开状态。

从Java 7开始，您可以执行以下操作：

1
2
3
4
5

try (OutputStream out1 = new ...; OutputStream out2 = new ...) {
...
out1.close(); //if you want Exceptions-Handling; otherwise skip this
out2.close(); //if you want Exceptions-Handling; otherwise skip this
} // out1 and out2 are auto-closed when leaving this block

在大多数情况下，在关闭时引发异常时，您不需要异常处理，因此请跳过这些显式的close()调用。

编辑
这是一些针对非信徒的代码，在这些代码中使用此模式非常重要。您可能还想阅读有关closeQuietly()方法的Apache Commons IOUtils javadoc。

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22

OutputStream out1 = null;
OutputStream out2 = null;

try {
out1 = new ...;
out2 = new ...;

...

out1.close(); // can be skipped if we do not care about exception-handling while closing
out2.close(); // can be skipped if we ...
}
finally {
/*
* I've some custom methods in my projects overloading these
* closeQuietly() methods with a 2nd param taking a logger instance,
* because usually I do not want to react on Exceptions during close
* but want to see it in the logs when it happened.
*/
IOUtils.closeQuietly(out1);
IOUtils.closeQuietly(out2);
}

当创建out2引发异常时，使用@Tom的"建议"会使out1保持打开状态。这个建议是来自谈论It's a continual source of errors for obvious reasons.的人的，嗯，我可能是瞎子，但对我来说并不明显。在Tom的模式容易出错的情况下，我可以想到的每个用例中，我的模式都是白痴安全的。

相关讨论

请不要做这种null舞蹈。由于明显的原因，它是错误的持续来源。真是一团糟。
这是正常模式。您还想做什么？
acquire(); try { use(); } finally { release(); }
@布鲁诺右-那就是我在上一段中的意思。
@ TomHawtin-tackline因此，当获取引发异常时，您不会重新实现吗？这很不好，尤其是当使用特殊的OutputStreams(例如TeeOutputStream)时，其中一个已经创建，而另一个失败。
我认为Tom Hawtin想要说的是，这种null测试模式在这种情况下没有用。如果您没有通过初始化，它将抛出一个IOException。由于您仍然没有抓住它，它不会改变任何东西。此外，无论如何，您都必须在finally块中的in.close()的外部try / catch中捕获它。
这是有用且完全标准的模式。如此普遍以至于Apache Commons IOUtils中存在实用程序方法。对不起，但是他在这里根本错了。
只是说，在您的示例中，如果new BufferedReader(...)失败，则会将IOException抛出这些行的范围之外(并且in.close()将不会执行，因为in将是null)。如果将初始化与上面的声明BufferedReader in = new (...)一起放置，则无论如何抛出异常，都会发生完全相同的事情。
是的，在这种情况下，您必须在try块之外声明第二个变量，并在finally块中分别关闭它。但这对于@ TomHawtin-tackline建议是不可能的，因为根据他的建议，当获取过程中发生错误时，通常无法释放。
好吧，我认为acquire(); try { use(); } finally { release(); }块必须在try / catch (IOException ...)块(或使用throws的方法)内，而不一定是finally块或其他变量，仅因为两个close构造函数可以抛出它。在这种情况下，在块内外使用new BufferedReader语句不会更改任何内容。无论如何，如果在acquire()语句期间引发了异常，它将不会到达finally语句。
当您使用多个流时，您描述的模式确实有用。
刘海撞到墙上如何释放尚未获得的东西？不要混淆try-finally和try-catch(就像Java语言一样)。一个(正版)资源pre try-finally。在这里使用null不仅使代码复杂，而且经常引入(严重的)错误。即使指出，它也经常被错误地修复。把事情简单化。
好吧，我发布了一些代码，清楚地表明了为什么这种模式是正确的，应该使用它。这不是我个人的废话，而是一种非常普遍且可以接受的模式。甩头并不是我的职业，但爱因斯坦却想到："使事情尽可能简单，但不要简单。" 不要忘记最后一部分...

关闭最外面的一个就足够了(即bufferedreader)。阅读BufferedReader的源代码，我们可以看到它在调用自己的close方法时关闭了内部Reader：

1
2
3
4
5
6
7
8
9
10

513 public void close() throws IOException {
514 synchronized (lock) {
515 if (in == null)
516 return;
517 in.close();
518 in = null;
519 cb = null;
520 }
521 }
522 }

您只需要关闭实际资源。即使构造装饰器失败，也应关闭资源。对于输出，应该在满意的情况下刷新大多数装饰对象。

一些并发症：

有时，装饰器是不同的资源(某些压缩实现使用C堆)。
在糟糕的情况下关闭装饰器实际上会导致刷新，并产生混乱，例如实际上并未关闭基础资源。
看起来您的基础资源是URLConnection，它没有这样的disconnect / close方法。

您可能希望考虑使用Execute Around惯用语，因此您不必重复这种事情。

我将按照打开它们的相反顺序关闭所有它们，好像打开它们时会将阅读器推到堆栈中，而关闭将阅读器从堆栈中弹出。

最后，在全部关闭之后，"阅读器堆栈"必须为空。

相关讨论

根据经验，应按打开它们的相反顺序关闭所有组件。