[Struts-Validator] Pb de formulaire validé dans tous les cas

Version imprimable

Bonjour,

J'essaye de valider un formulaire en suivant la méthode des validators du livre de Jean-charles Félicité sur Struts.

J'ai suivi ses instructions, mais mon formulaire se valide tout le temps, c'est à dire que la méthode execute() de l'Action correspondante au formulaire et executée à tous les coups alors qu'on ne devrais pas arriver là.

Je vous donne mes fichiers :

le struts-config.xml

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
<struts-config>
    <form-beans>
        <form-bean name="NouvelAgentForm" type="com.myapp.struts.ActionForm.NouvelAgentForm"/>
        <form-bean name="NouvelArretForm" type="com.myapp.struts.ActionForm.NouvelArretForm"/>
        <form-bean name="ListeArretsAgentForm" type="com.myapp.struts.ActionForm.ListeArretsAgentForm"/>
        <form-bean name="ChoixAgentForm" type="com.myapp.struts.ActionForm.ChoixAgentForm"/>
         <form-bean name="LoginActionForm" type="com.myapp.struts.ActionForm.LoginActionForm"/>
    </form-beans>
 
    <action input="/Vues/listeArretsAgent.jsp" name="NouvelArretForm" path="/nouvelArret" scope="session" type="com.myapp.struts.Action.NouvelArretAction">
            <forward name="afficheFormulaire" path="site.nouvelArret.page"/>
            <forward name="ajoutSuccess" path="site.ajoutArretOk.page"/>
        </action>
 
 
    <!-- ========================= Validator plugin ================================= -->
    <plug-in className="org.apache.struts.validator.ValidatorPlugIn">
        <set-property
        property="pathnames"
        value="/WEB-INF/validator-rules.xml,/WEB-INF/validation.xml"/>
    </plug-in>

Le validation.xml

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
 
 <formset>
 
        <!-- FORMULAIRE D'AJOUT D'ARRET -->
        <form name="NouvelArretForm">
            <field
                property="dateDebut"
                depends="required">
                <arg key="NouvelArretForm.dateDebut"/>                
            </field>
            <field
                property="dateFin"
                depends="required, date">
                <arg key="NouvelArretForm.dateFin"/>
                <var>
                    <var-name>datePatern</var-name>
                    <var-value>dd/MM/YYYY</var-value>
                </var>
            </field>
 
        </form>
 
    </formset>

La jsp

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
<html:form action="nouvelArret">
    <html:errors/>
    <table border="0">
        <tr><td>Motif de l'arrêt :</td><td>
            <html:select property="motif"> 
                <html:optionsCollection property="listeMotifs" value="value" label="label"/> 
            </html:select> 
        </td></tr>
        <tr><td>Date de début :</td><td><input type="text" name="dateDebut" value="jjmmaaaa" /></td></tr>
        <tr><td>Date de fin :</td><td><input type="text" name="dateFin" value="jjmmaaaa" /></td></tr>
        <tr><td>Commentaires :</td><td> <textarea name="memo" rows="4" cols="20"></textarea></td></tr>   
    </table>
    <br>
    <html:submit>Ajouter cet arrêt</html:submit>
</html:form>

Et l'ActionForm

Code:

1
2
3
4
5
6
7
8
9
public class NouvelArretForm extends org.apache.struts.validator.ValidatorForm {
 
