WearOS应用生成Zip文件部分设备损坏问题排查求助

你的代码核心问题是多线程并发操作同一个非线程安全的ZipOutputStream，这会直接破坏Zip文件的内部结构，这就是为什么部分设备（CPU调度快、核心数多的设备）会出现Zip损坏，而部分设备（线程调度较慢的设备）看似正常的根本原因。

具体问题细节：

• ZipOutputStream本身不支持多线程并发操作，多个线程同时调用putNextEntry()、write()方法时，不同条目的数据会互相覆盖，导致Zip的目录结构和文件内容混乱，最终触发解压错误（0x80004005是Windows系统针对文件格式损坏抛出的通用错误）。
• 每个Zip条目写入完成后未调用zipOutputStream.closeEntry()，原代码仅在所有线程结束后调用一次，这会导致条目无法被正确标记为结束，Zip文件结构不完整。
• 线程内的IO异常仅打印栈信息，未中断整个压缩流程，可能导致部分条目写入失败但仍生成损坏的Zip文件返回。
• 流的关闭逻辑不严谨，未使用自动资源管理机制，存在资源泄漏风险。

1. 改用单线程处理所有文件 #

压缩属于IO密集型任务，多线程并发写入同一个输出流不仅不安全，也不会提升性能（反而可能因IO竞争降低效率）。改为单线程依次处理每个文件，可彻底解决线程安全问题。

2. 完善Zip条目处理流程 #

每个文件写入完成后，必须调用closeEntry()标记当前条目结束，再处理下一个条目。

3. 用try-with-resources自动管理流 #

确保所有输入输出流被正确关闭，避免资源泄漏和文件损坏。

修改后的代码示例 #

private File zip(File[] files) {
    File zipFile = null;
    try {
        sendHandlerMessage(0, COMPRESS_STATE);

        zipFile = new File(files[0].getParentFile(), files[0].getName().replace("txt", "zip"));
        // 使用try-with-resources自动关闭输出流
        try (ZipOutputStream zipOutputStream = new ZipOutputStream(new BufferedOutputStream(new FileOutputStream(zipFile)))) {
            for (File file : files) {
                byte[] buffer = new byte[BUFFER_SIZE];
                // 自动管理文件输入流
                try (FileInputStream fileInputStream = new FileInputStream(file)) {
                    ZipEntry entry = new ZipEntry(file.getName());
                    zipOutputStream.putNextEntry(entry);

                    int count;
                    long fileSize = file.length();
                    long actualSize = 0;

                    while ((count = fileInputStream.read(buffer)) > 0) {
                        zipOutputStream.write(buffer, 0, count);
                        actualSize += count;
                        // 用浮点运算避免整数除法导致的进度计算错误
                        sendHandlerMessage((int) Math.ceil((actualSize * 100.0) / fileSize), COMPRESS_STATE);
                    }
                    // 每个条目写完后立即关闭
                    zipOutputStream.closeEntry();
                    file.delete();
                } catch (IOException e) {
                    e.printStackTrace();
                    // 出现异常时删除已生成的损坏Zip文件
                    if (zipFile.exists()) {
                        zipFile.delete();
                    }
                    throw e; // 抛出异常终止压缩流程
                }
            }
        }
    } catch (IOException e) {
        e.printStackTrace();
        zipFile = null; // 返回null表示压缩失败
    }
    SystemClock.sleep(10);
    return zipFile;
}

额外优化点 #

• 进度计算改用浮点运算，避免整数除法导致的进度显示错误。
• 异常发生时主动删除损坏的Zip文件，避免返回无效文件。
• 移除不必要的线程池，简化代码逻辑同时保证线程安全。

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