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

Question

Tôi nghĩ rằng đây có thể là một bản sao của Strict Violation using this keyword and revealing module pattern

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) được phàn nàn. Nó nói "Vi phạm nghiêm ngặt". cho dòng nhấn mạnh:

là sử dụng của tôi Function.call() và sau đó tham khảo các ví dụ, bằng cách nào đó không phù hợp?

Đây có phải là kiểu xấu không?

Answer 1

JSHint nói "Có thể vi phạm nghiêm ngặt" vì bạn đang sử dụng this bên trong cái gì đó, theo như nó có thể nói, không phải là một phương pháp.

Ở chế độ không nghiêm ngặt, gọi gotoPage(5) sẽ ràng buộc this với đối tượng chung (window trong trình duyệt). Ở chế độ nghiêm ngặt, this sẽ là 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 ngữ cảnh this bị 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. Nhưng, nó nói với bạn rằng mã của bạn là không rõ ràng cho bất cứ ai đọc nó, bởi vì sử dụng this bên trong của một cái gì đó mà không rõ ràng là một phương pháp khá khó hiểu. Sẽ tốt hơn nếu chỉ truyề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); 
} 

Answer 2

Tôi đã có thông báo này cho một chức năng 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 = ""; 
} 

Answer 3

Nếu bạn khai báo hàm thay vì sử dụng khai báo hàm chuẩn, jshint sẽ không gắn cờ điều này là vi phạm nghiêm ngặt. 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); 
}; 

Answer 4

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

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

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