Go 1.21.4 archive/zip压缩最后一个文件出现静默异常

你的核心问题是没有调用zip.Writer的Close()方法。Go的archive/zip库要求必须调用writer.Close()来完成归档的收尾工作——包括写入每个文件的CRC校验值、原始文件大小，以及整个zip文件的中央目录结构，缺少这些信息Windows就会判定zip无效，7-Zip也会显示最后一个文件的元数据缺失。

代码问题分析 #

1. 你只延迟关闭了底层的archive文件句柄，但zip.Writer本身需要主动关闭才能完成元数据写入，单纯关闭archive不会触发这个流程。
2. 调用writer.Flush()没用，它只是把当前缓冲的文件内容写到底层文件，不会处理zip归档的核心元数据和中央目录。

修改后的代码 #

func zipFiles(clipFolder string, zipName string, files []string) int {
    archive, err := os.Create(zipName)
    if err != nil {
        _ = elog.Errorf("Error creating %s: %s", zipName, err.Error())
        return 0
    }
    defer archive.Close()
    
    writer := zip.NewWriter(archive)
    // 关键：必须延迟关闭writer，确保归档收尾工作完成
    defer func() {
        if err := writer.Close(); err != nil {
            elog.Errorf("Failed to close zip writer: %s", err.Error())
        }
    }()

    zipped := 0
    for _, file := range files {
        zipped += writeFileToArchive(writer, clipFolder, file)
    }
    return zipped
}

// writeFileToArchive adds a file to the zip archive and returns the number of files added (0 or 1)
func writeFileToArchive(writer *zip.Writer, path string, filename string) int {
    w, err := writer.Create(filename)
    if err != nil {
        elog.Warnf("could not add %s to archive", filename)
        return 0
    }

    fullPath := filepath.Join(path, filename)
    source, err := os.Open(fullPath)
    if err != nil {
        elog.Warnf("could not open %s file for reading", fullPath)
        return 0
    }
    defer source.Close()
    
    _, err = io.Copy(w, source)
    if err != nil {
        elog.Warnf("could not copy %s into archive", filename)
        return 0
    }
    // Flush可以移除，writer会在Close时处理所有缓冲
    // _ = writer.Flush()

    return 1
}

关键修改说明 #

• 新增defer writer.Close()：这一步会自动为所有写入的文件计算CRC、填充原始大小，并写入zip必需的中央目录，让归档文件完整有效。
• 移除了不必要的writer.Flush()：io.Copy已经会把文件内容写入到writer的缓冲，Close时会一次性完成所有收尾操作。

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