[Struts]Response already committed

Version imprimable

Bonjour,

Je me suis basé sur l'exemple de la FAQ pour envoyer au client un fichier avec Struts. Voici mon code :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 ByteArrayOutputStream out = new ByteArrayOutputStream(); // export excel ExportExcel export = new ExportExcel(); response.setContentType(export.getMimeType()); response.setHeader("Content-disposition", "filename=rechOrganeConso.xls"); export.setParameters(model, true, true, true); export.doExport(out); OutputStream outFinal = new BufferedOutputStream(response.getOutputStream()); outFinal.write(out.toByteArray()); outFinal.flush(); out.close(); outFinal.close();
L'export Excel marche bien, mais le forward echoue a cause de l'exception suivante :

Citation:

java.lang.IllegalStateException: Cannot forward. Response already committed.

Pouvez-vous m'expliquer qu'est ce que je ne fais pas dans le bon ordre ou ce qui ne va pas dans ce bout de code ?
D'avance merci !

La7su

15/02/2006, 12h03
denisC

Re: [Struts]Response already committed

Citation:

Envoyé par la7su

Pouvez-vous m'expliquer qu'est ce que je ne fais pas dans le bon ordre ou ce qui ne va pas dans ce bout de code ?
D'avance merci !

La7su

Qu'est ce que tu fais comme return après cette fonction?

Pourquoi ne pas exporter le fichier excel directement dans le outputstream de ta réponse? Le ByteArrayOutputStream ne sert à mon avis à rien du tout....
15/02/2006, 12h17
la7su
en fin d'action je fais :
Code:

1 2 return mapping.findForward(target);
Citation:

Pourquoi ne pas exporter le fichier excel directement dans le outputstream de ta réponse? Le ByteArrayOutputStream ne sert à mon avis à rien du tout....

Oui c'est juste, j'avais essayé cette méthode car je n'ai pas la main sur l'écriture de l'excel dans l'outputstream passé en parametre de l'export excel, je voulais donc vérifier que le pb ne venais pas de la. Dsl, j'ai oublié de virer la modif.
15/02/2006, 12h21
denisC
Citation:
Envoyé par la7su

en fin d'action je fais :

Code:

1 2 return mapping.findForward(target);
Ben voila ton problème....

Si la réponse est générée dans l'Action et est fermée dedans, tu ne peux plus rien écrire d'autre dedans, et donc le seul forward que tu puisses renvoyer à la fin de la méthode est:
Code:

1 2 return null;

Ok mais quelle est la différence entre ce que je fais et ce qui est fait dans la FAQ ?

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
 
 
response.setContentType("multipart/zip"); 
response.setHeader("Content-Disposition",
                   "attachment; filename=\"" + 
                   filename.trim().substring(1,filename.length()) + "\";"); 
response.setContentLength((int)f.length()); 
 
try
{
      OutputStream os = response.getOutputStream();
      FileInputStream stream = new FileInputStream(f);
      BufferedInputStream  bis = new BufferedInputStream(stream);
      InputStream is = new BufferedInputStream(bis);
      int count;
      byte buf[] = new byte[4096];
      while ((count = is.read(buf)) > -1)
      {
          os.write(buf, 0, count);
      }
      is.close(); 
      os.close();
}
catch (Exception ex)
{ 
   ex.printStackTrace();
}
..
return mapping.findforward("fichieenvoye");

A moins que le mapping.findForward("fichieenvoye") renvoie null (dans ce cas je pouvais pas le deviner...) je ne vois pas ce qui cloche dans ce que je fais...

En tout cas merci de ta reponse denisC !