Envoye de Fichier au client

**Cronycs** · 26/04/2007, 17h34

Bonjour,

j'essaye d'envoyé un fichier a partir du serveur au client, j'ai donc suivi la FAQ et j'ai le code suivant dans une méthode execute d'une Action :

Code JAVA :

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
17
18
19
20
21
22
23
24
25
 
File f = new File(path);
response.setContentType("multipart/zip"); 
response.setHeader("Content-Disposition", "attachment; filename=\"" + f.getName().trim().substring(1,f.getName().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();
}

(C'est le code de la FAQ)

Mais lorsque je l'exécute, j'obtiens cette erreure :

Code erreur JAVA :

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

java.lang.IllegalStateException: "getWriter()" a déjà été appelé pour cette réponse

J'avoue être un peu perdu et ne pas savoir quoi faire .. je vous appelle donc a l'aide !

Merci d'avance !

**chtig** · 26/04/2007, 17h56

Tu peux donner la stack complète ? Notamment à quelle ligne ça se produit ...

**Cronycs** · 26/04/2007, 17h57

Ca se produit a la ligne suivante :

Code :

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

 OutputStream os = response.getOutputStream();

la stack entière donne ca :

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
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
java.lang.IllegalStateException: "getWriter()" a déjà été appelé pour cette réponse
	at org.apache.catalina.connector.Response.getOutputStream(Response.java:568)
	at org.apache.catalina.connector.ResponseFacade.getOutputStream(ResponseFacade.java:180)
	at javax.servlet.ServletResponseWrapper.getOutputStream(ServletResponseWrapper.java:101)
	at planning.web.action.FileUpAction.executeOk(FileUpAction.java:57)
	at planning.web.action.MyAction.execute(MyAction.java:66)
	at org.apache.struts.action.RequestProcessor.processActionPerform(RequestProcessor.java:431)
	at org.apache.struts.action.RequestProcessor.process(RequestProcessor.java:236)
	at org.apache.struts.action.ActionServlet.process(ActionServlet.java:1196)
	at org.apache.struts.action.ActionServlet.doGet(ActionServlet.java:414)
	at javax.servlet.http.HttpServlet.service(HttpServlet.java:689)
	at javax.servlet.http.HttpServlet.service(HttpServlet.java:802)
	at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:252)
	at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:173)
	at planning.web.filter.FilterSpace.doFilter(FilterSpace.java:27)
	at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:202)
	at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:173)
	at org.displaytag.filter.ResponseOverrideFilter.doFilter(ResponseOverrideFilter.java:125)
	at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:202)
	at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:173)
	at org.apache.catalina.core.StandardWrapperValve.invoke(StandardWrapperValve.java:213)
	at org.apache.catalina.core.StandardContextValve.invoke(StandardContextValve.java:178)
	at org.apache.catalina.core.StandardHostValve.invoke(StandardHostValve.java:126)
	at org.apache.catalina.valves.ErrorReportValve.invoke(ErrorReportValve.java:105)
	at org.apache.catalina.core.StandardEngineValve.invoke(StandardEngineValve.java:107)
	at org.apache.catalina.connector.CoyoteAdapter.service(CoyoteAdapter.java:148)
	at org.apache.coyote.http11.Http11Processor.process(Http11Processor.java:869)
	at org.apache.coyote.http11.Http11BaseProtocol$Http11ConnectionHandler.processConnection(Http11BaseProtocol.java:664)
	at org.apache.tomcat.util.net.PoolTcpEndpoint.processSocket(PoolTcpEndpoint.java:527)
	at org.apache.tomcat.util.net.LeaderFollowerWorkerThread.runIt(LeaderFollowerWorkerThread.java:80)
	at org.apache.tomcat.util.threads.ThreadPool$ControlRunnable.run(ThreadPool.java:684)
	at java.lang.Thread.run(Unknown Source)

**chtig** · 26/04/2007, 18h35

Tu peux nous poster l'intégralité de ta méthode exécute ?

**Cronycs** · 27/04/2007, 09h07

