问题如图:

相信大多数人都见过如此状态,流多了需要层层关闭,需要层层加上try-catch,为保证下一个流能关闭又要加上finally,烦人!!

 

我们先来一个测试:

public class Test {
    public static void main(String[] args) {
        try {
            System.out.println("第一try");
            throw new Exception("异常");
        } catch (Exception e) {
            e.printStackTrace();
        } finally {
            System.out.println("finally");
            try {
                throw new Exception("又是异常");
            } catch (Exception e) {
                // TODO Auto-generated catch block
                e.printStackTrace();
            }
            System.out.println("会执行吗");
        }
    }
}

运行结果:

提取出重要信息,finally里面可以抓到异常后可以继续往下执行,也就是说finally里面try-catch抓到异常后也必须把这个代码块执行完。

那么有人就写了一个工具类,这种工具类对吗?


public class IOUtil {
    public static void close(Closeable... closeableList) {
        try {
            for (Closeable closeable : closeableList) {
                if (closeable != null){
                    closeable.close();
                }
            }
        } catch (IOException e) {
            e.printStackTrace();
        }
    }
}

然后

finally{
      IOUtil.close(is, os);
}

看似正确,实则错误!!

如果第一个流关闭的时候异常,那么第二个流close()就不可达,执行不到

这和并列写没区别,如下:

finally{

    try{

          is.close();

          os.close();   // 第一个流关闭出异常第二个流关闭就不可达,执行不到,等于没关

    } catch(Exception e) {

    }

}

 

那怎么办呢?不能写工具类吗?

不是不能写,而是这种写法是错误的,这种情况下应该把try-catch放在for循环里面,这样才能保证循环会依次关闭流

 

正确写法1:

public static void close(Closeable... closeableList) {
    for (Closeable closeable : closeableList) {
        try {
            if (closeable != null) {
                closeable.close();
            }
        } catch (IOException e) {
            e.printStackTrace();
        }
    }
}

然后

finally{
      IOUtil.close(is, os);
}

 

正确解法2:

public <T extends java.io.Closeable> void close(T t) {
    try {
        if (t != null) {
            t.close();
        }
    } catch (Exception e) {
        e.printStackTrace();
    }
}

然后

finally {
    close(in);
    close(raf);
    close(br);
}

哪怕其中有流关闭出了异常,也不会影响到其他流的关闭,finally{...}里面的东西是要执行完的

 

=============================Talk is cheap, show me the code===========================