Discussion :

[mima_mine]

bonjour j'ai fait 2 forward dans une action

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
 
 
public ActionForward execute(ActionMapping mapping, ActionForm form,
HttpServletRequest request, HttpServletResponse response) throws SQLException   {
 
 
		Connexion connexion=new Connexion();
		Connection con=connexion.createConnection();
		Statement st = null;
		ResultSet rs = null;
 
 
 
    	st = con.createStatement();
    	rs = st.executeQuery("select ART_TYPE_LIBELLE from GSS_ARTICLE_TYPE ");
 
 
    	while (rs.next()) {
    		String art = rs.getString(1).trim();
    	 	System.out.println(art);
 
 
    	 	if  (art.equals("SIMPOST")){
 
	 		return mapping.findForward("vue1");
    	 	break;
    	 }
 
 
 
    	 	else
    	 	{
    	 		return mapping.findForward("vue2");
 
    	 	}
    	}
 
 
	}

mais j'ai 2 erreurs

Code :

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

1
2
3
4
5
6
7
8
9
 
 
1. pour le return :  Return type for the method is missing
2. pour le break que j'ai placé dans le if : 
 
Multiple markers at this line
	- Line breakpoint:EssayConnectAction [line: 53] - execute(ActionMapping, ActionForm, 
	 HttpServletRequest, HttpServletResponse)
	- Unreachable code

j'ai essayé de faire l'action extends DispatchAction mais tjrs ya ce pb

svp pouvez-vous m'indiquer ou est l'erreur?

merci de me repondre

[c_nvy]

Déjà, pour éviter le break, pourquoi ne pas ajouter tout simplement une clause where ART_TYPE_LIBELLE='SIMPOST' directement dans le select.
Au résultat du select, tu auras un seul record ou aucun s'il ne trouve pas l'article SIMPOST.
Dans ce cas, tu n'as pas besoin de coder une boucle while mais directement un if ( rs.next() ).

Quand au return mapping.findForward, il faut le coder en dernière ligne de la méthode.

Ce que tu peux faire, c'est définir un objet de type String au début de ta méthode qui contient le nom du forward par défaut, soit vue2 :

Code :

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

String target="vue2" ;

et ensuite, affecter la valeur vue1 à la target quand il trouve l'article SIMPOST :

Code :

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

1
2
3
4
if ( rs.next() ) 
{
  target="vue1" ;
}

et à la fin de la méthode :

Code :

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

return mapping.findForward(target) ;

Par contre, si tu dois faire une DispatchAction, tu ne dois pas coder de méthode execute mais deux méthodes avec des noms différents et qui font chacune un forward vers la bonne page.