Discussion :

[winzou]

Salut a tous,

Je me heurte à un petit problème dont je ne vois vraiment pas la raison ni la solution.

J'ai un objet, mettons comme ca :

Code :

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

1
2
3
4
5
6
7
8
var asso_book_datatables = {
 
	'datatable': function() { return 'lala'; }
 
	'delete_entry': function() {
		alert(this.datatable());
	}
};

J'ai ma requete ajax comme ca :

Code :

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

1
2
3
4
5
6
7
var callback = asso_book_datatables.delete_entry;
$.ajax({
	type:     'POST',
	success:  function(data) {
		callback();
	}
});

L'idée est relativement simple : je fais un appel ajax en passant ma première fonction en callback, mon callback étant appelé lors du success de la requete.

Et bien ca ne marche pas ! L'erreur est "Uncaught TypeError: Object [object DOMWindow] has no method 'datatable'". Effectivement, si depuis ma méthode delete_entry (qui est bien exécutée) j'affiche this, eh bien JS me dit que c'est un objet de type... DOM Window ?? Alors que ca devrait être mon objet asso_book_datatables...

Et, si dans mon appel ajax je remplace callback() par asso_book_datatable.delete_entry(), alors ca marche sans problème !

La question c'est donc : pourquoi est-ce que JS ne veut pas utiliser ma variable callback pour exécuter la méthode de mon objet ? D'où vient ce DOM Windows au milieu de _mon_ objet ? Sachant qu'à d'autres endroits de mon code j'ai ce genre d'appel qui marche très bien.

Merci !

[RomainVALERI]

Il faut "fixer" le contexte de this :

Code :

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

1
2
3
4
5
6
7
8
var asso_book_datatables = {
        var that = this;
	'datatable': function() { return 'lala'; }
 
	'delete_entry': function() {
		alert(that.datatable());
	}
};

[winzou]

Salut,

Merci, effectivement il faut utiliser cette astuce. Ne pas oublier de faire un objet-fonction, parce que la en fait j'ai un objet littéral (je le savais pas :p), et la technique du that ne marche pas.

C'est assez moche quand même le JS des fois

Merci

[RomainVALERI]

Merci, effectivement il faut utiliser cette astuce. Ne pas oublier de faire un objet-fonction, parce que la en fait j'ai un objet littéral (je le savais pas :p), et la technique du that ne marche pas.

Ah oui tiens j'avais pas fait attention, tu as raison, j'ai du répondre trop vite. ^^

C'est assez moche quand même le JS des fois

Disons que j'ai rien entendu

[Watilin]

C'est assez moche quand même le JS des fois

C'est de la logique, au-delà de toute notion de moche et de beau

Une seule règle à retenir : this c'est ce qu'il y a avant le point au moment de l'appel. Je m'explique :

Code JS :

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

asso_book_datatables.delete_entry();

Ici, this est asso_book_datatables.

Code JS :

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

callback(); // implicitement : window.callback();

Ici, this est window. Par conséquent, le alert(this.datatable()); se transforme en alert(window.datatable());, ce qui explique ton message d'erreur.

Note que les fonctions ont deux méthodes natives, call et apply, qui permettent de « restaurer le contexte ».

Code JS :

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

1
2
maFonction.call(contexte, arg1, arg2, etc.)
maFonction.apply(contexte, args)

En l'occurence :

Code JS :

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

1
2
3
4
5
6
7
var callback = asso_book_datatables.delete_entry;
$.ajax({
	type:     'POST',
	success:  function(data) {
		callback.call(asso_book_datatables);
	}
});

[Bovino]

Oui, de toute manière, lorsque l'on utilise des objets comme des constructeurs (avec une certaine analogie aux classes habituelles en POO), il faut les utiliser au travers d'instances (avec le mot clé new pour définir chaque instance) et dans ce cas, this référence bien le constructeur.