Salut les gens,

Je m'adresse principalement à Lisarael parce que je connais (et loue !) ses compétences en JS, mais je reste ouvert aux autres avis.

J'ai quelques plugins jQuery que je viens "d'upgrader" un peu en profitant du boilerplate jQuery (https://github.com/jquery-boilerplate/jquery-boilerplate/blob/master/src/jquery.boilerplate.js), cependant il me semble qu'il y a quelques points à améliorer... Et je n'arrive pas à trouver comment.

Pour exemple, celui-ci :
https://github.com/EmmanuelBeziat/jquery-modern-form/blob/master/src/js/jquery-modern-form.js

Premier point : On m'a fait remarquer que $.data permettait d'appeler librement les méthodes de mon plugin, alors que je n'ai effectivement pas prévu de méthodes plubliques. Mais je ne sais pas comment faire en sorte de vérifier l'instanciation "unique" du plugin sur l'élément sans pour autant renvoyer un $.data...

Second point : Je redéfinis $(event.target) plusieurs fois dans le plugin, à travers plusieurs fonctions... Comment faire pour mettre en cache l'objet jQuery de façon propre ?

Y aurait-il d'autres points à voir ?

Merci !

Je n'ai pas compris, tu veux faire un singleton ?

Pas tout à fait ; en gros, le $.data du return this.each(); est là pour empêcher que le plugin ne soit défini deux fois sur le même élément (donc singleton), mais ça laisse l'utilisateur libre d'appeler le plugin sur des éléments distincts (et donc d'en avoir 5 par page si ça lui chante). Mais cette utilisation du $.data semble donner l'accès aux méthodes du plugin, que je ne veux pas publiques, et je cherche comment changer ça.

Hey,

J'ai pas des masses de temps et suis pas sûr d'avoir tout saisi, mais si tu veux juste singletoniser ton truc, pourquoi passer ton plugin entier à $.data ?

    /**
	 * jQuery plugin wrapper
	 */
	$.fn[ pluginName ] = function( options ) {

		return this.each( function() {
			var _oPlugin;
			if ( $.data( this, 'plugin_' + pluginName ) !== true ) {
				_oPlugin = new Plugin( this, options );
				$.data( this, 'plugin_' + pluginName, true );
			}
		} );

	};

Bon, c'est à la grosse louche et je suis concentré sur un truc qui n'a rien du tout à voir, donc si j'ai tapé à côté de la plaque, ce serait même pas étonnant... mais tel que c'est là dans ton code, en laissant une référence à ton plugin directement dans $.data, oui, tes méthodes sont publiquement accessibles.

Alors soit tu passes par ma méthode, soit tu fais en sorte que tes méthodes soient privées en ne les liant pas au prototype, mais ça implique une réécriture en profondeur du plugin...

Ceci étant, ma méthode a pitêt besoin de t'obliger à garder une référence à l'instance de Plugin quelque part... mais bon... un p'tit tableau privé et c'est réglé (mais à première vue, je doute que tu en ais besoin).