เหตุใด JSHINT จึงบ่นว่านี่เป็นการละเมิดอย่างเข้มงวด

ฉันคิดว่านี่อาจซ้ำกับการละเมิดอย่างเข้มงวดโดยใช้คำหลักนี้และรูปแบบโมดูลที่เปิดเผย

ฉันมีรหัสนี้:

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) กำลังบ่น มีข้อความว่า "ละเมิดอย่างเข้มงวด" สำหรับบรรทัดที่ไฮไลต์:

การใช้งานFunction.call()และอ้างถึงอินสแตนซ์ของฉันไม่เหมาะสมหรือไม่

ถือว่าเป็นลักษณะที่ไม่ดีหรือไม่?

JSHint ระบุว่า "อาจมีการละเมิดอย่างเข้มงวด" เนื่องจากคุณกำลังใช้thisสิ่งที่อยู่ภายในสิ่งที่สามารถบอกได้ว่าไม่ใช่วิธีการ

ในโหมดที่ไม่เข้มงวดการโทรgotoPage(5)จะผูกthisกับวัตถุส่วนกลาง ( windowในเบราว์เซอร์) ในโหมดเข้มงวดthisจะเป็นundefinedและคุณจะมีปัญหา

สมมุติคุณหมายถึงจะเรียกฟังก์ชั่นนี้ด้วยผูกพันthisบริบทเช่นหรือgotoPage.bind(myObj)(5) gotoPage.call(myObj, 5)ในกรณีนี้คุณสามารถเพิกเฉยต่อ JSHint ได้เนื่องจากคุณจะไม่สร้างข้อผิดพลาดใด ๆ แต่มันเป็นการบอกคุณว่ารหัสของคุณไม่ชัดเจนสำหรับทุกคนที่อ่านมันเนื่องจากการใช้thisสิ่งที่ไม่ใช่วิธีการอย่างชัดเจนจึงค่อนข้างสับสน มันจะดีกว่าถ้าส่งผ่านวัตถุเป็นพารามิเตอร์:

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

ฉันมีข้อความนี้สำหรับฟังก์ชันที่ไม่ได้ขึ้นต้นด้วยอักษรตัวใหญ่

"use strict";

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


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

หากคุณประกาศฟังก์ชันเป็นตัวแปรแทนที่จะใช้การประกาศฟังก์ชันมาตรฐาน jshint จะไม่ตั้งค่าสถานะว่าเป็นการละเมิดที่เข้มงวด ดังนั้นคุณสามารถทำสิ่งต่อไปนี้ -

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

หากคุณกำลังพยายามใช้วิธีการคุณอาจต้องการกำหนดให้กับต้นแบบแทน:

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

JSHint จะไม่เตือนเมื่อมีการกำหนดฟังก์ชัน