FileResult 引发 System.ObjectDisposedException: 无法访问已关闭的文件

Question

我正在创建一个在生成 Excel 文件后返回File下载的端点，我有 2 种方法，第一种是将对象作为异步返回FileStream第二个是返回从Http调用File下载。

许多人说我必须让流再次seek到开头，然后才能被FileResult读取，但这似乎不起作用。

第一种方法：

private async Task<FileStream> Generate(int projectId, DateTime period)
{
...
if (...)
{
using (FileStream fs = new FileStream(filename, FileMode.Create, FileAccess.Write))
{
...
return fs;
}
}
return null;
}

第二种方法：

[HttpPost]
public async Task<IActionResult> Index([FromBody]ReportFilter filter)
{
FileStream fs = await Generate(filter.projectId, DateTime.Parse(filter.period));
if (fs != null)
{
fs.Seek(0, SeekOrigin.Begin);
return File(fs, "application/vnd.openxmlformats-officedocument.spreadsheetml.sheet", "kpi.xlsx");
}
return Json(new { status="error", message="Error while processing request" });
}

不幸的是，它抛出：

System.ObjectDisposedException: Cannot access a closed file.
at System.IO.FileStream.Seek(Int64 offset, SeekOrigin origin)

[更新]
没有using块：

private async Task<FileStream> Generate(int projectId, DateTime period)
{
...
if (...)
{
FileStream fs = new FileStream(filename, FileMode.Create, FileAccess.Write);
IWorkbook workbook = new XSSFWorkbook();
ISheet sheet1 = workbook.CreateSheet("Sheet1");
sheet1.AddMergedRegion(new CellRangeAddress(0, 0, 0, 10));
var rowIndex = 0;
IRow row = sheet1.CreateRow(rowIndex);
row.Height = 30 * 80;
var cell = row.CreateCell(0);
var font = workbook.CreateFont();
font.IsBold = true;
font.Color = HSSFColor.DarkBlue.Index2;
cell.CellStyle.SetFont(font);
cell.SetCellValue("A very long piece of text that I want to auto-fit innit, yeah. Although if it gets really, really long it'll probably start messing up more.");
sheet1.AutoSizeColumn(0);
rowIndex++;
workbook.Write(fs);
return fs;
}
return null;
}

[更新]
使用 jalsh 的建议(通过在准备下载时重新打开 FileStream(：

if (System.IO.File.Exists(filename))
{
FileStream fs = new FileStream(filename, FileMode.Open, FileAccess.Read);
return File(fs, "application/vnd.openxmlformats-officedocument.spreadsheetml.sheet", "kpi.xlsx");
}

Answer 1

通常，当您想要使用这样的一次性对象时，最好注入所需的操作，而不是在创建它的方法之外公开一次性对象。

我已经简化了您的代码，但这是基本思想：

private async Task Generate(int projectId, DateTime period, Action<FileStream> operation)
{
using (FileStream fs = new FileStream(filename, FileMode.Create, FileAccess.Write))
{
operation(fs);
}
}

现在你这样称呼它：

public async Task Index(int projectId, string period)
{
await Generate(projectId, DateTime.Parse(period), fs =>
{
if (fs != null)
{
fs.Seek(0, SeekOrigin.Begin);
return File(fs, "application/vnd.openxmlformats-officedocument.spreadsheetml.sheet", "kpi.xlsx");
}
});
}

这允许operation(fs)在using语句结束之前完成。

Answer 2

using 语句在其作用域末尾调用dispose()。 请参阅 MSDN 链接，它有一个像你一样的情况...

您可以不使用 using 语句，并在完成对象时手动释放对象。或者，您可以再次重新打开文件流，或者可以从您的方法返回一个File实例，您可以使用该方法重新打开流

现在您已经共享了完整的代码，在我看来，您的Write调用正在释放 FileStream 或关闭它，您可以在执行 Write(( 调用后尝试重新打开文件流。