问题:
下载文件夹压缩文件为zip格式时打开报不可预料的压缩文件末端,而且文件夹里有多个文件但是下载的压缩包里面只有一个文件,还是损坏的;
原因:
检查了自己的代码,发现没什么问题,然后上网查找有说是流没关闭的问题,按照先打开的流后关闭的原则,检查了发现没错啊,最后自己写了个demo也能正常压缩下载,一步步排查最后发现原因是:我给响应设置了内容的大小:
//设置文件总的大小
response.setContentLengthLong(contentLength);
我把文件夹的所有文件的大小加起来然后再设置大小,没想到会导致下载压缩时出问题;
解决办法:去掉这行代码就好了
代码:
@GetMapping("/down")
public void down(HttpServletResponse response) {
String fileZipName = "test.zip";
fileZipName = new String(fileZipName.getBytes(StandardCharsets.UTF_8), StandardCharsets.ISO_8859_1);
response.addHeader("Content-Disposition", "attachment;fileName=" + fileZipName);
response.setContentType(MediaType.APPLICATION_OCTET_STREAM_VALUE);
try (ZipOutputStream zos = new ZipOutputStream(new BufferedOutputStream(response.getOutputStream()));
BufferedOutputStream bos = new BufferedOutputStream(zos)) {
for (int i = 1; i <= 2; i++) {
try (InputStream inputStream = getInputStream(i);
BufferedInputStream bis = new BufferedInputStream(inputStream)) {
//命名文件
String fileName = "1\\2\\" + i + ".txt";
zos.putNextEntry(new ZipEntry(fileName));
int len;
byte[] buf = new byte[10 * 1024];
while ((len = bis.read(buf, 0, buf.length)) != -1) {
bos.write(buf, 0, len);
}
bos.flush();
} catch (IOException e) {
log.error("下载文件异常 err:", e);
}
}
} catch (IOException e) {
log.error("下载文件异常 err:", e);
}
}
public InputStream getInputStream(int i) {
byte[] bytes = new byte[0];
try {
Path path = Paths.get("C:\\Users\\22891\\Desktop\\1\\2\\" + i + ".txt");
bytes = Files.readAllBytes(path);
} catch (IOException e) {
e.printStackTrace();
}
return new ByteArrayInputStream(bytes);
}
补充知识:
自动关闭流:try-with-resources
语法:
try () {
} catch (IOException e) {
}
只要需要关闭的流是实现了AutoCloseable接口的,就可以写在try语句的括号里面,就不需要手动关闭流;
其实反编译其字节码可以看到,自动关闭流也是写了try-catch-finally来关闭的。
如果需要手动关闭可以使用apache的common工具类,代码会简洁点:
import org.apache.commons.io.IOUtils;
IOUtils.closeQuietly();
如果使用 MappedByteBuffer,关闭流的时候要重写其clean方法:
/**
* 关闭 MappedByteBuffer流
*
* @param buffer
*/
public static void clean(final Object buffer) {
AccessController.doPrivileged(new PrivilegedAction() {
@Override
public Object run() {
try {
Method getCleanerMethod = buffer.getClass().getMethod("cleaner");
getCleanerMethod.setAccessible(true);
sun.misc.Cleaner cleaner = (sun.misc.Cleaner) getCleanerMethod.invoke(buffer, new Object[0]);
cleaner.clean();
} catch (Exception e) {
e.printStackTrace();
}
return null;
}
});
}