Закрытие компилятора Предупреждение `опасное использование глобального объекта?

Ответ 1

"this" может иметь разное значение в разных контекстах, поэтому он точно сообщает об этом. Вместо этого вы можете использовать закрытие:

Вместо

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

имеет:

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

Ответ 2

Если вы знаете тип переменной "this", вы можете объявить ее с помощью JsDoc, чтобы остановить компилятор от жалобы:

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

Предостережение: это, однако, предполагает, что вы знаете тип переменной "this". Это может быть не так просто, как кажется. Например:

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

Вы бы знали, что "this" в doSomething относится к foo. Однако, если вы используете расширенный режим компилятора Closure, компилятор может "сгладить" пространство имен foo, и в итоге вы получите:

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

когда foo.doSomething "сплющено" к одной глобальной переменной a. В этом случае переменная "this", очевидно, указывает на глобальный объект! Ваш код сломается!

Следовательно, компилятор Closure совершенно непреклонен, предупреждая вас не использовать "this" в функциях, которые можно сгладить. Вы можете использовать "this" в конструкторах и прототипах без этого предупреждения.

Чтобы решить эту проблему, лучше избегать использования "this" с помощью самого пространства имен:

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

Ответ 3

Просто добавьте пример того, что @marcinkuzminski добавил комментарий к @stephen Chung answer

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

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

Источник: https://developers.google.com/closure/compiler/docs/js-for-compiler