ADVERTENCIA - El uso peligroso de lo global este objeto

Question

En Google Closure Compiler tengo la advertencia

ADVERTENCIA - El uso peligroso de lo global este objeto

Este es un ejemplo. La línea de error y el desplazamiento se refiere al principio de la palabra this

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

me acaba de cambiar a un método anónimo, pero estoy reutilizando aToggle en el archivo en otro lugar, por lo que debe ser identificado.

Pude marcar aToggle como - pero no es un constructor. ¿Hay alguna otra anotación que pueda utilizar para eliminar esta advertencia, o estoy atrapado entre marcarlo como constructor o tener un montón de advertencias inútiles?

Answer 1

Editar: He estado leyendo Cierre: La guía definitiva, y sólo se dio cuenta de que sólo tiene que añadir el /** @this {Element} */ annotation antes de que su controlador de eventos para hacer Cierre Compilador detener las protestas.

Ver el Closure Compiler warning reference. El compilador de cierre proporciona esta advertencia cuando usa this dentro de una función que no tiene la anotación /** @constructor */ o está dentro del prototype de una clase. El compilador asume que nunca usará this cuando se invoca una función en el contexto de otro objeto (que es lo que hacen las devoluciones de llamada de evento).

Algunos de los lugares que puede que tenga que cambiar para que Closure Compiler dejar de quejarse con esta advertencia:

• no utilice link.onclick = ... directamente, porque hay que meterse con this y e || window.event. En su lugar, use jQuery para ajustar el controlador de eventos, ya que jQuery's event object has e.currentTarget.
• Si está usando this dentro de un jQuery.each, reemplace this con el segundo parámetro de su función. Por ejemplo, jQuery.each([1, 2, 3], function(i, val) { ... val ... };.

Answer 2

No sé jQuery muy bien, pero creo que se puede usar algo como:

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); 

donde se recupera el blanco se ha hecho clic de un objeto de evento genérico cruz navegador.

EDITAR: como consejo, use { } con su if else y use punto y coma, no confíe en que su navegador lo haga por usted.

para hacer el mejor uso de las herramientas de cierre se aconseja utilizar la biblioteca de cierre en combinación con el compilador (aunque no es necesario)

Answer 3

En primer lugar, es probable que esté haciendo mal. :-)

@Jan tuvo la idea correcta. Pero probablemente debería ir con lo siguiente:

(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(); 
    }); 
}()); 

y al compilar:

Usar el siguiente fichero externo jQuery que le dice Cierre compilador lo hace qué en jQuery: http://code.google.com/p/closure-compiler/source/browse/trunk/contrib/externs/jquery-1.4.3.externs.js

Si lo que desea es la mensaje de advertencia para irse reemplazar this con link.