javascript - Closure Compiler Warning `uso peligroso del global este objeto`?

jquery debugging

Question

Estimados amigos, Closure Compiler da estas advertencias en Modo Avanzado, subrayando {this.

JSC_USED_GLOBAL_THIS: el uso peligroso del objeto global en la línea 200 del carácter 33 se hovers[i4].onfocus = function() {this.className += "Hovered";}

JSC_USED_GLOBAL_THIS: el uso peligroso de este objeto global en la línea 201 del carácter 32 hovers[i4].onblur = function() {this.className = this.className.replace(/Hove...

JSC_USED_GLOBAL_THIS: el uso peligroso del objeto global en la línea 201 del carácter 49 hovers[i4].onblur = function() {this.className = this.className.replace(/Hove...

JSC_USED_GLOBAL_THIS: uso peligroso de este objeto global en la línea 218 caracteres 38 buttons[i5].onmouseover = function() {this.className += "Hovered";}

Q1. ¿Qué es tan peligroso acerca de esto?
Q2. ¿Debo cambiar esto?
Q3. ¿Cómo mejoro / resuelvo este código?

merci

Answer 1

"esto" puede tener un significado diferente en un contexto diferente, por lo que te dice exactamente eso. Puedes usar cierres en su lugar:

En lugar de

hovers[i4].onfocus = function() {this.className += "Hovered";}

tener:

hovers[i4].onfocus = function(self) 
{
    return function() {self.className += "Hovered";}
}(hovers[i4])

Answer 2

Si conoce el tipo de la variable "this", puede declararla con un JsDoc para evitar que el compilador se queje:

hovers[i4].onfocus = 
/** @this {Element} */
function() {this.className += "Hovered";}

Advertencia: esto, sin embargo, supone que usted sabe con certeza el tipo de la variable "this". Esto puede no ser tan fácil como parece. Por ejemplo:

foo.doSomething = function(x) { this.bar = x; }
foo.doSomething("Hello");

Habrías sabido que "esto" en doSomething refiere a foo . Sin embargo, si usa el modo avanzado del compilador de cierre, el compilador puede "aplanar" el espacio de nombres foo y terminará con:

a = function(x) { this.b = x }
a("Hello");

con foo.doSomething se "aplana" a una única variable global a . ¡En este caso, la variable "esto" obviamente apunta al objeto global en su lugar! ¡Tu código se romperá!

Por lo tanto, el compilador de cierre es bastante inflexible al advertirle que no use "esto" en las funciones que pueden ser aplanadas. Sin embargo, puede usar "esto" en constructores y funciones de prototipo sin esta advertencia.

Para resolver esto, es mejor evitar usar "esto" utilizando el propio espacio de nombres:

foo.doSomething = function(x) { foo.bar = x; }
foo.doSomething("Hello");

Answer 3

Solo para agregar un ejemplo de lo que @marcinkuzminski agregó comentario a @stephen Chung responde

 /**
 * Model for ListBox
 *
 * @constructor <-- add this to remove the warning
 */
MyProject.ListBoxModel = function ( data ){

  this.data_  = data || {};   /* this gives warning */
};

Fuente: https://developers.google.com/closure/compiler/docs/js-for-compiler