Spring Boot多线程复制ZipFile到ZipOutputStream出现CRC校验错误

嘿，这个问题我太熟了——你遇到的是典型的线程安全坑！ZipFile类本身并不是线程安全的，当多个请求同时操作同一个ZipFile实例（或者共享它的entries()枚举）时，会把内部状态搞乱，直接导致CRC校验不匹配的错误。

问题出在哪？ #

你当前的代码里，大概率是多个请求复用了同一个ZipFile实例，或者在多线程环境下同时遍历templateFileEntries这个枚举。ZipFile的entries()返回的枚举是依赖于它内部状态的，多个线程同时操作会打乱读取指针和数据逻辑，读出来的文件内容自然是错的，CRC校验肯定过不了。

怎么修复？ #

核心就是每个请求单独创建ZipFile实例，让每个线程都用自己的资源副本，别共享：

// 每个请求处理时，单独初始化ZipFile，用try-with-resources自动关资源
try (ZipFile templateFile = new ZipFile(new File(dir_to_zip_file))) {
    Enumeration<? extends ZipEntry> templateFileEntries = templateFile.entries();
    // 接着处理遍历、加文件、写响应流的逻辑
    while (templateFileEntries.hasMoreElements()) {
        ZipEntry entry = templateFileEntries.nextElement();
        // 你的条目处理代码...
    }
} catch (IOException e) {
    // 记得捕获异常做处理
    log.error("处理Zip文件出错", e);
}

额外提几个优化点 #

• 用try-with-resources：上面的写法能自动帮你关闭ZipFile，避免资源泄漏，比手动写close()靠谱多了。
• 别搞全局共享：绝对不要把ZipFile或者它的entries枚举放到类的成员变量里，一定要在每个请求的处理方法内部创建和使用。
• 加线程日志排查：如果还出问题，可以在处理逻辑里打个线程ID的日志，看看是不是还有资源共享的情况没清理干净。

为啥单请求没事多请求就炸？ #

单请求的时候，所有操作都在同一个线程里，ZipFile的内部状态不会被干扰；但多请求并发时，多个线程同时改同一个ZipFile的内部数据结构，比如读取指针被来回跳，读出来的内容全乱了，CRC校验值自然和预期对不上。

内容的提问来源于stack exchange，提问作者Fede Brasburg