[liste elements]

Version imprimable

[liste elements]

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:

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

12/10/2005, 16h49
Mr N.

Qu'est ce qui te fais dire qu'il ne marche pas ?
12/10/2005, 16h49
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 ;)

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

Code:

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...

Code:

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

12/10/2005, 16h59
agougeon

merci je test...

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:

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

12/10/2005, 17h08
Pymm

Effectivement c'est un !trouve
J'ai pas l'habitude d'utiliser les break dans les for
je trouve cela un peu brutal :lol:
12/10/2005, 17h10
Mr N.

C'est vrai que c'est brutal !
:arrow:

Code:

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