Impossible d'obtenir la syntaxe correcte pour dynam. créer onclick la manière ie

Question

J'espérais garder la compatibilité cross-browser et c'est le seul problème à ce jour.

.setAttribute("onclick", "return showRapidText("+itemnum+");"); 

Cela fonctionne parfait, mais je voudrais rendre IE compatible en le plaçant dans cette syntaxe

.onclick = new Function("fnDisplay_Computers('" + alines[i] + "')"); 

donc ... J'ai essayé

.onclick = new Function("showRapidText('" + itemnum + "')"); 

et

.onclick = new Function("return showRapidText('" + itemnum + "')"); 

et environ 40 autres façons mais rien RKS

Answer 1

element.onclick = function() { 
    return showRapidText(itemnum); 
}; 

Answer 2

+1 réponse J-P: les littéraux de fonction sont plus massivement que les chaînes. Pirater du code dans des cordes est une horreur que vous devriez éviter à tout prix. En outre, setAttribute ne devrait presque jamais être utilisé dans un HTMLDocument en raison de la compatibilité IE (et parce que les propriétés HTML DOM niveau 1 sont plus lisibles de toute façon).

Un piège potentiel cependant: si vous faites cela dans une boucle, ce qui semble être le cas, vous n'obtiendrez pas le comportement souhaité, en raison du closure loop problem. Vous pouvez résoudre cela avec une autre fermeture:

for (var i= 0; i<alines.length; i++) { 
    elements[i].onclick= function(i) { 
     return function() { 
      fnDisplay_Computers(alines[i]); 
     }; 
    }(i); 
} 

ou plus proprement en utilisant la cinquième édition ECMAScript disposent Function#bind:

for (var i= 0; i<alines.length; i++) { 
    elements[i].onclick= fnDisplay_Computers.bind(null, alines[i]); 
} 

Adding Function#bind aux navigateurs qui ne supportent pas encore.