Jshint: डिफ़ॉल्ट पैरामीटर के बाद नियमित पैरामीटर नहीं आने चाहिए

यह संदेश मूल रूप से JSHint "त्रुटि" के रूप में लागू किया गया था (जिसका अर्थ है कि यह हो सकता है
नजरअंदाज नहीं किया जाना चाहिए): gh-1779। हालांकि यह कुछ शुरुआती दिनों में एक सिंटैक्स त्रुटि हो सकती है
मसौदा, इसे उस तरह से अंतिम रूप नहीं दिया गया था, इसलिए gh-2543 में, हमने "डाउनग्रेड" किया
चेतावनी के लिए संदेश। इसका मतलब है कि आप _may_ इसे JSHint 2.9.1 के माध्यम से अनदेखा कर सकते हैं
-W138 ।

उस ने कहा, यह स्पष्ट नहीं है कि क्या यह चेतावनी बिल्कुल उचित है। मैं
व्यक्तिगत रूप से सोचते हैं कि इस तरह से डिज़ाइन किए गए कार्यों का उपयोग करना मुश्किल है, लेकिन
मैं किसी भी संभावित कोड सुरक्षा मुद्दों की पहचान करने में सक्षम नहीं हूं।

@rwaldron @caitp क्या आप में से किसी के पास इस पर कोई विचार है?

बस यह उल्लेख करना चाहता हूं कि इस तरह का हस्ताक्षर रेडक्स कोड नमूने का एक हिस्सा है। लाइन की तलाश करें:

