Tại sao JSHINT lại phàn nàn rằng đây là một vi phạm nghiêm trọng?

Tôi nghĩ rằng đây có thể là một bản sao của Vi phạm nghiêm trọng bằng cách sử dụng từ khóa này và tiết lộ mẫu mô-đun

Tôi có mã này:

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

Và JSHINT (JSLINT) đang phàn nàn. Nó nói "Vi phạm nghiêm trọng." cho dòng được đánh dấu:

Việc tôi sử dụng Function.call()và sau đó tham chiếu phiên bản, bằng cách nào đó không phù hợp?

Đây có được coi là phong cách xấu không?

JSHint nói rằng "Có thể có vi phạm nghiêm trọng" bởi vì bạn đang sử dụng thisbên trong một thứ gì đó, theo như nó có thể nói, không phải là một phương thức.

Ở chế độ không nghiêm ngặt, việc gọi gotoPage(5)sẽ liên kết thisvới đối tượng chung ( windowtrong trình duyệt). Ở chế độ nghiêm ngặt, thissẽ có undefined, và bạn sẽ gặp rắc rối.

Có lẽ, bạn muốn gọi hàm này với một thisngữ cảnh ràng buộc , ví dụ: gotoPage.bind(myObj)(5)hoặc gotoPage.call(myObj, 5). Nếu vậy, bạn có thể bỏ qua JSHint, vì bạn sẽ không tạo ra bất kỳ lỗi nào. Tuy nhiên, nó cho bạn biết rằng mã của bạn không rõ ràng đối với bất kỳ ai đọc nó, bởi vì việc sử dụng thisbên trong một thứ gì đó không rõ ràng là một phương pháp khá khó hiểu. Sẽ tốt hơn nếu chỉ chuyển đối tượng dưới dạng tham số:

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

Tôi đã nhận được thông báo này cho một hàm không bắt đầu bằng chữ hoa.

"use strict";

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


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

Nếu bạn khai báo hàm dưới dạng một biến thay vì sử dụng khai báo hàm tiêu chuẩn, jshint sẽ không gắn cờ điều này là vi phạm nghiêm trọng. Vì vậy, bạn có thể làm như sau:

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

Nếu bạn đang cố gắng triển khai một phương pháp, bạn có thể muốn gán cho nguyên mẫu thay thế:

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

JSHint sẽ không cảnh báo khi chức năng đang được chỉ định.