最近在开发中,使用到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;
}