AVERTISSEMENT - utilisation dangereuse du monde cet objet

Question

Dans Google Closure Compiler je reçois l'avertissement

AVERTISSEMENT - utilisation dangereuse du monde cet objet

Voici un exemple. La ligne d'erreur et le décalage fait référence au début du mot this

function aToggle() { 
    if(shown) 
    toggle.show() 
    else 
    toggle.hide() 
    $(this).text(shown ? 'Click to hide' : 'Click to show') 
    shown = !shown 
} 
link.onclick = aToggle 

Je voudrais juste changer une méthode anonyme, mais je suis réutilisant aToggle ailleurs dans le fichier, il doit être nommé. Je pourrais marquer aToggle comme /**@constructor*/ - mais ce n'est pas un constructeur. Y a-t-il une autre annotation que je peux utiliser pour éliminer cet avertissement, ou suis-je coincé entre le marquage en tant que constructeur ou l'affichage d'un tas d'avertissements inutiles?

Answer 1

Modifier: J'ai lu Fermeture: The Definitive Guide, et je viens de réaliser que vous pouvez simplement ajouter le /** @this {Element} */ annotation avant que votre gestionnaire d'événements pour faire se plaindre contre la fermeture du compilateur. Voir la Closure Compiler warning reference Le compilateur de fermeture affiche cet avertissement lorsque vous utilisez this dans une fonction qui n'est pas annotée /** @constructor */ ou dans la classe prototype d'une classe. Le compilateur suppose que vous n'utiliserez jamais this lorsqu'une fonction est appelée dans le contexte d'un autre objet (ce que font les callbacks d'événement).

Certains endroits que vous pourriez avoir à changer pour faire se plaindre arrêter la fermeture du compilateur avec cet avertissement:

• Ne pas utiliser directement link.onclick = ..., parce que vous devez jouer avec this et e || window.event. Au lieu de cela, utilisez jQuery pour envelopper le gestionnaire d'événements, car jQuery's event object has e.currentTarget.
• Si vous utilisez this dans un jQuery.each, remplacez this par le second paramètre de votre fonction. Par exemple, jQuery.each([1, 2, 3], function(i, val) { ... val ... };.

Answer 2

Je ne sais pas JQuery très bien mais je pense que vous pouvez utiliser quelque chose comme:

function aToggle(event) { 
    if(shown) { 
    toggle.show(); 
    } else { 
    toggle.hide(); 
    } 
    $(event.target).text(shown ? 'Click to hide' : 'Click to show'); 
    shown = !shown; 
} 

$(link).bind('click', aToggle); 

où vous récupérez la cible cliquée d'un objet événement générique croix navigateur.

EDIT: pour un conseil, utilisez { } avec votre if else et utilisez des points-virgules, ne comptez pas sur votre navigateur pour le faire pour vous.

Pour tirer le meilleur parti des outils de fermeture, il est conseillé d'utiliser la bibliothèque de fermeture en combinaison avec le compilateur (mais pas nécessaire)

Answer 3

D'abord, vous faites probablement mal. :-)

@Jan avait la bonne idée. Mais vous devriez probablement aller avec ce qui suit:

(function(){ 
    var toggle = $("#toggle"); 
    $("#yourLinksID, .orClassName").click(function(e) { 
     var shown = toggle.toggle().is(":visible"); 
     $(this).html(shown ? "Click to hide" : "Click to show"); 
     e.preventDefault(); 
    }); 
}()); 

et lors de la compilation:

Utilisez le fichier externals jQuery suivant qui indique la fermeture du compilateur ce qui fait quoi dans jQuery: http://code.google.com/p/closure-compiler/source/browse/trunk/contrib/externs/jquery-1.4.3.externs.js

Si vous voulez juste la message d'avertissement pour disparaître remplacer this avec link.