JSHINT neden bunun katı bir ihlal olduğundan şikayet ediyor?

Bunun, bu anahtar kelimeyi kullanan ve modül modelini açığa çıkaran Katı İhlalin bir kopyası olabileceğini düşünüyorum

Bu koda sahibim:

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

Ve JSHINT (JSLINT) şikayet ediyor. "Kesin ihlal" yazıyor. vurgulanan çizgi için:

Örneği kullanmam Function.call()ve sonra buna atıfta bulunmam bir şekilde uygunsuz mu?

Bu kötü bir tarz olarak kabul edilir mi?

JSHint, "Olası katı ihlal" diyor çünkü thisanlayabildiği kadarıyla bir yöntem olmayan bir şeyin içinde kullanıyorsunuz .

Katı olmayan modda, çağrı genel nesneye ( tarayıcıda) gotoPage(5)bağlanır . Katı modda, olur ve başınız belaya girer.thiswindowthisundefined

Muhtemelen, bu işlevi bağlı bir thisbağlamla çağırmak istiyorsunuz , örneğin gotoPage.bind(myObj)(5)veya gotoPage.call(myObj, 5). Öyleyse, herhangi bir hata oluşturmayacağınız için JSHint'i yok sayabilirsiniz. Ancak, size kodunuzun onu okuyan biri için net olmadığını söylüyor, çünkü thisaçıkça bir yöntem olmayan bir şeyin içini kullanmak oldukça kafa karıştırıcı. Nesneyi bir parametre olarak basitçe iletmek daha iyi olacaktır:

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

Büyük harfle başlamayan bir işlev için bu mesajı aldım.

"use strict";

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


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

Standart işlev bildirimini kullanmak yerine işlevi bir değişken olarak bildirirseniz, jshint bunu katı bir ihlal olarak işaretlemeyecektir. Yani aşağıdakileri yapabilirsiniz -

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

Bir yöntem uygulamaya çalışıyorsanız, bunun yerine prototipe atamak isteyebilirsiniz:

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

JSHint, işlev atanırken uyarmaz.