Dlaczego JSHINT narzeka, że ​​jest to surowe naruszenie?

Myślę, że może to być duplikat Strict Violation używający tego słowa kluczowego i ujawniający wzorzec modułu

Mam ten kod:

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) narzeka. Jest tam napisane „Surowe naruszenie”. dla podświetlonej linii:

Czy korzystanie z Function.call()instancji, a następnie odwoływanie się do niej, jest w jakiś sposób nieodpowiednie?

Czy to jest uważane za zły styl?

JSHint mówi „Możliwe ścisłe naruszenie”, ponieważ używasz thiswewnątrz czegoś, co, o ile może stwierdzić, nie jest metodą.

W trybie nieścisłym wywołanie gotoPage(5)wiązałoby się thisz obiektem globalnym ( windoww przeglądarce). W trybie ścisłym thisbyłoby undefinedi wpadłbyś w kłopoty.

Przypuszczalnie masz na myśli wywołanie tej funkcji z powiązanym thiskontekstem, np . gotoPage.bind(myObj)(5)Lub gotoPage.call(myObj, 5). Jeśli tak, możesz zignorować JSHint, ponieważ nie wygenerujesz żadnych błędów. Ale mówi ci, że twój kod jest niejasny dla każdego, kto go czyta, ponieważ użycie thiswewnątrz czegoś, co nie jest oczywiście metodą, jest dość zagmatwane. Lepiej byłoby po prostu przekazać obiekt jako parametr:

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

Otrzymałem tę wiadomość dla funkcji, która nie zaczynała się wielką literą.

"use strict";

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


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

Jeśli zadeklarujesz funkcję jako zmienną zamiast używać standardowej deklaracji funkcji, jshint nie oznaczy tego jako ścisłe naruszenie. Możesz więc wykonać następujące czynności -

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

Jeśli próbujesz zaimplementować metodę, możesz zamiast tego przypisać ją do prototypu:

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

JSHint nie ostrzega, gdy funkcja jest przypisywana.