Warum beschwert sich JSHINT darüber, dass dies ein strikter Verstoß ist?

Ich denke, dies könnte ein Duplikat von Strict Violation sein, das dieses Schlüsselwort verwendet und das Modulmuster enthüllt

Ich habe diesen Code:

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);
}

Und JSHINT (JSLINT) beschwert sich. Es heißt "Strikte Verletzung". für die hervorgehobene Zeile:

Ist es Function.call()irgendwie unangemessen, die Instanz zu verwenden und dann darauf zu verweisen?

Wird dies als schlechter Stil angesehen?

JSHint sagt "Mögliche strikte Verletzung", weil Sie verwenden this in etwas verwenden, das, soweit es das kann, keine Methode ist.

In nicht-striktem Modus Berufung gotoPage(5)binden würde thisauf das globale Objekt ( windowim Browser). Im strengen Modusthis wäre undefined, und Sie würden in Schwierigkeiten geraten.

Vermutlich wollen Sie diese Funktion mit einem gebundenen thisKontext aufrufen , z . B. gotoPage.bind(myObj)(5)oder gotoPage.call(myObj, 5). In diesem Fall können Sie JSHint ignorieren, da Sie keine Fehler generieren. Es sagt Ihnen jedoch, dass Ihr Code für jeden, der ihn liest, unklar ist, da die Verwendung thisin etwas, das offensichtlich keine Methode ist, ziemlich verwirrend ist. Es wäre besser, das Objekt einfach als Parameter zu übergeben:

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);
}

Ich habe diese Nachricht für eine Funktion erhalten, die nicht mit einem Großbuchstaben begann.

"use strict";

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


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

Wenn Sie die Funktion als Variable deklarieren, anstatt die Standardfunktionsdeklaration zu verwenden, kennzeichnet jshint dies nicht als strikte Verletzung. Sie können also Folgendes tun:

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);
};

Wenn Sie versuchen, eine Methode zu implementieren, möchten Sie möglicherweise stattdessen den Prototyp zuweisen:

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

JSHint warnt nicht, wenn die Funktion zugewiesen wird.