Jshint: केस स्टेटमेंट को सक्षम करें

को निर्मित 18 फ़र॰ 2011 · 10टिप्पणियाँ · स्रोत: jshint/jshint

जिन चीजों के बारे में मैं क्रॉकफोर्ड को कभी नहीं समझा पाया उनमें से एक यह है कि केस स्टेटमेंट में गिरावट की अनुमति दी जाए।

वर्तमान में, यह ठीक है:

स्विच (फू) {
मामला एक:
मामला 2:
कुछ करो();
}

लेकिन यह नहीं है:

स्विच (फू) {
मामला एक:
डूसमथिंगफर्स्ट ();
मामला 2:
कुछ करो();
}

JSLint के एक संस्करण में जिसे मैंने पहले हैक किया था, मैंने एक टिप्पणी /_falls through_/ के लिए एक जांच की थी कि यह इंगित करने के लिए कि आप इसके माध्यम से गिरने का इरादा रखते हैं:

स्विच (फू) {
मामला एक:
डूसमथिंगफर्स्ट ();
/_के माध्यम से गिरता है_/
मामला 2:
कुछ करो();
}

मैं वास्तव में इसे जेएसहिंट में शामिल देखना चाहता हूं, क्योंकि जेएसलिंट में यह मेरे लिए बहुत लंबे समय से दर्द रहा है।

स्रोत

nzakas

सबसे उपयोगी टिप्पणी

.jshintrc ऑब्जेक्ट कुंजी की तलाश करने वालों के लिए:

"-W086": true, //allow fall-through

jpillora 13 अप्रैल 2014

👍2

सभी 10 टिप्पणियाँ

साथ ही, #11 से — JSHint के पास संभवतः default गिरने की अनुमति देने का विकल्प होना चाहिए।

valueof 19 फ़र॰ 2011

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

 स्विच (फू) {
 मामला एक:
 डॉस्मथ ();
 /* के माध्यम से गिरता है */
 मामला 2:
 डॉस्मथ ();
 }

संबंधित प्रतिबद्धता: 4a72da1.

valueof 24 फ़र॰ 2011

👍2

बहुत अच्छे धन्यवाद!

nzakas 24 फ़र॰ 2011

IIRC, क्रॉकफोर्ड ने अपनी पुस्तक में एक विशिष्ट उल्लेख किया है कि वह केस फॉल-थ्रू को नापसंद क्यों करता है। कहानी मनोरंजक भी है और कुछ हद तक कम करने वाली भी।

cowboy 24 फ़र॰ 2011

इस पर टिप्पणी करने के लिए क्षमा करें, लेकिन ब्लॉक के अंत में थ्रो के साथ case s के लिए चेतावनी अभी भी चालू है। आप /* falls through */ एनोटेशन के साथ चेतावनी को अक्षम कर सकते हैं, लेकिन पिछले मामले, एनोटेशन और अगले मामले के बीच व्हाइटस्पेस के अलावा किसी भी चीज़ से एनोटेशन "परेशान" है (इस अर्थ में कि चेतावनी तब ट्रिगर होती है) मामला इसलिए मैं इस पर टिप्पणी भी नहीं कर सकता कि टिप्पणी क्यों है।

dslmeinte 21 जन॰ 2013

/* falls through */ के बजाय टिप्पणी break; एक गैर-दस्तावेजी सुविधा है। कृपया इसे डॉक्स में जोड़ें।

/* falls through */ से पहले एक पंक्ति में अतिरिक्त टिप्पणियां जोड़ी जा सकती हैं।

Daghall 10 अप्रैल 2013

सेकेंडेड। कृपया डॉक्स में /*falls through*/ जोड़ें।

CrabDude 1 मई 2013

इसके अलावा, कृपया इसके लिए supprt जोड़ें:

case 'none':
default:

जबकि तकनीकी रूप से 'none' मामला अनावश्यक है, यह कोड की पठनीयता को जोड़ता है।

CrabDude 1 मई 2013

डॉक्स में इसके बारे में जानकारी जोड़ने के लिए +1।

रुचि के बिंदु के रूप में, मुझे हाल ही में ऐसी स्थिति मिली है जहां "गिर जाता है" मामले समझ में आते हैं। मैंने इसका इस्तेमाल लोकलस्टोरेज से पढ़े गए वर्जन वाले डेटा के माइग्रेशन के लिए किया था। भूतपूर्व:

function migrate( version, data ) {
    switch( version ) {
        case 1 :
            data.new1 = data.old;  // convert data from version 1 to version 2
            delete data.old;
        case 2 :
            data.new2 = data.new1;  // convert data from version 2 to version 3
            delete data.new1;
    }
    return data;  // return data in version 3 format
}

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

gregjacobs 13 अग॰ 2013

.jshintrc ऑब्जेक्ट कुंजी की तलाश करने वालों के लिए:

"-W086": true, //allow fall-through

jpillora 13 अप्रैल 2014

👍2

क्या यह पृष्ठ उपयोगी था?

0 / 5 - 0 रेटिंग्स

Jshint: केस स्टेटमेंट को सक्षम करें

सबसे उपयोगी टिप्पणी

सभी 10 टिप्पणियाँ

संबंधित मुद्दों