Discussion :

[ghohm]

Bonjour à tous,

j'ai bien regardé comment gérer les erreurs de saisies dans mon formulaire ... grâce à la FAQ Struts !

Voici comment j'ai fait :


Mon struts-config.xml :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
<action
	path="/FormAffichMachine"
	name="formChoixMachine"
	scope="session"
	validate="true"
	input="/choixMachine.do"
	type="com.manitou.pad.ChoixMachineAction"
>
	<forward name="succes" path="/vues/ficheMachine.jsp"/>
	<forward name="echec" path="/vues/choixMachine.jsp"/>
</action>

Dans ma Form :

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
[...]
// Validation
public ActionErrors validate(ActionMapping mapping, HttpServletRequest request) {
 
	// Gestion des erreurs
	ActionErrors erreurs = new ActionErrors();
 
	// Le numéro de fabrication doit être un numérique, de longueur : 6
	if (getManufactNumber() == null || getManufactNumber().length() < 6) {
		erreurs.add("manufactNumberVide", new ActionMessage("error.manufactNumber.vide"));
	}
 
	return erreurs;
}
[...]

Ma JSP (avant) :

Code :

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

1
2
3
[...]
<html:errors property="manufactNumberVide" />
[...]

Dans mon .properties (avant) :

Code :

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

1
2
3
errors.header=<span class="erreur">&nbsp;&nbsp;
errors.footer=</span>
error.manufactNumber.vide=Numéro de fabrication <i>incorrect</i>.

Jusqu'ici ... TOUT VA BIEN !


Par contre lorsque l'utilisateur saisit un numéro de fabrication "logique" : nnnnnn (un entier de 6 chiffres), je fais un contrôle dans mon Action et j'essaye de générer un nouveau message d'erreur.


Dans mon 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
if (id != 0) {
	// Récupération des données en BDD, puis affectation aux éléments de ma page JSP résultat
	
	return mapping.findForward("succes");
} else {
	// Gestion des erreurs
	ActionErrors erreurs = new ActionErrors();
	
	erreurs.add("manufactNumberInexistant", new ActionMessage("error.manufactNumber.inexistant"));
	
	//return erreurs;
	
	return mapping.findForward("echec");
}

Je suis obligé de faire comme cela, car ma méthode "validate" de mon Form n'a pas générer d'erreurs ... forcément la saisie de l'utilisateur est "logique" !

Vous suivez toujours ?!


MON PROBLEME est que la partie en rouge dans le dernier extrait de code, ne peut pas fonctionner ... on ne peut, évidemment pas, avoir deux "return" dans la même méthode !



MA QUESTION est donc : comment signaler à STRUTS qu'une nouvelle erreur a été détecté ? ou comment propager le "erreurs" jusqu'à ma page JSP ?


Ma page JSP (après) :

Code :

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

1
2
3
4
[...]
<html:errors property="manufactNumberVide" />
<html:errors property="manufactNumberInexistant" />
[...]

Dans mon .properties (après) :

Code :

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

1
2
3
4
errors.header=<span class="erreur">&nbsp;&nbsp;
errors.footer=</span>
error.manufactNumber.vide=Numéro de fabrication <i>incorrect</i>.
error.manufactNumber.inexistant=Numéro de fabrication <i>inexistant</i>.

Merci d'avance pour votre aide !

Gôm

[Rom1984]

Il faut que tu sauvegardes les erreurs avec la méthode saveErrors (dans ton Action)
Par exemple:

Code :

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

1
2
3
4
 
ActionMessages errors = new ActionMessages();
// ajout des erreurs ...
saveErrors(request.getSession(), errors);

[mattheox]

il fait appeler la méthode :

Code :

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

saveErrors(request, errors);

avant le return pour copier les erreurs dans la request.
soit :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
if (id != 0) {
	// Récupération des données en BDD, puis affectation aux éléments de ma page JSP résultat
	
	return mapping.findForward("succes");
} else {
	// Gestion des erreurs
	ActionErrors erreurs = new ActionErrors();
	
	erreurs.add("manufactNumberInexistant", new ActionMessage("error.manufactNumber.inexistant"));
	
	saveErrors(request, errors);
	
	return mapping.findForward("echec");
}

ps : Les normes de programmation Java préconnisent de ne faire qu'un seul return par méthode.. c'est bcp plus simple pour la lecture et le débogage.

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
 
String actionForward = "";
if (id != 0) {
	// Récupération des données en BDD, puis affectation aux éléments de ma page JSP résultat
	actionForward= "success";
} else {
	// Gestion des erreurs
	ActionErrors erreurs = new ActionErrors();
 
	erreurs.add("manufactNumberInexistant", new ActionMessage("error.manufactNumber.inexistant"));
 
	actionForward = "echec";
}
if (!errors.isEmpty()) {
	saveErrors(request, errors);
}
return (mapping.findForward(actionForward));
}

[ghohm]

Merci à tous les deux ! C'est exactement ce que je cherchais !


Merci pour le PS Mattheox ... j'ai également modifié mes "return".


Tant que vous êtes là, j'en profite pour poser une autre question !

Quel est le meilleur moyen de tester le contenu d'un champ texte (est-ce un numérique, un alpha-numérique, rien, etc.) ?

[mattheox]

Quel est le meilleur moyen de tester le contenu d'un champ texte (est-ce un numérique, un alpha-numérique, rien, etc.) ?

tu as la librairie Commons-lang qui a implémenter la plupart de ces tests surtout sur la class StringUtils

[ghohm]

tu as la librairie Commons-lang qui a implémenter la plupart de ces tests surtout sur la class StringUtils

Merci, mais l'on vient de me conseiller de me lancer carrément dans la technique "validator", exemple :

http://www.selbyinc.com/technology/validation.html


Gôm