我需要关闭 java.io 包的 Reader 装饰器吗？

Question

我需要解析一个对象，并且需要将收到的 Reader 包装到 PushbackReader 中。在任何情况下我是否需要关闭 PushbackReader 或者是否足够安全，可以将其保持打开状态，因为它是底层读取器（我没有打开），

或者

public static MyObject parse(Reader reader) throws IOException, ParseException {
  PushbackReader pReader = new PushbackReader(reader, PUSHBACK_SIZE);
  try {
    return parseMyObject(pReader);
  } finally {
    pReader.close();
  }
}

是否足够安全，可以仅写入以下内容：

public static MyObject parse(Reader reader) throws IOException, ParseException {
  return parseMyObject(new PushbackReader(reader, PUSHBACK_SIZE));
}

---

有关信息，以下是如何操作我调用我的解析器：

BufferedReader reader = new BufferedReader(...);
try {
  while (...) {
    list.add(MyObjectParser.parse(reader));
  }
} catch (IOException e) {
  throw new RuntimeException("Could not read the stream", e);
} catch (ParseException e) {
  throw new ParseRuntimeException("Could not parse the stream", e);
} finally {
  // No need for null check.
  reader.close();
}

Answer 1

如果您记住几个关键点，这并不太复杂：

• 多次关闭流/读取器/写入器是可以的。第二次（或后续）关闭没有任何效果。
• 流包装器（PushBackReader、BufferedReader 等）do 在包装器关闭时关闭底层流。
• 垃圾收集器通常不会自动关闭流，除非该流拥有操作系统资源（文件描述符、套接字等）。
• 通常，流的创建者会（使用try-finally 块）完成后关闭流，例如

Reader reader = new FileReader(file);
try {
    parse(reader);
} finally {
    reader.close();
}

因此，作为一般规则，parse() 不应关闭读取器，除非有充分的理由这样做（例如，IOException 使流处于不确定状态）如果调用者希望将流用于其他用途。

如果 parse 在完成流后关闭流，这可能不是世界末日，但如果确实如此，那么您绝对应该记录这一事实。

Answer 2

最佳实践是始终关闭阅读器 - 如果它们碰巧是其他阅读器的包装器，那么它们应该将关闭操作级联到它们。

在您的代码片段中，只需决定责任所在。有人应该关闭读取器，无论是 parse 方法还是 parseMyObject 方法。我更喜欢 parse 方法，因为这样代码更具可读性 - 创建/打开阅读器的方法与关闭它的方法相同。
（否则你会遇到奇怪的情况 - 其他人可以使用 parseMyObject 然后突然惊讶地发现它关闭了 reader 参数）

更新：

现在我看到了对你的更新问题，这归结为在循环中接受读取器参数，然后多次包装它。我建议您重构代码，以便仅使用 PushbackReader 包装它一次。或者将reader的初始化代码移到内部方法中。

Answer 3

如果被调用者将关闭底层 Reader，则无需在 PushbackReader 上调用 close()。 PushbaseReader 只是一个带有缓冲区的包装器，当您使用完它后，该缓冲区将被垃圾回收。对它调用 close() 将关闭底层的 Reader，如果您希望该方法关闭它，您将需要保留它。

更新：根据您的代码，您似乎无法在 PushbackReader 上调用 close()，因为它还会关闭您的基础 <代码>阅读器。据我所知，如果您这样做，下一次迭代应该会失败，并出现有关流被关闭的异常。例如这个例子失败了：

BufferedReader reader = new BufferedReader(new StringReader("foo"));
new PushbackReader(reader).close();
reader.read(); // IOException: Stream closed