If a switch statement has duplicate test expressions in its case clauses, the clause may be repeated or mistyped. It ensures that the offending case clause is correct.

We support Autofix for this issue. During Autofix, we remove the duplicate switch case (i.e., the second case) as all javascript engines ignore the duplicate case (i.e., the second case).

Bad Practice

var a = 1,
    one = 1;

switch (a) {
    case 1:
        break;
    case 2:
        break;
    case 1:         // duplicate test expression
        break;
    default:
        break;
}

switch (a) {
    case one:
        break;
    case 2:
        break;
    case one:         // duplicate test expression
        break;
    default:
        break;
}

switch (a) {
    case "1":
        break;
    case "2":
        break;
    case "1":         // duplicate test expression
        break;
    default:
        break;
}

Recommended

const a = 1,
    one = 1;

switch (a) {
    case 1:
        break;
    case 2:
        break;
    case 3:
        break;
    default:
        break;
}

switch (a) {
    case one:
        break;
    case 2:
        break;
    case 3:
        break;
    default:
        break;
}

switch (a) {
    case "1":
        break;
    case "2":
        break;
    case "3":
        break;
    default:
        break;
}

Sindarius / npm_gcodeviewer

No ignore rules found