Comment évaluer correctement les variables dans jQuery?

Question

Je travaille sur un plugin jQuery, mais j'ai du mal à faire correctement mes variables. Voici un exemple de mon code:

(function($) { 

$.fn.ksana = function(userOptions) { 
    var o = $.extend({}, $.fn.ksana.defaultOptions, userOptions); 

    return this.each(function() { 
     alert(rotate()); // o is not defined 
    }); 
}; 

function rotate() { 
    return Math.round(o.negRot + (Math.random() * (o.posRot - o.negRot))); 
}; 

$.fn.ksana.defaultOptions = { 
    negRot: -20, 
    posRot: 20 
}; 

})(jQuery); 

J'essaie d'obtenir la rotation de la fonction privée pour être en mesure de voir la variable o, mais il ne cesse 'o est alertes à pas défini. Je ne suis pas sûr de ce que je fais mal.

Answer 1

La variable o est SCOPED localement dans la fonction $.fn.ksana, afin de permettre l'rotate de l'atteindre, vous devez soit:

• Il suffit de passer la variable à o comme un argument.
• Définissez cette fonction au sein de ksana.
• Définissez o dans l'étendue externe.

OMI, en passant comme argument est assez propre:

(function($) { 
    $.fn.ksana = function(userOptions) { 
    var o = $.extend({}, $.fn.ksana.defaultOptions, userOptions); 

    return this.each(function() { 
     alert(rotate(o)); // pass o 
    }); 
    }; 

    function rotate(o) { // use passed object 
    return Math.round(o.negRot + (Math.random() * (o.posRot - o.negRot))); 
    } 
//... 
})(jQuery); 

Answer 2

Vous devez mettre o dans le champ qui l'entoure pour les rotate et ksana - dire dans votre champ racine function($). Comme ceci:

(function($) { 

var o; 

$.fn.ksana = function(userOptions) { 
    o = $.extend({}, $.fn.ksana.defaultOptions, userOptions); 

Mais pourquoi ne pas faire juste un argument de rotate? Pourquoi avez-vous besoin de le rendre "global"?

Answer 3

Vous pouvez mettre la rotate-fonction dans la même portée que o:

(function($) { 

$.fn.ksana = function(userOptions) { 
    var o = $.extend({}, $.fn.ksana.defaultOptions, userOptions); 

    function rotate() { 
     return Math.round(o.negRot + (Math.random() * (o.posRot - o.negRot))); 
    }; 

    return this.each(function() { 
     alert(rotate()); 
    }); 
}; 

Ou, il suffit de passer à tourner:

(function($) { 

    var o; 
    $.fn.ksana = function(userOptions) { 
     o = $.extend({}, $.fn.ksana.defaultOptions, userOptions); 

     return this.each(function() { 
      alert(rotate(o)); 
     }); 
    }; 

function rotate(o) { 
    return Math.round(o.negRot + (Math.random() * (o.posRot - o.negRot))); 
};