Question

在新的第三版Effective Java中，Joshua Bloch提到了Java Puzzlers的一段代码（关于在try-finally中关闭资源）：

对于初学者来说，我在Java Puzzlers的第88页上弄错了，多年来没有人注意到。事实上，2007年Java库中close方法的三分之二使用是错误的。

但我不确定哪个部分出错了？

} finally {
  if (in != null) {
    try {
      in.close();
    } catch (IOException ex) {
      // There is nothing we can do if close fails
    }
  }
  if (out != null) {
    try {
      out.close();
    } catch (IOException ex) {
      // Again, there is nothing we can do if close fails
    }
  }
}

这是此代码的新版本：

try {
  OutputStream out = new FileOutputStream(dst);
  try {
    byte[] buf = new byte[BUFFER_SIZE];
    int n;
    while ((n = in.read(buf)) >= 0) out.write(buf, 0, n);

  } finally {
    out.close();
  }
} finally {
  in.close();
}

Answer 1

如果in.close()抛出了未被catch块捕获的异常（例如任何RuntimeException），则out甚至不会被尝试关闭。

虽然在给定的例子中（普通流最有可能IOException）但这不是一个大问题，但代码不正确并且学习如此编写可能会导致更严重的问题。

这个Java Puzzlers代码有什么问题？

1 个答案: