Discussion :

[typikal]

Bonjour tlm,
Je rencontre un problème avec une boucle javascript qui s'arrête plus tôt que ce qui est prévu par la condition, la boucle ne va pas jusqu'au bout de la condition d'arrêt.

le code source :

Code :

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

1
2
3
4
5
6
7
8
9
10
function toutSuppr(f)
{
var nbrlig = f.options.length;
var i = 0;
while (i<nbrlig)
{
f.options[i]=null;
i++;
}
}

cette fonction sert à supprimer tout le contenu d'une liste déroulante.
f est à remplacer par this.form.nom_de_la_liste
la variable nbrlig est le nombre d'element contenu dans la liste

j'ai essayé les 3 types de boucles : while, for et do while ; le problème est le même la boucle s'arrête à mi chemin de son execution.

avez vous déjà rencontré ce type de problème avec javascript?

Merci d'avance pour votre aide.

typikal

[Bezout]

Pour vider une combo on fait comme ca :

Code javascript :

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

1
2
3
4
5
6
7
8
9
10
11
 
function viderCombo(cb) {
		if (!cb) return false;
		if (!cb.options) return false;
		// Suppression des items
		while (cb.options.length != 0) {
			cb.options.remove(0);
		}
		//-- OK
		return true;
}

[typikal]

ok Bizout
merci pour cette solution elle marche nikel
c'est vrai que c'est plus propre avec un remove

merci encore

[SpaceFrog]

SInon direct avec un pied de biche :

Code :

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

1
2
3
function toutSuppr(f)
{
f.options.length=0;}

[Bezout]

ok Bizout
merci pour cette solution elle marche nikel
c'est vrai que c'est plus propre avec un remove

merci encore

Sauf que je viens de me rendre compte qu'elle ne fonctionne que sous IE

Sinon le code suivant fonctionne sous IE et Firefox :

Code javascript :

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

1
2
3
4
5
6
7
8
9
10
11
function viderCombo(cb) {
	if (!cb) return false;
	if (!cb.options) return false;
	// Suppression des items
	while (cb.options.length != 0) {
		cb.options[0] = null;
	}
	//-- OK
	return true;
}

[javatwister]

alternative:

Code :

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

while(f.lastChild){f.removeChild(f.lastChild)}

[SpaceFrog]

oui ou comment se compliquer la vie

[Bezout]

alternative:

Code :

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

while(f.lastChild){f.removeChild(f.lastChild)}

Ca ca me plait bien et ca marche nickel IE/FF

[javatwister]

(je sais)

[SpaceFrog]

parceque ça ça marche sur la lune ???

Code :

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

1
2
3
4
 
function toutSuppr(f)
{
f.options.length=0;}

[Bezout]

euh on a jamais dit le contraire !

[javatwister]

quoi? qui parle à qui là?

[typikal]

Sauf que je viens de me rendre compte qu'elle ne fonctionne que sous IE

Sinon le code suivant fonctionne sous IE et Firefox :

Bezout, pour moi ta solution fonctionne sur IE sans problème, je l'ai mm simplifié


la solution au pied de biche SpaceFrog est encore plus simple et aussi efficace

javatwister merci pour ta solution mais bon.... je préfère faire simple cette fois

merci bien les mec

[Bezout]

oui mais elle ne fonctionne QUE SOUS IE. Ca c'est pas terrible.

Je savais pas que la propriété "length" etait en read/write. De ce fait la proposition de spacefrog me parait etre le meilleur compromis.

[typikal]

ok j'avais mal lu
j'avais compris que ça ne marchait pas sous IE

[SpaceFrog]

Il est vrai que la solution de javatwister est beaucoup plus propre.

Mettre de force le length à 0 c'est un peu brutal, mais jusqu'à présent je n'ai pas encore réussi à trouver en quoi cette methode pourrait être prise en défaut (le compte des childNodes est bon, le DOM semble s'actualiser correctement ...)

[typikal]

c'est vrai que la solution de javatwister est la plus propre
elle me sera utile pour la suite de mon dev