Créer un tableau de fonctions avec des namespaces

Bonjour,

J'essaye de créer un tableau de fonctions en Javascript, mais j'ai un petit soucis. Lorsque j'utilise ce tableau, les fonctions sont "undefined".

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20

var myns = myns || {}; myns.test = myns.test || {}; myns.test.util = { myOne: function(m) { return m; }, myTwo: function(m) { return m; }, processAllFunction: function(m) { for(var i=0; i<this.replaceFilters.length; i++) { if(typeof(this.replaceFilters[i])==='function') { m= this.replaceFilters[i](m); } } console.log(this.replaceFilters); // undefined functions return m; }, replaceFilters: [this.myOne, this.myTwo] };

---

Je déclare mon tableau en utilisant this.nomDeLaFonction et je pense que mon erreur est là.
Comment faire pour que ça fonctionne avec processAllFunction ?

Merci d'avance pour l'aide.

Tu as bien deviné : au moment où tu délares myns.test.util, le contexte est window, du coup ton tableau replaceFilters contient window.myOne et window.myTwo, qui évidemment sont indéfinis.

Une solution est de définir replaceFilters dans l'instruction suivante :

Code:

---

1 2 3 4 5 6

myns.test.util = { myOne: ... myTwo: ... processAllFunction: ... }; myns.test.util.replaceFilters = [myns.test.util.myOne, myns.test.util.myTwo];

---

Merci pour la réponse rapide.

J'avais déjà testé cette solution, elle me posait un problème :
Si j'ai des références à this dans une fonction, elle est undefined...

Exemple :

Code:

---

1 2 3 4 5 6 7 8 9

myns.test.util = { one: "Toto", myOne: function() { return this.one; // undefined }, myTwo: ... processAllFunction: ... }; myns.test.util.replaceFilters = [myns.test.util.myOne, myns.test.util.myTwo];

---

Je dois donc éviter d'utiliser this et utiliser le namespace à la place ? (ce qui augmente le taille du code et n'aide pas trop le développeur :aie:)

le this correspond à l'objet se trouvant devant le point (.) devant le nom de la méthode lors de l'appelle ! (sauf usage de call ou apply) exemple : objet.method();

donc si tu as

Code:

---

1 2 3 4 5 6 7

myns.test.util = { one: "Toto", myOne: function() { return this.one; } }; myns.test.util.myOne(); // affichera bien "Toto".

---

Par contre :

Code:

---

1 2 3 4 5 6 7 8

myns.test.util = { one: "Toto", myOne: function() { return this.one; } }; var func = myns.test.util.myOne; // sauve la fonction dans une variable func(); // appelle la fonction sans contexte, ce sera donc "window" l'objet courant et la fonction affichera undefined.

---

Enfin comme l'a signalé Watilin, lors de la création de l'objet littéral, tu ne te trouve dans aucune fonction, c'est donc window l'objet courant.

Ainsi :

Code:

---

1 2 3 4

myns.test.util = { one: "Toto", data: this.one // le this ici, vaut l'objet appelant de la fonction où tu te trouve, probablement window dans ton cas, donc this.one sera probablement undefined. };

---

Ah!

Donc quand je fais :

Code:

---

m = this.replaceFilters[i](m);

---

Le this pourrait être celui de la window au lieu de celui du namespace ?

Citation:

---

Envoyé par Aure7780

Ah!

Donc quand je fais :

Code:

---

m = this.replaceFilters[i](m);

---

Le this pourrait être celui de la window au lieu de celui du namespace ?

---

(j'ai édité/complété mon precédent message)

Ca dépend de où il se trouve, s'il se trouve dans une fonction appelé sur un objet alors this vaudra cet objet, sinon il vaudra window.

dans ton cas, si je m'en refère à ton premier message, cette ligne se trouve dans la méthode "processAllFunction" qui elle même sera probablement appelée depuis l'objet "util" donc le this vaudra bien (dans cette hypothèse) "util". :ccool:

edit:
dans ton code, la ligne qui est toujours mauvaise, c'est celle-ci :

Code:

---

replaceFilters: [this.myOne, this.myTwo]

---

vu que tu ne te trouves dans aucune fonction, les "this" valent "window".
le reste dépendra des appels.

Ok, merci pour vos réponses.
Je pense que je vais y arriver maintenant :ccool:

Si tu veux taper un peu moins de code, tu peux utiliser une closure. Le principe est simple, tu crées une fonction qui va s'exécuter immédiatement, et les variables locales sont protégées du mileu extérieur.

Code:

---

1 2 3 4 5 6 7 8

(function(){   var util = myns.test.util; util.replaceFilters = [util.myOne, util.myTwo];   })();   // ici, la variable util n'existe plus

---

Note que les closures peuvent être un moyen de se passer complètement des espaces de noms.