Appel d'une fonction javascript dans une page courante

Citation:

---

Envoyé par Watilin

Comme ça le test n’est fait qu’une fois ;)

---

Ben, dans mon exemple aussi :koi:

Code:

---

1 2 3 4 5 6 7 8 9 10

var addEvent = function(elt, declencheur, effet) { if (.addEventListener) { elt.addEventListener(declencheur, effet, false); addEvent = function(elt, declencheur, effet){elt.addEventListener(declencheur, effet, false);} } else if (elt.attachEvent) { elt.attachEvent(("on" + declencheur), effet); addEvent = function(elt, declencheur, effet) {elt.attachEvent(("on" + declencheur), effet);} } }

---

Sauf que là, le test n'est fait que s'il est utile :mrgreen:

Non non ! Ou alors y'a un truc qui m'échappe… :roll:

Avec ton code, le test est fait à chaque fois que tu attaches un évènement.
Avec le mien, il est fait au moment de définir la fonction addEvent.

Citation:

---

Envoyé par Watilin

Ou alors y'a un truc qui m'échappe… :roll:

---

Il semblerait :aie:

Dans mon if, j'attache l'événement voulu PUIS je réaffecte à ma variable la bonne version de la fonction ;)

J'adopte vos deux excellentes solutions :mrgreen:
(heu... tiens ce n'était pas mon post et j'ai encore appris un truc :aie:)

Citation:

---

Envoyé par RomainVALERI

J'adopte vos deux excellentes solutions :mrgreen:
(heu... tiens ce n'était pas mon post et j'ai encore appris un truc :aie:)

---

Bof bof...

Code:

---

1 2 3 4

if (document.all) { addEvent = function(obj, evt, func) { obj.attachEvent('on' + evt, func); };

---

Seul Firefox ne reconnait pas document.all et attachEvent n'est implémenté que sur IE et Opera, donc sur Chrome et Safari, ça risque d'être problématique :koi:

Citation:

---

Envoyé par Bovino

Bof bof...

Code:

---

if (document.all)

---

<... arguments ...>

---

Oui ta version semble en effet meilleure, mais je parlais notamment de l'idée, commune à vos deux solutions, de n'effectuer le test qu'une fois au départ plutôt qu'à chaque définition d'un gestionnaire d'événement ^^

Au sujet d'IE qui n'a pas accès à Element avant la version 8 >>> j'avais oublié :mur: j'avais décidé de me remettre d'ici quelques jours à travailler sur la version IE-compatible de mon générateur... ça va être un vrai plaisir :fou:

Non, à mon sens les deux se valent et ça reste ensuite une question de goût (et de couleurs :aie:), mais il faut modifier la condition :

Code:

---

1 2 3 4

if (window.attachEvent) { addEvent = function(obj, evt, func) { obj.attachEvent('on' + evt, func); };

---

Bonjour !
@Bovino : :mur: J’étais pas réveillé hier, j’ai relu trois fois ton code sans voir que tu réaffectais addEvent après le test… Sorry si je t’ai fait douter ;)