Discussion :

[nicko_73]

Bonjour,

Je commence par vous donner mon code :

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
21
22
 
manager = {	
	getFunction : function(name, arguements, class) {
		if(typeof(class) != 'undefined') {
			if(typeof(arguements) == 'object') {
				eval(class + '.' + name + '(\'' + arguements + '\')');
				return;
			}
 
			eval(class + '.' + name+ '()');
			return;
		}
 
		if(typeof(class) != 'undefined') {
			eval(name + '(' + arguements + ')');
			return;
		}
 
		eval(name + '()');
		return;
	},
}

La classe précédente avec le méthode "getFunction" permet de créer des fonctions "dynamiquement". D'ailleurs elle reste à améliorer car j'utilise abusivement eval(). Si vous avez une idée ? .

Code :

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

1
2
3
4
5
6
 
actions = {
	closeButton : function(el) {
		alert($(el).html());
	},
}

La classe précédente, quant à elle, regroupe plusieurs fonctions donc dans cette exemple "closeButton".

Code :

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

1
2
3
4
5
6
7
8
9
 
		$('*').live('click', function() {
			if($(this).attr('class') != '') {
				var el = $(this);
				manager.getFunction('closeButton', el, 'actions');
			}
 
			return false;
		});

Avec le code précédent, que j'ai simplifié, je créer une nouvelle fonction. J'arrive à obtenir la fonction avec en argument la variable "el" qui de coule de la ligne précédente.

Dans la fonction appelée (closeButton), je fait un "alert($(el).html());" mais j'ai l'erreur suivante :

"uncaught exception: Syntax error, unrecognized expression: [object Object]"

Pouvez-vous m'aider un peu car je ne vois pas pourquoi ça plante. De plus je ne suis pas encore assez bon en JS et accessoirement jQuery.

Merci,
Bien cordialement.

[Bovino]

Salut,

Ne pas utiliser les mots réservés (genre class) ou des mots appartenant à la syntaxe de JavaScript (genre name) pour nommer tes variables, paramètres etc. pourra probablement t'aider !

[nicko_73]

Merci des conseils que j'ai d'ailleurs appliqué tout de suite, voici donc la nouvelle fonction getFunction (qui est un peu différente depuis le deuxième "if(typeof(arg) != 'object') " ):

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
21
22
 
manager = {	
	getFunction : function(actName, arg, clName) {
		if(typeof(clName) != 'undefined') {
			if(typeof(arg) == 'object') {
				eval(clName + '.' + actName + '(\'' + arg + '\')');
				return;
			}
 
			eval(clName + '.' + actName + '()');
			return;
		}
 
		if(typeof(arg) != 'object') {
			eval(actName + '(\'' + arg + '\')');
			return;
		}
 
		eval(actName + '()');
		return;
	},
}

Mais cela n'a pas encore résolu mon problème. Peu être est-ce à cause de eval() ?

[nicko_73]

Voila la suite de mes teste et je pense que ca vient de la fonction eval(). Voici une variante de la fonction getFunction :

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
 
	getFunction : function(actName, arg, clName) {
		switch(clName) {
			case 'actions':
				switch(actName) {
					case 'closeButton':	
						actions.closeButton(arg);
						break;
					case 'wdw':	
						actions.wdw();
						break;
				}
				break;
		}
	}

Dans ce cas, il n'y aucune erreur. Par contre je souhaite éviter de faire appel à la fonction switch en cascade. De plus je risque d'avoir une quantité importante de fonctions dans dans des classes différentes.

Pouvez-vous m'aider sur ce point ?

Merci

[danielhagnoul]

Bonjour.

Si :

Code :

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

var el = $(this);

Alors :

Code :

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

alert(el.html());

Détail :

Code :

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

if($(this).hasClass('class')) {

[nicko_73]

Bonjour danielhagnoul,

Je n'ai pas très bien compris ce que tu m'a donné comme solution, mais je pense avoir trouvé comme optimiser la fonction geFunction() sans eval
voila un exemple :

Code :

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

1
2
3
4
5
6
7
8
 
window["test"]["bibi"]()
 
test = {
	bibi : function() {
		alert('')	
	}
}

Il n'y a plus qu'a mettre des variable a la place de test et bibi. Encore merci pour vos aides.