使用 ITextSharp 和 mvc 损坏的 pdf

Question

我正在尝试从 MVC3 网页生成 pdf。我已经看过所有常见的教程，但就像经常出现的情况一样，当一个人很匆忙并且不知道自己在做什么时，我正在用它做狗的早餐。

当我单击视图上的操作链接来生成 pdf 时，该文件似乎已创建，但当我尝试打开它时，我从 Adob​​e Reader 收到非常有用的消息：“...该文件已损坏，无法使用予以修复”。

我哪里出错了？

    public FileStreamResult PDFGenerator()
    {
        Stream fileStream = GeneratePDF();

        HttpContext.Response.AddHeader("content-disposition", "attachment; filename=form.pdf");

        return new FileStreamResult(fileStream, "application/pdf");
    }

    private Stream GeneratePDF()
    {
        MemoryStream ms = new MemoryStream();

        Document doc = new Document();
        PdfWriter writer = PdfWriter.GetInstance(doc, ms);
        doc.Open();
        doc.Add(new Paragraph("Hello"));

        ms.Position = 0;
        ms.Flush();

        writer.Flush();

        return ms;
    }

Answer 1

您必须关闭该文档。尝试这样：

public ActionResult PDFGenerator()
{
    var doc = new Document();
    using (var stream = new MemoryStream())
    {
        var writer = PdfWriter.GetInstance(doc, stream);
        doc.Open();
        doc.Add(new Paragraph("Hello"));
        doc.Close();
        return File(stream.ToArray(), "application/pdf", "test.pdf");
    }
}

但这很丑。我会向您推荐一种更加 MVC 的方法，其中包括编写自定义 ActionResult。这样做的另一个优点是，您的控制器操作将更容易单独进行单元测试：

public class PdfResult : FileResult
{
    public PdfResult(): base("application/pdf")
    { } 

    public PdfResult(string contentType): base(contentType)
    { }

    protected override void WriteFile(HttpResponseBase response)
    {
        var cd = new ContentDisposition
        {
            Inline = false,
            FileName = "test.pdf"
        };
        response.AppendHeader("Content-Disposition", cd.ToString());

        var doc = new Document();
        var writer = PdfWriter.GetInstance(doc, response.OutputStream);
        doc.Open();
        doc.Add(new Paragraph("Hello"));
        doc.Close();
    }
}

然后在您的控制器操作中：

public ActionResult PDFGenerator()
{
    return new PdfResult();
}

当然，这可以更进一步，让这个 PdfResult 采取将模型视为构造函数参数，并根据此视图模型上的一些属性生成 PDF：

public ActionResult PDFGenerator()
{
    MyViewModel model = ...
    return new PdfResult(model);
}

现在事情开始看起来不错。