    private ArrayList listeMotifs=new ArrayList();
    private String motif;
    private String dateDebut;
    private String dateFin;
    private String memo;
 
.....getters/setters et constructeur....

Voilà, je ne comprend pas le formulaire est validé à chaque fois !

merci d'avance

Seb.

11/04/2006, 12h05
c_nvy

Tu n'aurais pas par hasard laissé la méthode validate dans ton ActionForm NouvelArretForm ?
Il me semble en avoir vu une dans ton précédent post. :wink:

Si ta validation est seulement basée sur le validator, il ne faut pas redéfinir de méthode validate.
11/04/2006, 13h56
vallica

Salut,

non ca ne change rien, mon formulaire est quand même validé lorsque je ne rempli pas les 2 champs date...
11/04/2006, 13h59
c_nvy

J'étais en train de modifier ma précédente réponse parce qu'en fait, le validate="true" est positionné par défaut si tu n'en codes pas un.
Donc ma remarque n'était pas valable. Je l'ai donc supprimé. :oops:

Regarde donc ce que j'ai écrit à la place.

non j'ai bien fait gaffe à virer la validate() et pas non plus de reset()

Voici ma méthode exectue() de l'Action à tout hasard, mais je ne pense pas qu'elle soit concernée par ce problème.

A savoir que j'arrive sur ce formulaire en appellant l'Action un première fois, qui ensuite redirige vers la vue, mais je ne vois pas en quoi ca empecherai la validation de se dérouler...

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
 
