Discussion :

[grunk]

Bonjour à tous,

Je suis face à un problème que je n'arrive pas à résoudre , c'est sans doute simple mais j'ai le nez dedans depuis tellement longtemps que je ne me sort pas

J'ai dans un boucle besoin de construire X liens. Sur ces liens je dois appliquer un événement "click" :

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
 
for(var i=0;i<size;i++)
{
	var lien	= new Element('a',{href:'#'});
	Event.observe(lien,'click',function(event){importer(i)});
	lien.update(decodeURIComponent(data.files[i]));
	msgbox.insert({bottom:lien});
	msgbox.insert({bottom:new Element('br')});
	msgbox.insert({bottom:new Element('br')});
}
 
function importer(id)
{
 console.log(id);
}

Quand je vais cliquer sur les liens générer j'aurais en résultat la plus grande valeur de ma boucle pour chacun des liens.
En gros à chaque fois que j'attache un événement il remplace les précédents.

Comment puis je éviter cela ?

merci

[gwyohm]

Salut,

Il faut faire comme ca :

Code :

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

Event.observe(lien,'click',function(event, num){importer(num)}.bindAsEventListener(lien, i));

[grunk]

MErci pour cette réponse.
J'avais entre temps résolu le problème avec une closure :

Code :

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

1
2
3
4
5
6
7
8
9
 
(function(){        
var lien	= new Element('a',{href:'#'});
	Event.observe(lien,'click',function(event){importer(i)});
	lien.update(decodeURIComponent(i));
	msgbox.insert({bottom:lien});
	msgbox.insert({bottom:new Element('br')});
	msgbox.insert({bottom:new Element('br')});
})(i);

Mais le bindAsEventListener est bien plus simple