在MVC中使用memorystream和DotNetZip给出了#34;无法访问封闭的流"

时间:2012-10-16 16:58:38

标签: asp.net-mvc dotnetzip

我正在尝试使用DotNetZip组件在MVC方法中创建一个zipfile。

这是我的代码:

    public FileResult DownloadImagefilesAsZip()
    {
        using (var memoryStream = new MemoryStream())
        {
            using (var zip = new ZipFile())
            {
                zip.AddDirectory(Server.MapPath("/Images/"));
                zip.Save(memoryStream);

                return File(memoryStream, "gzip", "images.zip");
            }
        }
    }

当我运行它时,我得到一个“无法访问关闭的流”错误,我不知道为什么。

2 个答案:

答案 0 :(得分:15)

不要处置MemoryStreamFileStreamResult一旦完成将其写入回复,就会小心:

public ActionResult DownloadImagefilesAsZip()
{
    var memoryStream = new MemoryStream();
    using (var zip = new ZipFile())
    {
        zip.AddDirectory(Server.MapPath("~/Images"));
        zip.Save(memoryStream);
        return File(memoryStream, "application/gzip", "images.zip");
    }
}

顺便提一下,我建议您编写一个自定义操作结果来处理此问题,而不是在控制器操作中编写管道代码。不仅如此,您将获得可重复使用的操作结果,但请记住,您的代码非常低效=>您正在内存中执行ZIP操作,从而将整个〜/ images目录内容+ zip文件加载到内存中。如果此目录中有许多用户和大量文件,您将很快耗尽内存。

更有效的解决方案是直接写入响应流:

public class ZipResult : ActionResult
{
    public string Path { get; private set; }
    public string Filename { get; private set; }

    public ZipResult(string path, string filename)
    {
        Path = path;
        Filename = filename;
    }

    public override void ExecuteResult(ControllerContext context)
    {
        if (context == null)
        {
            throw new ArgumentNullException("context");
        }

        var response = context.HttpContext.Response;
        response.ContentType = "application/gzip";
        using (var zip = new ZipFile())
        {
            zip.AddDirectory(Path);
            zip.Save(response.OutputStream);
            var cd = new ContentDisposition
            {
                FileName = Filename,
                Inline = false
            };
            response.Headers.Add("Content-Disposition", cd.ToString());
        }
    }
}

然后:

public ActionResult DownloadImagefilesAsZip()
{
    return new ZipResult(Server.MapPath("~/Images"), "images.zip");
}

答案 1 :(得分:0)

无法发表评论。

达林的答案很棒!仍然收到内存异常,但必须添加response.BufferOutput = false;因此必须将内容处理代码移得更高。

所以你有:

...
        var response = context.HttpContext.Response;
        response.ContentType = "application/zip";
        response.BufferOutput = false;

        var cd = new ContentDisposition
        {
            FileName = ZipFilename,
            Inline = false
        };
        response.Headers.Add("Content-Disposition", cd.ToString());

        using (var zip = new ZipFile())
        {
...

以防万一不明显:)