Вызов Response.TransmitFile() из статического метода

Question

У меня есть несколько страниц, которые должны поддерживать экспорт данных в электронную таблицу Excel. Я могу генерировать файлы Excel просто отлично, но я пытаюсь понять, как абстрагироваться от этого поведения, поэтому его легко использовать повторно со всех страниц, где он мне нужен. Моя текущая идея заключается в том, чтобы использовать статический метод полезности следующим образом:

public static void SendExcelFile(System.Web.UI.Page callingPage, string downloadFileName, List<List<string>> data, string worksheetTitle) 
{ 
    string tempFileName = Path.GetTempFileName(); 

    try 
    { 
     // Generate file using ExcelPackage 
     GenerateExcelDoc(tempFileName, data, worksheetTitle); 

     callingPage.Response.AddHeader("Content-Disposition", "attachment;filename=" + downloadFileName); 
     callingPage.Response.ContentType = "application/vnd.openxmlformats-officedocument.spreadsheetml.sheet"; 
     callingPage.Response.AddHeader("Content-Length", new FileInfo(tempFileName).Length.ToString()); 
     callingPage.Response.TransmitFile(tempFileName); 
    } 
    finally 
    { 
     //When this is removed, the method works as expected. 
     if (File.Exists(tempFileName)) 
      File.Delete(tempFileName); 
    } 
} 

Обработчика щелчок, когда я звоню SendExcelFile выглядит следующим образом:

protected void lnkExport_Click(object sender, EventArgs e) 
{ 
    List<List<string>> dataList = GatherDataForSpreadsheet(); 
    Utility.SendExcelFile(this, "fileNameForDownload.xlsx", dataList, "MyReports"); 
} 

Этот код прекрасно работает как метод экземпляра вызывающей страницы. Однако, как статический метод, он вообще не работает. Когда я нажимаю кнопку, которая вызывает это, браузер показывает анимацию загрузки неограниченно, но никогда не запрашивает загрузку файла.

Я очень новичок в ASP.NET (и в основном в веб-программировании), поэтому я уверен, что здесь что-то не хватает. Может ли кто-нибудь объяснить объяснение поведения, которое я вижу, и предложить разумную альтернативу этому подходу?

EDIT: Если я удалю вызов в File.Delete() в конце, метод работает так, как ожидалось. Выполняет ли Response.TransmitFile() асинхронную передачу?

EDIT 2: Мне просто нужно было вызвать Response.Flush(), прежде чем я удалю файл. См. Мой ответ ниже. Спасибо!

Answer 1

Проблема заключалась в том, что временный файл удалялся до того, как данные были отправлены вниз. Мне просто нужно было вызвать Response.Flush() следующим образом:

public static void SendExcelFile(System.Web.UI.Page callingPage, string downloadFileName, List<List<string>> data, string worksheetTitle) 
{ 
    string tempFileName = Path.GetTempFileName(); 

    try 
    { 
     // Generate file using ExcelPackage 
     GenerateExcelDoc(tempFileName, data, worksheetTitle); 

     callingPage.Response.AddHeader("Content-Disposition", "attachment;filename=" + downloadFileName); 
     callingPage.Response.ContentType = "application/vnd.openxmlformats-officedocument.spreadsheetml.sheet"; 
     callingPage.Response.AddHeader("Content-Length", new FileInfo(tempFileName).Length.ToString()); 
     callingPage.Response.TransmitFile(tempFileName); 
     callingPage.Response.Flush(); //This is what I needed 
    } 
    finally 
    { 
     if (File.Exists(tempFileName)) 
      File.Delete(tempFileName); 
    } 
} 

Answer 2

Попробуйте это, вы можете получить Request и Response непосредственно от HttpContext.Current:

public static void SendExcelFile(string downloadFileName, List<List<string>> data, string worksheetTitle) 
{ 
    var context = HttpContext.Current; 
    string tempFolder = context.Request.PhysicalApplicationPath + "temp"; 
    string tempFileName = tempFolder + "tempFileName.xlsx" 

    if (!Directory.Exists(tempFolder)) 
     Directory.CreateDirectory(tempFolder); 

    // Generate file using ExcelPackage 
    GenerateExcelDoc(tempFileName, data, worksheetTitle); 

    context.Response.AddHeader("Content-Disposition", "attachment;filename=" + downloadFileName); 
    context.Response.ContentType = "application/vnd.openxmlformats-officedocument.spreadsheetml.sheet"; 
    context.Response.AddHeader("Content-Length", new FileInfo(tempFileName).Length.ToString()); 
    context.Response.TransmitFile(tempFileName); 

    File.Delete(tempFileName); 
} 

Другой альтернативой является базовым классом для ваших страниц, которые содержат этот метод, который может быть гораздо проще маршрутом. Вы страницы не должны наследовать от System.Web.UI.Page, они могут наследовать от чего-то другого, например:

public class BasePage : System.Web.UI.Page 
{ 
    public void SendExcelFile(string downloadFileName, List<List<string>> data, string worksheetTitle) 
    { 
     string tempFolder =Request.PhysicalApplicationPath + "temp"; 
     string tempFileName = tempFolder + "tempFileName.xlsx" 

     if (!Directory.Exists(tempFolder)) 
      Directory.CreateDirectory(tempFolder); 

     // Generate file using ExcelPackage 
     GenerateExcelDoc(tempFileName, data, worksheetTitle); 

     Response.AddHeader("Content-Disposition", "attachment;filename=" + downloadFileName); 
     Response.ContentType = "application/vnd.openxmlformats-officedocument.spreadsheetml.sheet"; 
     Response.AddHeader("Content-Length", new FileInfo(tempFileName).Length.ToString()); 
     Response.TransmitFile(tempFileName); 

     File.Delete(tempFileName); 
    } 
} 

Затем на странице класс выглядит следующим образом:

public partial class MyPage : BasePage 
{ 
    //Stuff! 
} 

Answer 3

Я бы использовал это вместо этого. Текущий HTTP-контекст будет доступен на каждой странице.

HttpContext.Current.Response.AddHeader("Content-Disposition", "attachment;filename=" + downloadFileName); 
HttpContext.Current.Response.ContentType = "application/vnd.openxmlformats-officedocument.spreadsheetml.sheet"; 
HttpContext.Current.Response.AddHeader("Content-Length", new FileInfo(tempFileName).Length.ToString()); 
HttpContext.Current.Response.TransmitFile(tempFileName); 

Answer 4

Нам нужна дополнительная информация - то, что вы делаете, должно работать.

Я создал урезанную версию, которая просто отправляет копию страницы вызывающему клиенту и он работает, как ожидалось:

public class Utility { 
    // This just sends the client a copy of the calling page itself 
    public static void SendExcelFile(Page callingPage) { 
     string path = callingPage.Request.PhysicalPath; 
     callingPage.Response.AddHeader("Content-Disposition", "attachment;filename=test.xls"); 
     callingPage.Response.ContentType = "application/vnd.openxmlformats-officedocument.spreadsheetml.sheet"; 
     callingPage.Response.AddHeader("Content-Length", new FileInfo(path).Length.ToString()); 
     callingPage.Response.TransmitFile(path); 
    } 
} 

Вот мое призвание страницы:

public partial class main : System.Web.UI.Page { 
    protected void SendButton_Click(object sender, EventArgs e) {   
     Utility.SendExcelFile(this); 
    } 
} 

ли вам см. какие-либо отличия от вашей реализации?

Answer 5

В этот момент я бы использовал прокси-сервер отладки HTTP, например Fiddler, для сравнения сеансов HTTP, сгенерированных как рабочими (страницами кода), так и неработоспособными (статическими) версиями вашего кода.

В стороне, вы должны знать, что ваш код в письменной форме не будет работать, если несколько пользователей одновременно нажимают кнопку - файл временного файла первого пользователя может быть перезаписан вторым файлом пользователя, и файл второго пользователя может быть удален в середине передачи! Рассмотрите возможность использования Path.GetTempFileName() или руководства в имени файла, чтобы гарантировать, что файл каждого пользователя уникально назван.