Discussion :

[agougeon]

Bonjour, voila j'ai une liste et je veu verifier si l'option qui est dedans n'est pas deja rentré.... voici mon code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
function ajouterTerme(chaine,id) {
	new_option = new Option(chaine,id);
	for (i = 0; i < document.getElementById('liste').options.length; i++)
    {
    	if(document.getElementById('liste').options[i] == new_option){
    	alert('Terme déjà selectionné!');
    	}else{
        document.getElementById('liste').options[document.getElementById('liste').options.length]=new_option;
        }
    }
}

Malheureusement ce code ne marche pas..... :'(
Merci

[Mr N.]

Qu'est ce qui te fais dire qu'il ne marche pas ?

[fred777888999]

C'est assez normal, tu teste si ton objet option est deja dans la liste des options de ton select (tu l'as cree avec un new, la reponse est donc forcement non de toute evidence) au lieu de tester si les caracteristiques de ce dernier (code//libelle) sont deja presentes dans ta liste.
++
Fred.

PS : plutot que 'le code ne marche pas', il est mieux de dire 'l'option est quand meme ajoutee' (par ex) c'est beaucoup plus simple de comprendre. Et plus tu pose ta question de facon simple est claire, plus il y aura de gens pour te repondre. Quand on a l'impression (je ne parle pas forcement pour toi) que les gens postent un code en attendant qu'on le debugge, on peut avoir tendance a ne pas s'y arreter

[agougeon]

oui merci tu as raison... je dois donc faire :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
function ajouterTerme(chaine,id) {
	new_option = new Option(chaine,id);
	for (i = 0; i < document.getElementById('liste').options.length; i++)
    {
    	if(document.getElementById('liste').options[i].value == new_option.value){
    	alert('Terme déjà selectionné!');
    	}else{
        document.getElementById('liste').options[document.getElementById('liste').options.length]=new_option;
        }
    }
}

ok, merci je test...

[Pymm]

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
function ajouterTerme(chaine,id) {
  trouve=false;
  i=0;
   while( i < document.getElementById('liste').options.length && !trouve)
    {
       if(document.getElementById('liste').options[i].value ==chaine ){
          trouve=true;
          alert('Terme déjà selectionné!');
       }     
       i++;           
    }
    if (trouve){
       new_option = new   Option(chaine,id);
document.getElementById('liste').options[document.getElementById('liste').options.length]=new_option;
    }
}

[agougeon]

merci je test...

[fred777888999]

Hummm, pas tout a fait, tu dois creer ton option ssi elle n'est pas presente dans ta liste, pas a chaque tour de boucle. Je n'ai pas teste, mais je verrai plutot qq chose comme

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
 
function ajouterTerme(chaine,id) { 
   trouve = false;
   for (i = 0; i < document.getElementById('liste').options.length; i++) 
    { 
       if(document.getElementById('liste').options[i].value == new_option.value){ 
       trouve=true;
       break;
       }
    } 
   if ( !trouve ) {
        new_option = new Option(chaine,id); 
        document.getElementById('liste').options[document.getElementById('liste').options.length]=new_option; 
   }
   else {
       alert('Terme déjà selectionné!'); 
   }
}

a debugger bien sur et a adapter, mais l'idee est la
++
Fred.

PS grille

[Pymm]

Effectivement c'est un !trouve
J'ai pas l'habitude d'utiliser les break dans les for
je trouve cela un peu brutal

[Mr N.]

C'est vrai que c'est brutal !

Code :

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

for (i = 0; i < document.getElementById('liste').options.length && !trouve ; i++)