1

我正在定义一个简单的对象“浏览器”，它允许显示列表中的“上一个”和“下一个”图像。

function Browser(image, elements) {
    this.current = 0;
    this.image = image;
    this.elements = elements;
}
Browser.prototype.next = function () {
    if (++this.current >= this.elements.length) {
         this.current = 0;
    }
    return this.show(this.current);
};
Browser.prototype.prev = function () {
    if (--this.current < 0) {
        this.current = this.elements.length - 1;
    }
    return this.show(this.current);
};
Browser.prototype.show = function (current) {
    this.image.src = this.elements[current];
    return false;
};

JSlint 几乎喜欢这段代码。但是“高级优化”中的 Google Closure Compiler 并没有对其进行编译。

它说：

JSC_USED_GLOBAL_THIS: dangerous use of the global this object at line 3 character 0
this.current = 0;
JSC_USED_GLOBAL_THIS: dangerous use of the global this object at line 4 character 0
this.image = image;
JSC_USED_GLOBAL_THIS: dangerous use of the global this object at line 5 character 0
this.elements = elements;

这告诉我我不懂javascript oop。

我究竟做错了什么？

4

2 回答 2

6

JSC_USED_GLOBAL_THIS：危险使用全局this对象。

此警告意味着您在原型函数或构造函数之外使用了关键字 this。例如，以下代码将产生此警告：

// Produces JSC_USED_GLOBAL_THIS warning:
this.foo = 1;

在这种情况下，这实际上是指全局 this 对象。对于标准网页，全局对象与窗口对象是一样的。如果您收到此警告，请确保您确实打算引用全局对象。

请注意，编译器仅将带有 @constructor JSDoc 注释的函数识别为构造函数，如下所示：

/**
 * @constructor
 */
function MyFunction() {
  this.foo = 1;
}

https://developers.google.com/closure/compiler/docs/error-ref

于 2013-02-08T17:33:02.430 回答
4

从文档：

请注意，编译器仅将带有 @constructor JSDoc 注释的函数识别为构造函数，如下所示：

/**
 * @constructor
 */
function MyFunction() {
  this.foo = 1;
}
于 2013-02-08T17:33:10.463 回答