程序员都知道,对于计算机资源的利用必须严谨,在使用完资源后一定要释放它,否则就会造成内存泄漏,资源一直占用的危害。今天笔者就对文件IO操作完后,没有关闭流造成一定的问题,在拍错过程中也是有一定的难度。以下为对多个文件进行压缩的方法:
/**
* 将多个文件打包成压缩包
* @param srcFiles
* @param zipFile
*/
public static void zipFiles(List<File> srcFiles, File zipFile) {
// 判断压缩后的文件存在不,不存在则创建
if (!zipFile.exists()) {
try {
zipFile.createNewFile();
} catch (IOException e) {
logger.error("创建压缩包文件出错",e);
}
}
// 创建 FileOutputStream 对象
FileOutputStream fileOutputStream = null;
// 创建 ZipOutputStream
ZipOutputStream zipOutputStream = null;
// 创建 FileInputStream 对象
FileInputStream fileInputStream = null;
try {
// 实例化 FileOutputStream 对象
fileOutputStream = new FileOutputStream(zipFile);
// 实例化 ZipOutputStream 对象
zipOutputStream = new ZipOutputStream(fileOutputStream);
// 创建 ZipEntry 对象
ZipEntry zipEntry = null;
// 遍历源文件数组
for (int i = 0; i < srcFiles.size(); i++) {
// 将源文件数组中的当前文件读入 FileInputStream 流中
fileInputStream = new FileInputStream(srcFiles.get(i));
// 实例化 ZipEntry 对象,源文件数组中的当前文件
zipEntry = new ZipEntry(srcFiles.get(i).getName());
zipOutputStream.putNextEntry(zipEntry);
// 该变量记录每次真正读的字节个数
int len;
// 定义每次读取的字节数组
byte[] buffer = new byte[1024];
while ((len = fileInputStream.read(buffer)) > 0) {
zipOutputStream.write(buffer, 0, len);
}
}
zipOutputStream.closeEntry();
zipOutputStream.close();
fileInputStream.close();//------------错误---------------
fileOutputStream.close();
} catch (IOException e) {
logger.error("压缩文件出错",e);
}
}
- 乍一看这段代码并没有什么大问题,就是给多个文件生成压缩包,但是一般情况下,生成压缩包后需要删除原始文件只保留压缩包,在这里会出现的问题是,在删除原始文件时只能删除 最后一个文件,其余的文件没办法删除。(Windows上会出现这个问题,Linux与Mac上并不会)
- 此时笔者在进行错误排查时也是很想不通,如果删除文件的操作出错,但是为什么能删除部分文件,其余的删除不了。也想到过没有关闭文件流造成的文件删除不成功,但是如果文件流没关闭应该是都没关闭才对,经过多次实验发现每次都只能删除压缩文件时的最后一个文件,所以检查压缩方法,发现关闭文件流是在所有的文件压缩进去后关闭,所以之前的文件都没有被释放,只释放了最后一次,所以造成了这种问题。
所以在开发过程中,代码逻辑一定要严谨,否则会出现意想不到的损失。