Discussion :

[Bovino]

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

Ben, dans mon exemple aussi

Code :

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

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

[Watilin]

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

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.

[Bovino]

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

Il semblerait

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

[RomainVALERI]

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

[Bovino]

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

Bof bof...

Code :

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

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

[RomainVALERI]

Bof bof...

Code :

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

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é 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

[Bovino]

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

Code :

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

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

[Watilin]

Bonjour !
@Bovino : 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