[Débutant] Gestion des erreurs

Version imprimable

Bonjour à tous,

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

Voici comment j'ai fait :

:arrow: Mon struts-config.xml :
Code:

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>
:arrow: Dans ma Form :
Code:

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; } [...]
:arrow: Ma JSP (avant) :
Code:

1 2 3 [...] <html:errors property="manufactNumberVide" /> [...]
:arrow: Dans mon .properties (avant) :
Code:

1 2 3 errors.header=   errors.footer= error.manufactNumber.vide=Numéro de fabrication incorrect.
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.

:arrow: Dans mon Action :
Code:

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 ?

:arrow: Ma page JSP (après) :
Code:

1 2 3 4 [...] <html:errors property="manufactNumberVide" /> <html:errors property="manufactNumberInexistant" /> [...]
:arrow: Dans mon .properties (après) :
Code:

1 2 3 4 errors.header=   errors.footer= error.manufactNumber.vide=Numéro de fabrication incorrect. error.manufactNumber.inexistant=Numéro de fabrication inexistant.
Merci d'avance pour votre aide ! :oops:

Gôm

07/06/2007, 11h43
Rom1984
Il faut que tu sauvegardes les erreurs avec la méthode saveErrors (dans ton Action)
Par exemple:
Code:

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

il fait appeler la méthode :

Code:

saveErrors(request, errors);

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

Code:

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:

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));
}

07/06/2007, 12h00
ghohm

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

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.) ?
07/06/2007, 12h02
mattheox

Citation:

Envoyé par ghohm

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
07/06/2007, 12h17
ghohm

Citation:

Envoyé par mattheox

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 :merci: