java - 对嵌套的 try/catch block 有偏好吗？

Question

在 Java 中使用 Readers 和 Streams 时一直困扰我的一件事是 close() 方法可能会引发异常。由于将 close 方法放在 finally block 中是个好主意，因此需要一些尴尬的情况。我通常使用这种结构:

FileReader fr = new FileReader("SomeFile.txt");
try {
    try {
        fr.read();
    } finally {
        fr.close();
    }
} catch(Exception e) {
    // Do exception handling
}

但我也见过这种结构:

FileReader fr = new FileReader("SomeFile.txt");
try {
    fr.read() 
} catch (Exception e) {
    // Do exception handling
} finally {
    try {
        fr.close();
    } catch (Exception e) {
        // Do exception handling
    }
}

我更喜欢第一种构造，因为只有一个 catch block ，而且它看起来更优雅。是否有理由更喜欢第二种或另一种结构？

更新:如果我指出 read 和 close 都只抛出 IOExceptions，会有所不同吗？所以在我看来，如果读取失败，关闭也会因为同样的原因而失败。

Answer 1

恐怕第一个例子有一个大问题，就是如果在读取时或读取后发生异常，finally block 就会执行。到目前为止，一切都很好。但是如果 fr.close() 然后导致另一个异常被抛出呢？这将“胜过”第一个异常(有点像将 return 放在 finally block 中)并且您将丢失有关实际导致问题的所有信息 开始。

你的 finally block 应该使用:

IOUtil.closeSilently(fr);

这个实用方法的作用:

public static void closeSilently(Closeable c) {
    try { c.close(); } catch (Exception e) {} 
}