Estimados amigos, Closure Compiler da estas advertencias en modo avanzado, subrayando {this.
JSC_USED_GLOBAL_THIS: uso peligroso del objeto global this en la línea 200 carácter 33 hovers[i4].onfocus = function() {this.className += "Hovered";}
JSC_USED_GLOBAL_THIS: uso peligroso del objeto global this en la línea 201 carácter 32 hovers[i4].onblur = function() {this.className = this.className.replace(/Hove...
JSC_USED_GLOBAL_THIS: uso peligroso del objeto global this en la línea 201 carácter 49 hovers[i4].onblur = function() {this.className = this.className.replace(/Hove...
JSC_USED_GLOBAL_THIS: uso peligroso del objeto global this en la línea 218 carácter 38 buttons[i5].onmouseover = function() {this.className += "Hovered";}
Q1. ¿Qué tiene de peligroso esto?
Q2. ¿Debería cambiar esto?
Q3. ¿Cómo mejoro/resuelvo este código?
¡misericordia!
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 sabe con seguridad el tipo de la variable "esto". 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 se 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 siendo "aplanado" 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, Closure Compiler es bastante firme al advertirle que no use "esto" en funciones que se pueden aplanar. Sin embargo, puede usar "esto" en constructores y funciones prototipo sin esta advertencia.
Para resolver esto, es mejor evitar usar "esto" usando el propio espacio de nombres:
foo.doSomething = function(x) { foo.bar = x; } foo.doSomething("Hello");"esto" puede tener un significado diferente en un contexto diferente, por lo que te dice exactamente eso. Puedes usar cierres en su lugar:
En vez de
hovers[i4].onfocus = function() {this.className += "Hovered";}tener:
hovers[i4].onfocus = function(self) { return function() {self.className += "Hovered";} }(hovers[i4])Solo para agregar un ejemplo de lo que @marcinkuzminski agregó un comentario a la respuesta de @stephen Chung
/** * 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