Tslint: नो-स्विच-केस-फॉल-थ्रू नेस्टेड स्विच स्टेटमेंट पर फेंका गया जहां आंतरिक स्विच के सभी मामलों में रिटर्न स्टेटमेंट होता है

को निर्मित 2 सित॰ 2016 · 3टिप्पणियाँ · स्रोत: palantir/tslint

बग रिपोर्ट

TSLint संस्करण : 3.2.0
टाइपस्क्रिप्ट संस्करण : 1.8.10
TSLint के माध्यम से चल रहा है : Node.js API
टाइपस्क्रिप्ट कोड पंक्तिबद्ध किया जा रहा है

function foo (a: number, b: number): number {
    switch (a) {
        case 1:
            switch (b) {
                case 2:
                    return 1;
                default:
                    return 2;
            }
        default:
            return 3;
    }
}

tslint.json कॉन्फ़िगरेशन के साथ:

(include if relevant)

वास्तविक व्यवहार

बाहरी लूप के डिफ़ॉल्ट केस से पहले नो-स्विच-केस-फॉल-थ्रू लाइन पर फेंकता है।

अपेक्षित् व्यवहार

कोई त्रुटि नहीं।

P2 Fixed Bug

स्रोत

sumitkmr

👍4

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

इसे फेंकने के लिए नेस्ट करने की भी जरूरत नहीं है।

function(a: number, b:number): void {
    switch (a) {
        case 1: {
            return;
        } // Throws
        case 2:
            return;
        default:
            return;
    }
}

michaeljota 14 अक्तू॰ 2016

👍3

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

उसी के लिए

function(a: number, b:number): void {
    switch (a) {
        case 1:
            if (b > 10) {
                return;
            } else {
                return;
            }
        case 2:
            return;
        default:
            return;
    }
}

Buuhuu 29 सित॰ 2016

इसे फेंकने के लिए नेस्ट करने की भी जरूरत नहीं है।

function(a: number, b:number): void {
    switch (a) {
        case 1: {
            return;
        } // Throws
        case 2:
            return;
        default:
            return;
    }
}

michaeljota 14 अक्तू॰ 2016

👍3

यहां वही - no-switch-case-fall-through break/return अंदर केस ब्लॉक case ... { ... } को नहीं पहचानता है।