C# 两个Zip压缩包之间文件复制的更优实现方案咨询

现有代码问题与可优化点 #

• 逻辑bug：逐字节读写使用do-while循环，会将读取结束的标记-1转换为0xFF写入目标文件末尾，导致目标文件损坏
• 读写效率问题：逐字节读写的方式在处理大文件时IO性能极差，内置的Stream.CopyTo方法默认使用8KB缓冲区，能大幅降低IO开销，同时也避免了手动写读写循环的逻辑错误
• 资源释放风险：手动调用Close()释放流的方式，在代码运行抛出异常时会出现资源泄露，推荐使用using语句自动管理流生命周期
• 冗余API调用：ZipArchive.CreateEntry执行后会直接返回新建的ZipArchiveEntry实例，无需再次调用GetEntry做冗余查找
• 缺少压缩策略配置：默认创建Entry使用系统默认压缩级别，可根据需求显式指定压缩策略，也可选择和源Entry保持一致的压缩参数

优化后代码示例 #

// 源文件流指定只读权限更安全
using FileStream sourceFileStream = new FileStream(source.FileName, FileMode.Open, FileAccess.Read);
// 若业务要求目标压缩包必须存在，可将OpenOrCreate改回Open
using FileStream targetFileStream = new FileStream(target.FileName, FileMode.OpenOrCreate, FileAccess.ReadWrite);
using ZipArchive sourceZip = new ZipArchive(sourceFileStream, ZipArchiveMode.Read);
using ZipArchive targetZip = new ZipArchive(targetFileStream, ZipArchiveMode.Update);

ZipArchiveEntry sourceEntry = sourceZip.GetEntry(filePathInArchive);
if (sourceEntry == null)
    return;

// 目标存在先删除旧Entry
var existingTargetEntry = targetZip.GetEntry(filePathInArchive);
existingTargetEntry?.Delete();

// 直接获取新建的Entry，此处选择和源Entry保持一致的压缩级别，也可自定义
ZipArchiveEntry targetEntry = targetZip.CreateEntry(filePathInArchive, sourceEntry.CompressionLevel);

// using自动管理流释放，无需手动Close
using Stream reader = sourceEntry.Open();
using Stream writer = targetEntry.Open();
// 内置缓冲区拷贝，效率远高于逐字节读写，也避免了手动循环的逻辑错误
reader.CopyTo(writer);

额外建议 #

如果是异步场景，可替换为CopyToAsync方法配合await使用，避免阻塞主线程；如果需要批量复制多个文件，可在打开一次两个ZipArchive的前提下批量处理Entry，避免重复解压压缩的开销。

内容的提问来源于stack exchange，提问作者a.rlx