经典的IO代码为什么有资源泄漏？

修复IO资源泄漏

最新推荐文章于 2023-11-15 14:33:57 发布

mooninwell

最新推荐文章于 2023-11-15 14:33:57 发布

阅读量956

点赞数

CC 4.0 BY-SA版权

分类专栏： Java 文章标签： io null 测试 file 编程

本文链接：https://blog.youkuaiyun.com/mooninwell/article/details/638712

Java 专栏收录该内容

0 篇文章

订阅专栏

本文介绍了一个常见的IO操作代码陷阱，即在反序列化过程中未能正确关闭底层文件输入流而导致资源泄漏的问题，并提供了一种改进方案。

最近在开发中，使用到IO操作进行反序列化操作，使用了在各种资料中常见的关于IO操作的经典代码，在自测和测试人员的测试中都没有发现任何问题！
代码如下

public Object readObject(File file)
    {
        Object o = null ;
         if (file.exists())
        {
            ObjectInputStream ois = null ;
             try
            {
                ois = new ObjectInputStream( new FileInputStream(file));
                o = ois.readObject();
            }
             catch (Throwable e)
            {
                e.printStackTrace();
            }
             finally
            {
                 if (ois != null )
                {
                     try
                    {
                        ois.close();
                    }
                     catch (IOException e)
                    {
                        e.printStackTrace();
                    }
                }
            }
        }
         return o;
    }

忽然一天测试人员联系我说这部分代码导致系统无法启动，原因仅仅是因为让该部分代码不断的处理一个为序列化文件！
我仔细分析一下代码，没有问题呀，反序列化失败就直接关闭了ois对象，不可能造成资源泄漏，于是用单步跟踪的方式
查看处理非法文件的详细情况：
1、创建FileInputStream 没有问题
2、创建ObjectInputStream失败没有问题
3、进入catch块，打印堆栈没有问题
4、进入finnaly块，if (ois != null)为false，这时意识到出问题了
原来创建FileInputStream成功，此时已经申请了资源，但在后续创建ObjectInputStream时失败，但此时又没有保存FileInputStream
的引用，无法释放资源，最终导致了资源泄漏。看来在以后的IO的编程中一定要保存基本流的引用，否则在类似上面的情况
在转换流失败后，无法安全的释放资源！
下面为修改后的代码：

public Object readObject(File file)
    {
        Object o = null ;
         if (file.exists())
        {
            FileInputStream fis = null ;
            ObjectInputStream ois = null ;
             try
            {
                fis = new FileInputStream(file);
                ois = new ObjectInputStream(fis);
                o = ois.readObject();
            }
             catch (Throwable e)
            {
                e.printStackTrace();
            }
             finally
            {
                 if (fis != null )
                {
                     try
                    {
                        fis.close();
                    }
                     catch (IOException e)
                    {
                        e.printStackTrace();
                    }
                }
            }
        }
         return o;
    }

注：首发http://www.blogjava.net/mooninwell