  public ActionForward execute(ActionMapping mapping, ActionForm  form, HttpServletRequest request, HttpServletResponse response)   throws Exception {
 
        // Test si premier passage, ou soumission du formulaire
        int idMotif=0;
 
        if (request.getParameter("motif")!=null)
            idMotif = Integer.parseInt(request.getParameter("motif"));
 
 
        // On rentre ici si un motif a été choisi dans la liste déroulante
        if (idMotif>0){
            // ici on crée l'arrêt, car si on est là c'est que le form a été valiudé
 
            return mapping.findForward("ajoutSuccess");
 
        }else {
            // Avant d'afficher le formulaire de saisie, on doit récuperer les motifs type d'arret
            // Passe ici avant le premier affichage afin de récuperer la liste qui remplira le select
            NouvelArretForm c=(NouvelArretForm) form;
            ArrayList l = new MotifArretMetier().getListePourSelect();
            c.setListeMotifs(l);
            return mapping.findForward("afficheFormulaire");
 
        }       
 
    }

11/04/2006, 14h11
c_nvy

Et si tu enlèves l'attribut value="jjmmaaaa" pour les deux champs dates dans ta jsp et que tu fais submit sans renseigner les champs dates, ça donne quoi ?
11/04/2006, 14h15
vallica

J'ai déjà essayé ca ce matin, et je viens de le refaire, c'est pareil le formulaire est validé ! c'est quand même épatant
11/04/2006, 14h37
yolepro

Je comprend pas bien comment fonctionne ton appli ? tu n'as qu'une seule action dans ton struts-config ??

tu reboucles dessus a chaque fois ?
Si c'est le cas, rien de plus normal qu'il utilise la validation de formulaire a chaque fois que tu valides.
11/04/2006, 14h40
c_nvy

Ce qui me gêne, c'est que c'est la même action qui initialise le formulaire et qui ensuite est appelée au submit du formulaire. :wink:

PS : Pardon yolepro, je n'avais pas vu ta réponse. :wink:
11/04/2006, 14h43
vallica

j'ai trouvé que ca pour initialiser le select qui est chargé depuis la base de données, si vous avez une autre technique je suis preneur !
11/04/2006, 14h44
vallica

yolepro j'ai d'autres actions mais j'ai pas tout mis pour pas charger le topic
11/04/2006, 14h46
yolepro

Dans ce cas tu utilises deux Action bien distincte (a généraliser a ton application)

une BuilderAction qui va se charger de construire ta page a afficher et une ValidationAction qui se charge de récupérer les valeurs de ton fomulaire et de faire le fonctionnel (controle bloquant, insere en base...)

Ensuite dans ton BuilderAction tu forces le validate="false" et tu laisses par defaut a true dans l'action du ValidationAction.
11/04/2006, 14h50
c_nvy

C'est comme ça que je procède aussi. :wink:
Une action qui initialise le formulaire et une qui est appelée au submit du formulaire, les deux actions pointant sur le même ActionForm.
11/04/2006, 14h52
vallica

ah ok et j'appelle l'affichage de mon formuler par la builerAction toujours?

roo j'ai buggé mon appli je sais pas comment à force de chercher l'erreur du validator :cry:
11/04/2006, 14h53
c_nvy

En effet, la première action à appeler est celle qui initialise ton formulaire, celle que yolepro a appelé BuilderAction.

Ok je vais faire comme ca dorénavant, car j'ai souvent des formulaires à initialiser et ca sera plus clair en faisant comme vous dites !

par contre la c'est pas mal, j'ai rien touché à ma source de données et voilà ce que me sors tocmat lorsque je rentre sur l'appli et que je m'authentifie :
Code:

1 2 3 4 5 6 7 javax.naming.NameNotFoundException: Le Nom comp n'est pas lié à ce Contexte at org.apache.naming.NamingContext.lookup(NamingContext.java:769) at org.apache.naming.NamingContext.lookup(NamingContext.java:152) at org.apache.naming.SelectorContext.lookup(SelectorContext.java:136) at javax.naming.InitialContext.lookup(InitialContext.java:351) at cciacs.gesam.dao.dao.AdminDAO.<init>(AdminDAO.java:36) at com.myapp.struts.ActionForm.LoginActionForm.validate(LoginActionForm.java:61)

11/04/2006, 15h03
vallica

non c'est bon après redémarrage de Netbeans c'est redevenu normal...
11/04/2006, 16h30
yolepro

Si tu as fini avec ton problème, n'oublie pas le résolu
11/04/2006, 16h47
vallica

Juste une dernière question, plutot que d'ouvrir encore un topic :

tout marche, j'ai mis aussi en place le javascript, mais je me demande un truc : si on doit valider par des méthodes situées dans des classes metiers perso, on peut le faire avec les validators où on doit utiliser le validate() classique?

Tu peux très bien combiner la validation par le validator et une validation java classique.
Tu mets dans le validator tout ce qu'il est capable de gérer comme le required, date, ...
Et tu implèmentes une méthode validate dans ton ActionForm qui fait appel aux méthodes de tes classes métiers.

Pour que cela fonctionne, dans la méthode validate de ton ActionForm, il faut faire ceci :
Code:

1 2 3 4 5 6 7 8 9 10 public ActionErrors validate (ActionMapping mapping,HttpServletRequest request) { ActionErrors errors = super.validate(mapping,request); if ( ... ) { ActionMessage message = new ActionMessage(...) ; errors.add("...",message) ; } }

11/04/2006, 17h11
vallica

Ok, donc le super.validate() va lancer la validation définie dans le validation.xml, puis si ca passe, le code metier est executé, et si il ne renvoi aucune erreur le formulaire est validé c'est ca?

encore une autre question (je suis lourd!) qui concerne l'ActionForm, maintenant que c'est un ValidatorForm, est-ce que je peux toujours définir une méthode reset() et l'appeller une fois que j'ai fini mes actions afin de vider les champs.

Parce que actuellement vu que le bean est en session, les données dans les champs persitent.
11/04/2006, 17h17
vallica
C'est bon j'ai trouvé tout seul pour mon problème de champs qui gardent les valeurs, j'ai défini reset() et dans l'Action, après les traitements je fais ca :
Code:

1 2 3 NouvelArretForm c=(NouvelArretForm) form; c.reset(mapping,request);
11/04/2006, 17h27
c_nvy

Citation:

Ok, donc le super.validate() va lancer la validation définie dans le validation.xml, puis si ca passe, le code metier est executé, et si il ne renvoi aucune erreur le formulaire est validé c'est ca?

C'est bien cela. :wink:

Citation:

encore une autre question (je suis lourd!) qui concerne l'ActionForm, maintenant que c'est un ValidatorForm, est-ce que je peux toujours définir une méthode reset() et l'appeller une fois que j'ai fini mes actions afin de vider les champs.

Attention, il me semble que la méthode reset est automatiquement invoquée à la validation du formulaire et ce, avant la méthode validate.
11/04/2006, 17h33
vallica

mince... et si je place le code que j'ai mis dans reset() dans une autre methode genre efface() et que j'appelle cette methode?

vous vous le gérez comment ca d'habitude?