Appel de Response.TransmitFile() à partir de la méthode statique

Question

J'ai un certain nombre de pages qui doivent prendre en charge l'exportation de données vers une feuille de calcul Excel. Je peux très bien générer les fichiers Excel, mais j'essaie de trouver un moyen d'abstraire ce comportement pour qu'il soit facilement réutilisable à partir de toutes les pages où j'en ai besoin. Mon idée actuelle est d'utiliser une méthode utilitaire statique, comme suit:

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); 
    } 
} 

Le gestionnaire de clic où je vous appelle SendExcelFile ressemble à ceci:

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

Ce code fonctionne très bien comme une méthode d'instance de la page d'appel. En tant que méthode statique, cependant, cela ne fonctionne pas du tout. Lorsque je clique sur le bouton qui l'invoque, le navigateur affiche indéfiniment les animations de chargement, mais ne demande jamais le téléchargement d'un fichier.

Je suis très nouveau sur ASP.NET (et la programmation web en général), donc je suis sûr qu'il me manque quelque chose ici. Quelqu'un pourrait-il expliquer le comportement que je vois et proposer une alternative raisonnable à cette approche?

EDIT: Si je supprime l'appel à File.Delete() à la fin, la méthode fonctionne comme prévu. Est-ce que Response.TransmitFile() effectue le transfert de manière asynchrone?

EDIT 2: J'ai juste besoin d'appeler Response.Flush() avant de supprimer le fichier. Voir ma réponse ci-dessous. Merci!

Answer 1

Le problème était que le fichier temporaire était en cours de suppression avant l'envoi des données. Je juste besoin d'appeler Response.Flush() comme si:

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

Essayez, vous pouvez obtenir le Request et Response directement sur 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); 
} 

Une autre alternative est une classe de base pour vos pages qui contient cette méthode, qui peut être une voie beaucoup plus facile. Vous pages n'avez pas hériter de System.Web.UI.Page, ils peuvent hériter de quelque chose d'autre, comme celui-ci:

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); 
    } 
} 

Ensuite, dans votre page la classe ressemble à:

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

Answer 3

Je voudrais utiliser cela à la place. Le contexte HTTP actuel sera disponible sur chaque page.

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

Nous avons besoin de plus d'informations - ce que vous faites devrait fonctionner.

J'ai créé une version allégée qui envoie juste une copie de la page d'appel au client et il fonctionne comme prévu:

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); 
    } 
} 

Voilà ma page d'appel:

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

-vous voir les différences de votre mise en œuvre?

Answer 5

À ce stade, j'utiliserais un proxy de débogage HTTP comme Fiddler pour comparer les sessions HTTP générées à la fois par les versions de travail (code codebehind) et non workwork (statique) de votre code.En outre, sachez que votre code tel qu'il est écrit ne fonctionnera pas bien si plusieurs utilisateurs cliquent sur le bouton en même temps - le fichier temporaire du premier utilisateur peut être écrasé par le fichier du second utilisateur, et le fichier du second utilisateur peut être supprimé en cours de transmission! Pensez à utiliser Path.GetTempFileName() ou un guid dans le nom de fichier pour vous assurer que le nom de chaque utilisateur est unique.