有没有一种干净的方法可以给 Java 对象范围提供破坏？

Question

想象一下 MyOpenedFile 是用打开的流包装文件的东西。然后假设这段代码：

// method in an Util class
static void safeClose(MyOpenedFile f) {
  if (f != null) {
    try {
      f.close();
    } catch(IOException ex) { /* add logging or console output */ }
  }
}

问题的实际方法：

void doSomeFileOperation(...) throws IOException, ... {
    MyOpenedFile f1 = null;
    MyOpenedFile f2 = null;
    try {

      /* method's "business logic" code beings */
      f1 = new MyOpenedFile(...);
      //do stuff
      f2 = new MyOpenedFile(...);
      // do stuff
      f1.close(); f1 = null;
      // do stuff with f1 closed
      f2.close(); f2 = null;
      // do stuff with f2 closed
      /* method's "business logic" code ends */

    } finally {
      Util.safeClose(f1); f1 = null;
      Util.safeClose(f2); f2 = null; 
    }
}

现在这很混乱，特别容易出错（例如，finally 块中的某些代码可能很难在单元测试中调用）。例如，在 C++ 中，析构函数会负责清理（由作用域指针析构函数调用或直接调用），代码会更干净。

那么，有没有更好/更好/更简洁的方法来包装上面的业务逻辑代码，以便传播任何异常但 f1 和 f2 两个文件都被关闭（或者至少在两者上都尝试关闭，即使失败）？

还有指向任何开源库（例如 Apache Commons）的答案，欢迎提供漂亮的包装器。

Answer 1

文件是字符串的包装器，其中包含可能存在或不存在的文件名。它是无状态的，所以你不需要关闭它。

您需要关闭的资源是 FileInputStream 或 BufferedReader，您可以在 Java 7 中使用 ARM 隐式关闭它们

try(BufferedReader br = new BufferedReader(new FileReader(file))) {

}

这将在他退出时关闭br。

http://www.oracle.com/technetwork/articles/java/trywithresources-401775.html

Answer 2

看看The try-with-resources Statement，它将在try-block结束后关闭资源。

您使用的File 类似乎不是java.io.File，因为它没有任何close() 方法。在这种情况下，请确保您自己的 File 类实现 Closeable 以使其与 ARM 一起使用。

try (FileInputStream f1 = new FileInputStream("test1.txt");
     FileInputStream f2 = new FileInputStream("test2.txt")) {
    // Some code
} catch (FileNotFoundException e) {
    e.printStackTrace();
} catch (IOException e) {
    e.printStackTrace();
}

Answer 3

您不需要像这里提到的那样关闭文件（文件系统上文件的表示形式）：

Do I need to close files I perform File.getName() on?

我假设您正在询问有关文件流/阅读器的更多信息？

在这种情况下，java 7 有一个不错的新特性： http://www.vineetmanohar.com/2011/03/java-7-try-with-auto-closable-resources/

如果您正在使用旧版本的 java，我会保持简单：

void doSomeFileOperation(...) throws IOException, ... {
  FileInputStream f1 = null;
  FileInputStream f2 = null;
  try {

    // do stuff

  } finally {
    Util.safeClose(f1); 
    Util.safeClose(f2); 
  }
}

Answer 4

一个自动出现在我脑海中的选项是：将处理文件的代码与进行任何处理的代码分开。通过这种方式，您可以封装处理打开、关闭和异常处理的讨厌代码。

另外一点是，你的样本做了很多额外的、不需要的步骤

void doSomeFileOperation(...) throws IOException, ... {
    File f1 = null;
    File f2 = null;
    try {
      f1 = new File(...);
      f2 = new File(...);
      // callback to another class / method that does the real work
    } finally {
      Util.safeClose(f1);
      Util.safeClose(f2);
    }
}

您不需要将 File 实例设置为 null。如果你尝试使用它们，你会得到一个例外。

我不知道您使用的是什么 File 对象。 java中的标准File类没有close()方法。