Discussion :

[takinelinfo]

Bonsoir,
Alors là j'ai un gros soucis avec le js. Tout d'abord voici le code simplifier au maximun:

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
19
20
21
22
23
24
25
26
27
function ChargeListeAcheteur()
{
	var reponseXml 		= xmlHttpListeAcheteur.responseXML;
	var rootXmlElement	= reponseXml.documentElement;
 
	var divParent=document.getElementById('acheteur_bloc_resultat_liste');
	divParent.innerHTML="";
 
	var table = document.createElement('table');
 
	for(var i=0; i<(rootXmlElement.childNodes[1].childNodes).length;i++)
	{
		var id = rootXmlElement.childNodes[1].childNodes[i].childNodes[0].firstChild.data;
		var ligne = table.insertRow(-1);
		cell_nom = ligne.insertCell();
		cell_nom.innerHTML=rootXmlElement.childNodes[1].childNodes[i].childNodes[1].firstChild.data;
 
		alert(id);
		cell_nom.onclick=function(){ processChargeFiche(id); };				
	}
	divParent.appendChild(table);
}
 
function processChargeFicheAcheteur(elemt)
{
	alert(elemt);
}

C'est très simple, je construit une liste de membre en ajax, et je voudrais associer une fonction à l'évènement onclick pour chaques individu afin de visualiser sa fiche.
Imaginons 3 membres, je voudrais

membre 1 -> id=1 -> onclick=processChargeFiche('1');
membre 2 -> id=2 -> onclick=processChargeFiche('2');
membre 3 -> id=6 -> onclick=processChargeFiche('6');

Mon premier alert est la pour vérifier si j'ai les bon id, j'obtiens:

1
2
6

-> ok

par contre ils ont tous onclick=processChargeFiche('6');, il prend en compte le dernier id pour tous les membres, c'est à dire que quelque soit le membre sur lequel je clique, je vois systematiquement la fiche du dernier membre.
J'obtiens donc à l'écran:

6
6
6

J'ai essayé:
cell_nom.onclick="processChargeFiche(id)";
cell_nom.onclick="processChargeFiche("+id+")";
J'ai essayé avec une variable superglobale mais sans succés.

[DoubleU]

Code :

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

1
2
3
4
5
cell_nom.onclick=function(_id){ 
                          return function(){
                               processChargeFiche(_id);
                         } 
}(id);

Pour que chaque id soit locale à la fonction onclick a laquelle il doit s'appliquer

[marcha]

Salut,

L'approche de DoubleU me semble tout a fait pertinente. Cependant elle
crée autant de fonction qu'il y a d'éléments cliquables ce qui dans ton cas
n'est pas forcement nécessaire.

cell_nom.onclick= processChargeFiche;

permet d'appeler une seule et unique fonction pour tous.

et

Code :

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

1
2
3
4
 
function processChargeFiche() {
  var id = this.id; // permet de récupérer l'id
}

Autre remarque, tu "monopolise" le onclick de tous les éléments ce qui le
rend inexploitable pour d'autre usages futures par d'autres scripts qui pourraient
en avoir besoin. Plus d'info à ce propos ici

Encore une autre manière de voir serait d'associer un seul onclick à un parent
commun, et de détecter l'élément enfant réellement cliqué pour le traiter.