Discussion :

[WaterTwelve21]

Bonjour ,

Je suis sur un projet ASP.Net . J'ai besoin de lancer un téléchargement d'un fichier word (.docx) . Tout fonctionne correctement au premier téléchargement , mais impossible d'en effectuer des supplémentaire sans rafraichir la page.
Cela me renvoie une exception de type 'System.NullReferenceException' , classique .

Voici mon ActionResult en question :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
public ActionResult Download()
        {
            ReportByClientViewModel data = (ReportByClientViewModel)TempData["ReportByClient"];
            string filename = "export.docx";
 
            Response.AppendHeader("Content-Disposition", "attachment; filename=" + filename);
 
                MemoryStream s = new MemoryStream();
                ExportReport.ExportToWord(s, data.Report); //Bug at the second download
                s.Position = 0;
                s.CopyTo(Response.OutputStream);
 
 
 
            return File(s, "application/x-doc", filename);
        }

Je me doute que le problème vient du MemoryStream , mais si quelqu'un peut m'expliquer c'est pas de refus.

Merci

[Dokho1000]

Probablement la clôture de ton stream qui n'est pas faite. en tout cas dans le code fournis. Utilise un using avec ton stream pour être certain que la resource est bien relachée. (dispose)

[WaterTwelve21]

Merci , j'ai essayer avec le using mais téléchargement devient impossible puisque il est "Impossible d'accéder à un Stream fermé" ( c'est le message d'erreur) . En effet que lors de l'utilisation d'un MemoryStream le using est inutile.
J'ai plus l'impression que le problème vient d'une des variables de ma méthode ExportToWord ... je pense ....

En tous cas je te remercie.

[DotNetMatt]

Merci , j'ai essayer avec le using mais téléchargement devient impossible puisque il est "Impossible d'accéder à un Stream fermé" ( c'est le message d'erreur).

C'est probablement parce que tu as fermé le using trop tôt, avant le return.

En effet que lors de l'utilisation d'un MemoryStream le using est inutile.

Il faut utiliser using partout où c'est possible, cela fait partie des bonnes pratiques en C#. Un using n'est inutile que dans le cas où tu souhaites explicitement disposer ton objet par toi-même, ou bien sûr si l'objet n'implémente pas IDisposable.

Ton code devrait ressembler à ceci :

Code C# :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
using (var s = new MemoryStream())
{
	ExportReport.ExportToWord(s, data.Report); //Bug at the second download
	s.Position = 0;
	s.CopyTo(Response.OutputStream);
 
	return File(s, "application/x-doc", filename);
}

Si avec le code ci-dessus ca ne marche pas, alors le problème peut venir de ta méthode ExportToWord, auquel cas il faudra que tu postes son contenu.

Autre remarque, le type MIME pour un fichier DOCX c'est "application/vnd.openxmlformats-officedocument.wordprocessingml.document" et non "application/x-doc". Cela n'a rien à voir avec le problème que tu rencontres, mais autant en profiter pour le préciser.

[WaterTwelve21]

Merci pour ton aide .

Si j'utilise le using de cette manière le stream est considéré comme fermé , c'est parce que ce stream est dans un using dans ma méthode ExportToWord, il est libéré a cet endroit .

Merci pour type MIME , les deux fonctionnent.

[Dokho1000]

Merci pour ton aide . Si j'utilise le using de cette manière le stream est considéré comme fermé , c'est parce que ce stream est dans un using dans ma méthode ExportToWord, il est libéré a cet endroit . Merci pour type MIME , les deux fonctionnent.

La responsabilité de ta méthode n'est pas de clôturer ton stream. laisse le ouvert ça te permettra lors de futur utilisation de pouvoir faire plusieurs fois un export sans devoir ouvrir et fermer un flux. Retire donc le using dans méthode et met la à l'extérieur comme discuté précédement. Tu ne devrais plus avoir l'erreur que le flux est déjà clôturé. A+

[WaterTwelve21]

Rebonjour , j'ai viré le using dans ma méthode aucune amélioration , stream fermé dès le premier téléchargement . Le using de ma méthode s'occupe d'un WordprocessingDocument ( sdk OpenXML 2.5 ) pardon de m'être trompé .
J'ai trouvé mon erreur , il s'agit du deuxième paramètre de ma méthode qui est a null au second passage .

En tous cas merci , cela m'apprendra à ne pas suffisamment utiliser le débogueur