Discussion :

[yacin]

bonjour,
voilà j'ai un petit souci, j'utilise un menu déroulant dans une page jsp (Strut) en utilisant la balise html <select> et <logic:iterate> ce qui m'évitera de définir un formulaire. Mon problème est que j'ai des répétitions au niveau de mon menu déroulant, et je sais pas comment faire pour les éviter!
voici le code :

Code :

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

1
2
3
4
5
6
7
8
 
<select name="nomliste" id ="idlist">
<logic:iterate id="idConfig" name="tests">
 <option value="<bean:write name="idConfig" property="ConfigCode"/>">
    <bean:write name="idConfig" property="ConfigCode"/>
 </option>
</logic:iterate>
</select>

Merci beaucoups pour votre aide

[c_nvy]

Qu'entends-tu par répétitions ?

Sinon, pour afficher ta liste déroulante, il y a plus simple :

Code :

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

1
2
3
<html:select property="nomliste">
   <html:optionsCollection name="tests" value="configCode" label="configCode"/>
</html:select>

J'ai volontairement écrit configCode et non ConfigCode car selon les conventions de nommage Java, les noms des propriétés doivent commencer par une minuscule et une majuscule à chaque nouveau mot à partir du deuxième.
Tu devrais donc renommer ConfigCode en configCode.

[yacin]

merci pour ta réponse, ainsi que pour ta remarque, mais j'ai éviter d'utiliser <html:select> car sinon je serai obliger de définir un ActionForm associé.
mon problème est que je ne sais pas comment faire afin de ne pas avoir les mêmes valeurs dans le menu, c'est à dire dans : <option>
par exemple maintenant j'ai dans le menu {valeur1,valeur2,valeur1}

[c_nvy]

Et quel est l'intérêt de ne pas avoir d'ActionForm associée ?

Sinon, c'est au moment où tu constitues la collection tests qu'il faut t'assurer que tu n'as pas de doublons.

[yacin]

si je défini un nouveau ActionForm (un nouveau formulaire), il faut l'associer à une action et cela perturbe le fonctionnement général de l'application. par contre comme ça je récupére l'option sélectionné par javascript et je fait correctement mes traitement.
Est ce que c'est possible de faire un test à l'aide de java script ou bien directement insérer du code java avant l'ecriture dans <option>, afin de vérifier que cette valeur n'existe pas déja.
je ne peux modifié le bean retournant la collection car il est utilisé dans beaucoups d'autres appels!
j'ai pensé à faire ça mais je connais pas trops en javascript

Code :

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

1
2
3
4
5
6
7
8
9
10
 
 function testExist(){
    for (var i=0; i<document.getElementById('idlistConfig').options.length; i++) {
         if (document.getElementById('idlistConfig').options[i].value=="<bean:write name="idConfig" property="configCode"/>") {
                 return false;
        }
      }
        return true;
     }
    </script>

[c_nvy]

Mais sans modifier la méthode existante, tu pourrais très bien créer une nouvelle méthode qui retourne une collection d'objets distincts non ?

Sinon, il y a toujours moyen de mettre du code Java dans la jsp mais bon, c'est franchement dommage.

Je ne suis pas forte en Javascript mais le code que tu as posté ne pourra pas fonctionner car tu utilises un tag Struts qui s'exécute côté serveur avec du javascript qui s'exécute côté client.
De plus, vu la logique que tu appliques, cela suppose que les valeurs en double se suivent, ce qui n'est peut-être pas le cas.