1

没有一个类似的问题是我正在寻找的！

以下代码有什么问题？ files是文件内容的文本数组，fileNames是对应的文件名数组。

这段代码在使用 Save 方法的倒数第二行总是失败，但我不明白为什么流会被关闭！

result = new MemoryStream();

using (ZipFile zipFile = new ZipFile())
{
    for (int i = 0; i < files.Count(); i++)
    {
        System.Text.ASCIIEncoding encoding = new System.Text.ASCIIEncoding();
        Byte[] bytes = encoding.GetBytes(files[i]);
        using (MemoryStream fs = new MemoryStream(bytes))
        {
            zipFile.AddEntry(fileNames[i], fs);
        }
    }
    zipFile.Save(result);
}

感谢您的帮助 - 在这里变得绝望！

这是我基于@spender 的第一条评论的解决方案，尽管他在下面发布的解决方案可能更好。

        try
        {
            result = new MemoryStream();
            List<Stream> streams = new List<Stream>();

            if (files.Count > 0)
            {
                using (ZipFile zipFile = new ZipFile())
                {
                    for (int i = 0; i < files.Count(); i++)
                    {

                        System.Text.ASCIIEncoding encoding = new System.Text.ASCIIEncoding();
                        Byte[] bytes = encoding.GetBytes(files[i]);
                        streams.Add(new MemoryStream(bytes));
                        zipFile.AddEntry(fileNames[i], streams[i]);
                    }
                    zipFile.Save(result);
                }
            }
        }
        catch (Exception ex)
        {
            throw;
        }
4

2 回答 2

6

似乎调用Save是读取源流的时间点。这意味着您必须在保存之前保持它们未被处理。在这种情况下放弃using语句，因为不可能将其范围扩展到循环之外。相反，收集您的 IDisposables 并在保存完成后处理它们。

result = new MemoryStream();

using (ZipFile zipFile = new ZipFile())
{
    List<IDisposable> memStreams = new List<IDisposable>();
    try
    {
        for (int i = 0; i < files.Count(); i++)
        {
            System.Text.ASCIIEncoding encoding = new System.Text.ASCIIEncoding();
            Byte[] bytes = encoding.GetBytes(files[i]);
            MemoryStream fs = new MemoryStream(bytes);
            zipFile.AddEntry(fileNames[i], fs);
            memStreams.Add(fs);
        }
        zipFile.Save(result);
    }
    finally
    {
        foreach(var x in memStreams)
        {
            x.Dispose();
        }
    }
}
于 2013-07-11T00:44:01.517 回答
1

@spender 答案是正确的，但如果你想成为使用模式的代名词，我的 2 美分你可以做类似的事情

班级

public class DisposableBucket : IDisposable
{
    readonly List<IDisposable> listOfDisposables = new List<IDisposable>();

    public TClass Using<TClass>(TClass disposable) where TClass : IDisposable
    {
        listOfDisposables.Add(disposable);
        return disposable;
    }

    public void Dispose()
    {
        foreach (var listOfDisposable in listOfDisposables)
        {
            listOfDisposable.Dispose();
        }

    }
}

如何使用

result = new MemoryStream();
using(var bucket = new DisposableBucket())
{
    using (var zipFile = new ZipFile())
    {
        List<IDisposable> memStreams = new List<IDisposable>();

        for (int i = 0; i < files.Count(); i++)
        {
            System.Text.ASCIIEncoding encoding = new System.Text.ASCIIEncoding();
            Byte[] bytes = encoding.GetBytes(files[i]);

            var fs = bucket.Using(new MemoryStream(bytes));
            zipFile.AddEntry(fileNames[i], fs);

        }
        zipFile.Save(result);

    }
}
于 2018-08-09T15:17:26.703 回答