Discussion :

[marcha]

Salut,

Tu aurai pu faire un peu plus court, mais c'est pas grave.

Ton problème est le suivant:

tu utilise liste.innerHTML += "....";

ce qui se traduit par liste.innerHTML = liste.innerHTML + "....";

Donc tu relis le contenu html de liste sous forme de chaine, tu ajoute le nouveau
code et tu écrase le innerHTML à nouveau.

Dans cette opération tu perds les noeuds dom sur lesquels tu as assigné
tes évènements, de nouvelles instances sont recrées à partir de la chaine.

Seule la dernière assignation d'évènement subsiste.

Tu devrais plutôt utiliser document.createElement et createTextNode pour
fabriquer tes inputs, ton texte et ton br

et surtout liste.appendChild(...) à chaque fois, ainsi tu ajoute au dom sans
supprimer les éléments déjà crées.

[speedev]

Mince...honnêtement je fermais les yeux sur le innerHTML car véritablement dans mon code j'ai plusieurs DIV et classes CSS qui m'imposent pas mal de createElement de setAttribute en tout genre etc... (un code beaucoup plus long donc) et comme j'étais pressé je suis passé par un innerHTML ne pensant pas sur le coup que ça perturberait les choses.

Au final j'ai perdu plus de temps avec cette foutue innerHTML...ça m'apprendra.

Merci.

[SpaceFrog]

un truc utile :
http://www.developpez.net/forums/d53...ynamique-form/

[speedev]

Je confirme que c'est OK avec les DOMs, merci Marcha...

Intéressant ton dev SpaceFrog, je vais me pencher la dessus dès que possible.

De mon côté, c'était vraiment de la "feignasserie" car j'ai codé le tout en DOM en qlq minutes... J'ai horreur d'être comme ça mais c'est de famille (vive les Bretons !)

Avouez que c'est tentant le innerHTML pour faire court...voici le code qui remplace la ligne de code fautive :

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
28
29
30
31
32
33
 
var _DIV = document.createElement("DIV");	_DIV.className="container_rc";			_DIV.setAttribute('onmouseover','this.style.backgroundColor=\'#fff\';');_DIV.setAttribute('onmouseout','this.style.backgroundColor=\'#f5f5f5\';');
var _TABLE = document.createElement("TABLE");
var _TBODY = document.createElement("TBODY");
var _TR = document.createElement("TR");
var _TD1 = document.createElement("TD");
_TD1.style.width = "20px";
_TD1.className = "c";									
try{  
	_RAD = document.createElement('<input type="radio" name="chp_code" value="'+code+'" />');  
}catch(bug){  
	_RAD = document.createElement('input');  
	_RAD.setAttribute('type','radio');  
	_RAD.setAttribute('name','chp_code'); 
	_RAD.setAttribute('value',code);
}  
	_RAD.id = code_relais; 
	_RAD.code = code;
	_RAD.valeur= valeur;
 
	addEvent(_RAD,"click", function () {
		alert(code+" "+valeur);
	}, false);
 
	_TD1.appendChild(_RAD);
       var _TD2 = document.createElement("TD");
	_TD2.innerHTML = "<span style=\"font-weight:bold\">"+code_relais+"</span><br />"+code+" - "+valeur+"</div><div class=\"clear\">";
	_TR.appendChild(_TD1);
	_TR.appendChild(_TD2);
	_TBODY.appendChild(_TR);
	_TABLE.appendChild(_TBODY);
 
	liste.appendChild(_TABLE);

[marcha]

speedev, tu devrais t'intéresser à des Framework si tu aimes vraiment
la "feignasserie" :-)

Deux d'entre eux devraient t'aider si tu fais beaucoup de choses
similaires à ton post.

Ma préférence pour celui-ci: Jquery
ou prototype suivant les goûts.

[speedev]

Oui Marcha
Mais à vrai dire, c'est plutôt les plannings serrés qui me font choisir des raccourcis à tord plutôt que de la vraie feignantise. Je ne suis pas adepte des frameworks mm si j'utilise prototype de temps à autre pour des petits gadgets rigolos.
J'ai encore trop de mal à vouloir standardiser ma façon de développer au moyen d'outils tous beaux tous propres que tout le monde connait. Mais je pense que j'y viendrai...comme beaucoup d'autres....