Discussion :

[Aure7780]

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 :

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

[Watilin]

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 :

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

1
2
3
4
5
6
myns.test.util = {
	myOne: ...
	myTwo: ...
	processAllFunction: ...
};
myns.test.util.replaceFilters = [myns.test.util.myOne, myns.test.util.myTwo];

[Aure7780]

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 :

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

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 )

[Willpower]

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 :

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

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 :

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

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 :

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

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.
};

[Aure7780]

Ah!

Donc quand je fais :

Code :

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

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

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

[Willpower]

Ah!

Donc quand je fais :

Code :

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

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".

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

Code :

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

	replaceFilters: [this.myOne, this.myTwo]

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

[Aure7780]

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

[Watilin]

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 :

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

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.