pb changement classe javascript

[manuura]

Bonjour,

je me permets de solliciter votre aide car je sèche bêtement sur un bout de code javascript tout con depuis un bon moment.

Je m'explique.

Je souhaite au moment d'un clic sur un élément, modifier la classe de cet élément ainsi que celle d'autres éléments de la page.

J'ai donc un code html qui ressemble à ça :

<ul>
<li class="left"><a class="off" href="#;" onclick="changeEtat(); this.className='on';">bla</a></li>
<li class="mid"><a class="on" href="#;" onclick="changeEtat(); this.className='on';">bla</a></li>
<li class="right"><a class="on" href="#;" onclick="changeEtat(); this.className='on';">bla</a></li>
</ul>

avec comme fonction javascript :

function changeEtat() {
var a = document.getElementsByTagName('a');
	for (var i=0; a.length; i++) {
		if(a[i].className == 'on') {
			a[i].className = 'off';
		}
	}
}

Donc ma fonction va chercher tous les éléments de classe 'on' et les passe en 'off'.

Sauf que firebug me renvoit l'erreur suivante :

a[i] has no properties
[Break on this error] if(a[i].className == 'on') {

Cela a pour conséquence de bien mettre mes autres éléments en classe 'off' mais je pense qu'il y a un pb avant la fin de ma boucle et que du coup mon action this.className='on' n'arrive jamais.

Voilà, je bloque là-dessus donc si quelqu'un pouvait m'indiquer comment débugger mon code, je lui en serai très reconnaissant.

Merci

P.S. : au passage, comment puis-je inclure mon this.className='on' dans ma fonction ?

[manuura]

Résolu :

j'ai trouvé, il faut savoir écrire une boucle tout simplement :

for (var i in a) {
	  if(a[i].className == 'on') {
		 a[i].className = 'off';
	  }
   }

[captain_torche]

Je ne connaissais pas cette syntaxe : for(var i in a) : c'est l'équivalent d'un foreach ?

Sinon, l'erreur de ton précédent message était :

for (var i=0; i < a.length; i++)

[Bovino]

Pour ta question subsidiaire, rajoute

function changeEtat() {
var a = document.getElementsByTagName('a');
	for (var i=0; a.length; i++) {
		if(a[i].className == 'on') {
			a[i].className = 'off';
		}
	}
Event.srcElement.className='on';
}

ça devrait le faire.

Modifié 12 Mai 2008 par Bovino