NPOI - 保存到文件后损坏.xlsx工作簿

时间:2017-06-22 09:42:27

标签: c# excel npoi

我使用此代码写入令人兴奋的excel文件。写入文件后,当我手动打开文件时,它已损坏。我正在使用NPOI二进制2.3.0.0请告诉我们如何避免excel被破坏。

[Authorize]
public void ExportUsers()
{
    var path = Server.MapPath(@"~\Content\ExportTemplates\") + "sample.xlsx";

    FileStream sw = new FileStream(path, FileMode.Open, FileAccess.Read);

    IWorkbook workbook = WorkbookFactory.Create(sw);       

    ISheet sheet = workbook.GetSheetAt(0);
    IRow row = sheet.GetRow(12);

    ICell cell = row.CreateCell(row.LastCellNum);
    cell.SetCellValue("test");
    workbook.CreateSheet("Ripon");            
    sw.Close();

    using (var exportData = new MemoryStream())
    {
        workbook.Write(exportData);
        string saveAsFileName = string.Format("Export-{0:d}.xls", DateTime.Now).Replace("/", "-");
        System.Web.HttpContext.Current.Response.ContentType = "application/vnd.ms-excel";
        System.Web.HttpContext.Current.Response.AddHeader("Content-Disposition", string.Format("attachment;filename={0}", saveAsFileName));
        System.Web.HttpContext.Current.Response.Clear();
        System.Web.HttpContext.Current.Response.BinaryWrite(exportData.GetBuffer());
        System.Web.HttpContext.Current.Response.End();                    
    }
}

新文件已创建但已损坏。我见过有人说这是在2.0.6版本中修复的,但仍然不适合我

1 个答案:

答案 0 :(得分:0)

这里有几个问题。

首先,您从.xlsx文件开始,然后将下载文件扩展名更改为.xls.xls.xlsx的文件格式不同;前者是二进制格式,后者是压缩XML格式。如果使用错误的扩展名保存文件,则Excel会在打开文件时将其报告为已损坏。

其次,您使用错误的方法从MemoryStream获取数据。 GetBuffer将返回整个分配的内部缓冲区数组,如果缓冲区未完全填满,该数组将包含超出数据末尾的任何未使用的字节。额外的空字节将导致下载的文件被破坏。如果您只想获取缓冲区中的数据,则应使用ToArray代替。

第三,看起来你正在使用ASP.NET MVC框架(基于方法上[Authorize]属性的存在)但是你直接操纵当前响应而不是使用控制器的内置用于返回可下载文件的File方法。我建议尽可能使用内置方法,因为这样可以使代码更清晰。

以下是更正后的代码:

[Authorize]
public ActionResult ExportUsers()
{
    var path = Server.MapPath(@"~\Content\ExportTemplates\") + "sample.xlsx";

    FileStream sw = new FileStream(path, FileMode.Open, FileAccess.Read);

    IWorkbook workbook = WorkbookFactory.Create(sw);

    ISheet sheet = workbook.GetSheetAt(0);
    IRow row = sheet.GetRow(12);

    ICell cell = row.CreateCell(row.LastCellNum);
    cell.SetCellValue("test");
    workbook.CreateSheet("Ripon");
    sw.Close();

    var exportData = new MemoryStream();
    workbook.Write(exportData);
    exportData.Seek(0, SeekOrigin.Begin);  // reset stream to beginning so it can be read

    string saveAsFileName = string.Format("Export-{0:d}.xlsx", DateTime.Now).Replace("/", "-");

    return File(exportData, "application/vnd.ms-excel", saveAsFileName);
}