为什么 JSHINT 抱怨这是一种严格的违规行为？

Question

我认为这可能与严格违反使用重复this 关键字和揭示模块模式

我有这样的代码：

function gotoPage(s){
    if(s<=this.d&&s>0){this.g=s; this.page((s-1)*this.p.size);}
}

function pageChange(event, sorter) {
    var dd = event.currentTarget;
    gotoPage.call(sorter, dd[dd.selectedIndex].value);
}

JSHINT (JSLINT) 正在抱怨。上面写着“严格违规”。对于突出显示的行：

我使用的是 Function.call() 和然后引用实例，不知何故不合适？

这被认为是不好的风格吗？

Answer 1

JSHint 说“可能的严格违规”，因为您在某个东西中使用了 this ，据它所知，这不是一个方法。

在非严格模式下，调用 gotoPage(5) 会将 this 绑定到全局对象（浏览器中的 window）。在严格模式下，this 将会是未定义，并且您会遇到麻烦。

据推测，您的意思是使用绑定的 this 上下文调用此函数，例如 gotoPage.bind(myObj)(5) 或 gotoPage.call(myObj, 5)。如果是这样，您可以忽略 JSHint，因为您不会生成任何错误。但是，它告诉您，您的代码对于任何阅读它的人来说都不清楚，因为在显然不是方法的内容中使用 this 是相当令人困惑的。最好简单地将对象作为参数传递：

function gotoPage(sorter, s) {
    if (s <= sorter.d && s > 0) {
        sorter.g = s;

        sorter.page((s - 1) * sorter.p.size);
    }
}

function pageChange(event, sorter) {
    var dd = event.currentTarget;
    gotoPage(sorter, dd[dd.selectedIndex].value);
}

Answer 2

我收到一条关于不以大写字母开头的函数的消息。

"use strict";

// ---> strict violation
function something() {
    this.test = "";
}


// ---> just fine (note the capital S in Something)
function Something() {
    this.test = "";
}

Answer 3

如果您将函数声明为变量而不是使用标准函数声明，jshint 不会将其标记为严格违规。所以你可以执行以下操作 -

var gotoPage = function (s){
    if(s<=this.d&&s>0){this.g=s; this.page((s-1)*this.p.size);}
};


var pageChange = function (event, sorter) {
    var dd = event.currentTarget;
    gotoPage.call(sorter, dd[dd.selectedIndex].value);
};

Answer 4

如果您尝试实现一个方法，您可能需要分配给原型：

ExampleClassName.protytpe.gotoPage = function gotoPage(s){
  // code using this
};

JSHint 在分配函数时不会发出警告。