Java 文件处理，我做错了什么？

Question

为 Java 家庭作业编写了一个基本的文件处理程序，当我拿回作业时，我收到了一些关于未能捕获一些实例的注释：

• 文件中的缓冲区可能为空。
• 未找到
• 文件 文件流未关闭

这是用于打开文件的代码块：

/**
 * Create a Filestream, Buffer, and a String to store the Buffer.
 */
FileInputStream fin = null;
BufferedReader buffRead = null;
String loadedString = null;

/** Try to open the file from user input */
try
{
    fin = new FileInputStream(programPath + fileToParse);
    buffRead = new BufferedReader(new InputStreamReader(fin));
    loadedString = buffRead.readLine();
    fin.close();
}
/** Catch the error if we can't open the file */
catch(IOException e)
{
    System.err.println("CRITICAL: Unable to open text file!");
    System.err.println("Exiting!");
    System.exit(-1);
}

我从他那里得到的一条评论是 fin.close(); 需要位于一个 finally 块，我根本没有。但我认为我创建 try/catch 的方式可以防止文件无法打开的问题。

让我澄清几件事：这不是当前的作业（不是试图让某人做我自己的工作），我已经创建了我的项目并已对其进行评分。我自己并不完全理解教授的推理。最后，我没有太多的 Java 经验，所以我有点困惑为什么我的 catch 不够好。

Answer 1

• 文件中的缓冲区可能为空。

文件可能为空。也就是说，打开文件时就到达文件结尾。 loadedString = buffRead.readLine() 将返回 null。

也许您应该通过添加类似 if (loadedString == null) returnedString = ""; 的内容来解决此问题

• 找不到文件

正如 FileInputStream(String) 构造函数的文档中所述，它可能会抛出 FileNotFoundException。您确实在 IOException 子句中捕获了这个（因为 FileNotFoundException 是一个 IOException），所以它很好，但是您也许可以这样做：

} catch (FileNotFoundException fnfe) {
    System.err.println("File not fonud!");
} catch (IOException ioex {
    System.err.println("Some other error");
}

• 文件流未关闭

您确实调用了fin.close()，它在正常情况下会关闭文件流。也许他的意思是它并不总是关闭的。 readLine 可能会抛出 IOException，在这种情况下，close() 会被跳过。这就是将其放在 finally 子句中的原因（这确保无论 try 块中发生什么情况都会调用它。(*)

---

(*) As @mmyers正确地指出，将 close() 放在 finally 块中实际上是不够的，因为您在中调用 System.exit(-1) catch 块。如果这确实是所需的行为，您可以在 catch 子句中设置一个错误标志，并在设置了该标志后在 finally 子句后退出。

Answer 2

但是，如果您的程序在 try 块的第二行或第三行抛出异常怎么办？

buffRead = new BufferedReader(new InputStreamReader(fin));
loadedString = buffRead.readLine();

至此，一个文件句柄已被打开并分配给fin。您可以捕获异常，但文件句柄将保持打开状态。

您需要将 fin.close() 语句移至 finally 块：

} finally {
    try {
        if (fin != null) {
            fin.close();
        }
    } catch (IOException e2) {
    }
}

Answer 3

假设 buffRead.readLine() 抛出异常，您的 FileInputStream 是否会被关闭，或者该行是否会被跳过？ finally 块的目的是，即使在特殊情况下，finally 块中的代码也将执行。

Answer 4

除了打开文件之外，还可能发生许多其他错误。

最后，您可能会得到一个已定义或未定义的 fin，您必须保护它免受空指针错误的影响，并且不要忘记关闭文件可能会引发新的异常。

我的建议是在一个单独的例程中捕获它，并让 IOExceptions 飞出它：

类似于

private String readFile() throws IOException {
  String s;
  try {
    fin = new FileInputStream(programPath + fileToParse);
    buffRead = new BufferedReader(new InputStreamReader(fin));
    s = buffRead.readLine();
    fin.close();
  } finally {
    if (fin != null {
       fin.close()
    }
  }
  return s
}

然后在您需要它的地方：

try {
   loadedString = readFile();
} catch (IOException e) {
   // handle issue gracefully
}