जेएसएचआईएनटी शिकायत क्यों कर रहा है कि यह सख्त उल्लंघन है?

Question

मुझे लगता है कि यह Strict Violation using this keyword and revealing module pattern

का डुप्लिकेट हो सकता है मेरे पास यह कोड है:

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() का मेरा उपयोग है और फिर इंस्टेंस का संदर्भ देना, किसी भी तरह अनुचित है?

क्या यह खराब शैली माना जाता है?

Answer 1

जेएसहिंट कहता है "संभावित सख्त उल्लंघन" क्योंकि आप this का उपयोग कर रहे हैं, जहां तक ​​यह कह सकता है, एक विधि नहीं है।

गैर-सख्त मोड में, gotoPage(5) पर कॉल करना this को वैश्विक ऑब्जेक्ट (window ब्राउज़र में) से बांध देगा। सख्त मोड में, thisundefined होगा, और आपको परेशानी होगी।

संभवतः, आप इस फ़ंक्शन को 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); 
} 

Answer 2

मेरे पास यह संदेश एक ऐसे फ़ंक्शन के लिए है जो पूंजी पत्र से शुरू नहीं हुआ है।

"use strict"; 

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


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

Answer 3

आप के बजाय मानक समारोह घोषणा उपयोग करने का एक चर के रूप में समारोह की घोषणा करते हैं, तो 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); 
}; 

Answer 4

आप एक विधि को लागू करने की कोशिश कर रहे हैं, तो आप के बजाय प्रोटोटाइप करने के लिए आवंटित करने के लिए चाहते हो सकता है:

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

JSHint चेतावनी नहीं होगा समारोह है जब असाइन किया जा रहा है।