Tslint: no-switch-case-fall-through jeté sur les instructions de commutateur imbriquées où tous les cas de commutateur interne ont des instructions de retour

Créé le 2 sept. 2016 · 3Commentaires · Source: palantir/tslint

Rapport d'erreur

Version TSLint : 3.2.0
Version TypeScript : 1.8.10
Exécution de TSLint via : API Node.js
Code TypeScript en train d'être lint

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

avec la configuration tslint.json :

(include if relevant)

Comportement réel

Lance le passage sans commutation à la ligne avant la casse par défaut de la boucle externe.

Comportement prévisible

Pas d'erreur.

P2 Fixed Bug

Source

sumitkmr

👍4

Commentaire le plus utile

Il n'a même pas besoin d'être imbriqué pour lancer.

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

michaeljota le 14 oct. 2016

👍3

Tous les 3 commentaires

Pareil pour

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

Buuhuu le 29 sept. 2016

Il n'a même pas besoin d'être imbriqué pour lancer.

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

michaeljota le 14 oct. 2016

👍3

Idem ici - no-switch-case-fall-through ne reconnaît pas break/return intérieur des blocs de casse case ... { ... } .