使用ByteArrayOutputStream生成文件时IOUtils.copy优化出错求助

你遇到的问题其实是两个关键错误导致的，咱们一步步拆解：

错误点分析 #

• 数据流向完全搞反：你在try块里写的IOUtils.copy(is, fos)，其中is是从刚创建的空文件newFile来的FileInputStream——这完全颠倒了逻辑！我们要做的是把ZipInputStream中的压缩内容复制到新文件的输出流里，而不是从空文件读数据再写回去，这就是文件无内容的核心原因。
• 未处理目录类型的ZipEntry：如果压缩包中有目录，直接把new File(destDir, zipEntry.getName())当成普通文件处理，后续打开输出流会报错；第二轮迭代的FileNotFoundException也大概率是因为第一个目录没被正确创建，导致后续子文件的路径不存在。
• 冗余且错误的流声明：你根本不需要创建FileInputStream，数据源应该是当前指向zipEntry的ZipInputStream（也就是zis）。

正确的改写代码 #

用IOUtils.copy优化后的正确代码如下，同时补上目录处理和更严谨的流管理：

final InputStream targetStream = new ByteArrayInputStream(byteOut.toByteArray());
final File destDir = new File(System.getProperty("user.dir"));

// 用try-with-resources自动管理ZipInputStream的关闭
try (ZipInputStream zis = new ZipInputStream(targetStream)) {
    ZipEntry zipEntry = zis.getNextEntry();
    while (zipEntry != null) {
        File newFile = new File(destDir, zipEntry.getName());
        
        // 处理目录：如果当前ZipEntry是目录，先递归创建目录
        if (zipEntry.isDirectory()) {
            if (!newFile.mkdirs()) {
                // 可根据需求添加目录创建失败的处理逻辑
                break;
            }
        } else {
            // 确保父目录存在（应对压缩包中的嵌套文件路径）
            File parentDir = newFile.getParentFile();
            if (!parentDir.exists() && !parentDir.mkdirs()) {
                // 父目录创建失败的处理逻辑
                break;
            }
            // 核心：将ZipInputStream的内容复制到文件输出流
            try (OutputStream fos = new FileOutputStream(newFile)) {
                IOUtils.copy(zis, fos);
            }
        }
        
        // 关闭当前ZipEntry，保证流状态正确
        zis.closeEntry();
        zipEntry = zis.getNextEntry();
    }
} catch (IOException e) {
    e.printStackTrace();
    // 自定义异常处理逻辑
}

额外说明 #

• 用try-with-resources包裹ZipInputStream，避免手动关闭遗漏导致的资源泄漏。
• 每次处理完一个zipEntry后调用zis.closeEntry()，这是规范操作，能保证流的状态正确，避免后续迭代出现异常。
• 增加父目录创建逻辑，解决压缩包中嵌套路径文件的写入失败问题。

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