Désolé, j'étais pas la hier soir, donc ma méthode execute :

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
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
protected void executeOk(ActionMapping mapping, ActionForm form, HttpServletRequest request, HttpServletResponse response) throws SQLException {
		target = "up";
 
		if(request.getServletPath().startsWith("/resetFileUp")){
			target = "reset";
			return;
		}
		if(request.getServletPath().startsWith("/sendFichier")){
			target = "ok";
			String path = request.getParameter("path");
			System.out.println(path);
 
			File f = new File(path);
			response.setContentType("multipart/zip"); 
			response.setHeader("Content-Disposition",
			                   "attachment; filename=\"" + 
			                   f.getName().trim().substring(1,f.getName().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;
		}

qui extends d'une classe MyAction avec pour vrai méthode execute ceci :

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
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
public ActionForward execute(ActionMapping mapping, ActionForm form, HttpServletRequest request,
			HttpServletResponse response) {
 
		if (logger == null) {
			initLogger();
		}
 
		logger.info(" Enter in controller ");
		time = System.currentTimeMillis();
		msg = new ActionMessages();
		target = "erreur";
 
 
		if (!baseConnect.isConnected()) {
			target = "erreurDB";
		} else if (verifierSession(request.getSession()) && isErrEmpty()) {
			try {
				readDroit();
				executeOk(mapping, form, request, response);
			} catch (SQLException e) {
				target = "erreur";
				addErr("erreur", "errors.bd");
				logger.error("SQLException", e);
			}
		}
 
		saveErr(request);
 
 
 
		ActionForward af = (mapping.findForward(target));
		if (af != null) {
			logger.info("Exit controller | Time: " + (System.currentTimeMillis() - time) / 1000.0 + " | Target: "
					+ target + " | Path: " + af.getPath());
		} else {
			logger.info("Exit controller | Time: " + (System.currentTimeMillis() - time) / 1000.0 + " | Target: "
					+ target);
		}
		return af;
 
	}

**OButterlin** · 27/04/2007, 09h35

J'ai fait un traitement similaire (qui fonctionne). J'ai juste une petite différence...

Code :

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

1
2
3
4
5
6
7
 
... Tu utilises ceci
OutputStream os = response.getOutputStream();
 
Il faudrait utiliser ceci
 
ServletOutputStream out = response.getOutputStream();

Le reste reste inchangé...

( J'utilise également setContentType("unknown/unknown"); )

A+

**Cronycs** · 27/04/2007, 09h49

J'ai toujours la même erreur

j'ai modifié mon code comme suit :

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
17
18
19
20
21
22
23
24
File f = new File(path);
response.setContentType("unknown/unknown");
response.setHeader("Content-Disposition", "attachment; filename=\"" + f.getName().trim().substring(0,f.getName().length()) + "\";"); 
response.setContentLength((int)f.length()); 
try
{
	ServletOutputStream 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;

j'ai mis "0" comme 1er attribut du substring car il m'enlevait a chaque fois la 1ere lettre de mon fichier ... Est-ce que l'erreur ne pourrait pas venir du path qui est mauvais ? Pourtant le fichier existe bien..

**zag** · 27/04/2007, 10h01

slt,

comme énoncé dans ton exception, le probleme vient du fait qu'un getWriter a déjà été effectué sur ton objet response au moment ou tu fais un getOutputStream, et tu peux faire soit l'un soit l'autre mais pas les 2!

A priori, j'imagine que c'est l'une de ces lignes :

Code :

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

1
2
3
4
 
response.setContentType("unknown/unknown");
response.setHeader("Content-Disposition", "attachment; filename=\"" + f.getName().trim().substring(0,f.getName().length()) + "\";"); 
response.setContentLength((int)f.length());

qui pose probleme en appelant getWriter avant response.getOutputStream().

Tu devrais donc tenter a mon avis de faire un response.getWriter à la place de response.getOutputStream, puis de travailler sur le PrintWriter récupéré à la pace de ton ServletOutputStream (tu trouveras les mêmes méthodes write ...)

a+

**OButterlin** · 27/04/2007, 10h04

Il faudrait que ton contrôleur étende Action ou DispatchAction mais pas MyAction
(je pense que le problème vient de saveErrors)

A+

**Cronycs** · 27/04/2007, 10h36

Envoyé par OButterlin

Il faudrait que ton contrôleur étende Action ou DispatchAction mais pas MyAction
(je pense que le problème vient de saveErrors)

A+

Avec ceci j'ai tjs la meme erreure

Par contre, avec l'idée de Zag, j'arrive a récupérer le PrintWriter, mais le problème est que ca me génere un fichier corrompu, apparemment car il écrit tout le buffer (même s'il y en a trop a la fin) dans le fichier, et j'avoue que je ne sais plus comment on fait pour passé outre ca.

Mon nouveau code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
PrintWriter os = response.getWriter();
FileReader fr = new FileReader(f);
int count;
char buf[] = new char[4096];
while((count = fr.read(buf)) > -1){
 
	os.write(buf, 0, count);
 
}
fr.close();
os.close();

**OButterlin** · 27/04/2007, 10h54

C'est vrai qu'en regardant de plus près ce que j'avais codé, j'étends HttpServlet et pas Action (ou DispatchAction etc...)
Je crois que Struts fait des traitements sur request (ça c'est certain) et response (peut-être)
Le premier, j'en suis sûr, j'ai du passer par un Filtre pour traiter correctement l'encodate UTF-8...
Ca t'obligerait à definir et mapper la servlet dans le fichier web.xml, c'est tout...

A+

**Cronycs** · 27/04/2007, 11h28

Ahem ..

Ca ne fait pas longtemps que je développe avec Struts, et j'ai du mal a comprendre ce que tu essayes de me dire ^^"

**OButterlin** · 27/04/2007, 11h52

Alors en résumé, fait étendre ton contrôleur de HttpServlet, renomme la méthode executeOk en service avec cette signature

Code :

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

1
2
3
4
5
 
public void service(HttpServletRequest request, HttpServletResponse response)
{
 ... le code de executeOk
}

et rajoute la servlet dans ton fichier web.xml

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
...
<servlet>
  <servlet-name>leNomVoulu</servlet-name>
  <servlet-class>package.nomClasse.servlet</servlet-class>
</servlet>
...
<servlet-mapping>
  <servlet-name>leNomVoulu</servlet-name>
  <url-pattern>/leNomMappeDansLaPage</url-pattern>
</servlet-mapping>

**zag** · 27/04/2007, 12h12

re,

je pense que ca doit venir du fait que tu ne fais pas de flush et qu'en plus tu fermes ton writer à la fin.
Concrètement, tu peux essayer comme ca :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
PrintWriter os = response.getWriter();
FileReader fr = new FileReader(f);
int count;
char buf[] = new char[4096];
while((count = fr.read(buf)) > -1){
 
	os.write(buf, 0, count);
	os.flush();				  
}
fr.close();

à vérifier ...

**Cronycs** · 27/04/2007, 14h58

Le problème a en effet l'air de venir de l'encodage qui est mal géré (et non pas du PrintWriter), je vais donc essayer de mettre en place la servlet comme le suggère OButterlin.

En tout cas merci beaucoup a vous deux ! ^^

Envoye de Fichier au client

Struts 1 Java

Discussions similaires

Partager

Partager