function counter(state = 0, action) {

इसलिए, इसका व्यापक रूप से उपयोग किया जाता है, IMHO।

इस प्रकार का पैटर्न "लिंट" की परिभाषा _बिल्कुल_ है।

इसलिए, इसका व्यापक रूप से उपयोग किया जाता है, IMHO।

मैं इस निहितार्थ से दृढ़ता से असहमत हूं कि एक पैटर्न जो रेडक्स में दिखाई देता है वह कुछ "व्यापक रूप से उपयोग किए जाने" का संकेत है। मैंने रेडक्स के माध्यम से देखा और counter(undefined, action) उदाहरण पाए और मुझे आश्चर्य हुआ कि इसका क्या मतलब हो सकता है, उनमें से प्रत्येक को वास्तव में _requires_ action तर्क पर विचार करना, या रनटाइम का सामना करना पड़ता है त्रुटि। यदि action _हमेशा_ आवश्यक है और state वैकल्पिक है, तो ऐसे कॉल की आवश्यकता क्यों है जो स्पष्ट रूप से undefined पास होना चाहिए - जो डिफ़ॉल्ट पैरामीटर मानों के उद्देश्य को हरा देता है ।

...मैं एक बग फाइल करने के लिए ललचा रहा हूं।

उस ने कहा, यह स्पष्ट नहीं है कि क्या यह चेतावनी बिल्कुल उचित है।

मेरा मानना ​​​​है कि यह है, और जो कोई भी चेतावनी नहीं चाहता है उसका इसे बंद करने के लिए स्वागत है।

बेझिझक इसे बंद करें @jugglinmike

@rwaldron ठीक है, वास्तव में हम redux पर चर्चा नहीं करते हैं। क्या आप त्रुटि का एक नमूना प्रदान कर सकते हैं जो ऐसे हस्ताक्षर के साथ प्रकट हो सकता है?
मेरे लिए यह सिर्फ भाषा की संपत्ति है। तो, इसे "गलत" के रूप में चिह्नित करने का क्या कारण है?

क्या आप त्रुटि का एक नमूना प्रदान कर सकते हैं जो ऐसे हस्ताक्षर के साथ प्रकट हो सकता है?

आपके सामने आने वाली एकमात्र रनटाइम त्रुटि कॉल कर रही है: counter() और counter(undefined) , लेकिन यह मेरी बात नहीं है। मेरा कहना है कि यह भयानक डिजाइन है और प्रोग्रामर और उनके टूलिंग पर अनुचित बोझ डालता है। उदाहरण के लिए, एक मिनीफायर निम्नलिखित का यथोचित विश्लेषण कर सकता है:

function counter(action, state = 0) {
  return [action, state];
}
counter({});
counter({}, 0);
counter({}, undefined);

और उत्पादन करें:

function c(a,s=0){return[a,s]}
c({});
c({});
c({});

जबकि, डिफ़ॉल्ट को पहले रखना:

function counter(state = 0, action) {
  return [state, action];
}
counter(0, {});
counter(undefined, {});

उत्पादन करेगा:

function c(s=0,a){return[s,a]}
c(0, {});
c(undefined, {});

यह एक काफी काल्पनिक उदाहरण है, लेकिन फिर भी मेरी बात को दिखाता है कि यह एक डिफ़ॉल्ट पैरामीटर का उपयोग पूरी तरह से और पूरी तरह से व्यर्थ बनाता है।

मेरे लिए यह सिर्फ भाषा की संपत्ति है।

सिर्फ इसलिए कि आप कर सकते हैं, इसका मतलब यह नहीं है कि आपको चाहिए।

इसलिए, फिलहाल आप यह नहीं समझा सकते हैं कि मिनीफ़ायर में बग के बारे में चिंता करने के अलावा यह एक खराब डिज़ाइन क्यों है

सभी कॉल साइटों को एक स्पष्ट undefined पास करने की आवश्यकता है, एक बुरी तरह से डिज़ाइन किए गए कॉल हस्ताक्षर को प्राप्त करने के लिए एक बुरा अभ्यास और डिफ़ॉल्ट पैरामीटर का गलत उपयोग माना जाता है।

इस व्यवहार को रिएक्ट/रेडक्स द्वारा प्रोत्साहित किया जाता है। रिएक्ट/रेडक्स को हटाने की तुलना में मुझे jshint को हटाने की अधिक संभावना है: /

http://redux.js.org/docs/basics/Reducers.html

इसे और अधिक कॉम्पैक्ट तरीके से लिखने के लिए ES6 डिफ़ॉल्ट तर्क सिंटैक्स का उपयोग करने के लिए एक साफ-सुथरी चाल है:

फ़ंक्शन todoApp (राज्य = प्रारंभिक स्थिति, क्रिया) {
// अभी के लिए, कोई भी क्रिया न करें
// और बस हमें दिया गया राज्य लौटाएं।
वापसी की स्थिति
}

@jugglinmike wdyt ^^?

@txm उस action पैरामीटर का क्या होता है जिसका कभी उपयोग नहीं किया गया?

मेरा अब भी मानना ​​है कि एक ठोस खतरे के अभाव में (और यहां तक ​​कि
अन्यथा-अवांछनीय पैटर्न की उपस्थिति), JSHint चुप रहना चाहिए।
उस ने कहा, मुझे यह समझने में परेशानी हो रही है कि कैसे @txm का उदाहरण अलग है।

मैं इसे कैसे निष्क्रिय कर सकता हूं ..?

@thalesfsp : /* jshint -W138 */ जोड़ें अपनी फ़ाइल की शुरुआत जोड़ें। (सुनिश्चित करें कि आप jshint v2.9.1 या नए का उपयोग कर रहे हैं)

Redux के साथ एक ही समस्या पर ठोकर खाई। क्या इसे .jshintrc कॉन्फ़िग फ़ाइल में अक्षम किया जा सकता है, कृपया?

@derwaldgeist ज़रूर! यहां एक उदाहरण .jshintrc फ़ाइल है जो चेतावनी को शांत कर देगी:

{
  "esversion": 6,
  "-W138": true
}

(वैसे, JSHint's documentation में विशिष्ट चेतावनियों को अक्षम करने के बारे में अधिक जानकारी है।)

@jugglinmike धन्यवाद। मुझे नहीं पता था कि .jshintrc फ़ाइल में भी "-Wxxx" सिंटैक्स का उपयोग करना संभव है। मैंने इसे हमेशा एक फाइल के शीर्ष पर इस्तेमाल किया। जान कर अच्छा लगा!

किसी फ़ंक्शन के अंतिम पैरामीटर के रूप में 'कॉलबैक' का व्यापक रूप से उपयोग किए जाने के साथ, यह लिंट चेतावनी मुझे मूर्खतापूर्ण लगती है

openDialog(url, नाम, args = {}, स्थिति) {
नया वादा वापस करें (फ़ंक्शन (समाधान, अस्वीकार) {
chrome.windows.create({
यूआरएल: यूआरएल,
टाइप करें: "पॉपअप",
चौड़ाई: स्थिति && स्थिति.चौड़ाई || अपरिभाषित,
ऊंचाई: स्थिति && स्थिति ऊंचाई || अपरिभाषित,
बाएँ: pos && pos.left || अपरिभाषित,
शीर्ष: pos && pos.top || अपरिभाषित
}, फ़ंक्शन (डब्ल्यू) {

जाहिरा तौर पर अन्य मापदंडों को undefined पर डिफॉल्ट करना वैध है और लाइनिंग प्रक्रिया को पास करता है। मैं यह नहीं कह रहा हूं कि यह एक बेहतर पैटर्न है लेकिन यह लाइनिंग प्रक्रिया से गुजरता है।