no-dupe-else-if

Disallow duplicate conditions in if-else-if chains

当需要根据某些条件在几个可能的分支中只执行一个分支（或最多一个分支）时，通常使用 if-else-if 链。

if (a) {
    foo();
} else if (b) {
    bar();
} else if (c) {
    baz();
}
1
2
3
4
5
6
7

同一链中的两个相同的测试条件几乎总是代码中的一个错误。除非表达式中存在副作用，否则重复的表达式将评估为与链中早期的相同表达式相同的 true 或 false 值，这意味着其分支永远无法执行。

if (a) {
    foo();
} else if (b) {
    bar();
} else if (b) {
    baz();
}
1
2
3
4
5
6
7

在上面的例子中，永远不可能执行 baz()。显然 baz() 只有在 b 值为 true 时才能执行，但在这种情况下会执行 bar()，因为它出现在链中更早的位置。

规则细节

这条规则不允许在同一个 if-else-if 链中出现重复的条件。

使用此规则的错误示例：

/*eslint no-dupe-else-if: "error"*/

if (isSomething(x)) {
    foo();
} else if (isSomething(x)) {
    bar();
}

if (a) {
    foo();
} else if (b) {
    bar();
} else if (c && d) {
    baz();
} else if (c && d) {
    quux();
} else {
    quuux();
}

if (n === 1) {
    foo();
} else if (n === 2) {
    bar();
} else if (n === 3) {
    baz();
} else if (n === 2) {
    quux();
} else if (n === 5) {
    quuux();
}
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31

使用此规则的正确示例：

/*eslint no-dupe-else-if: "error"*/

if (isSomething(x)) {
    foo();
} else if (isSomethingElse(x)) {
    bar();
}

if (a) {
    foo();
} else if (b) {
    bar();
} else if (c && d) {
    baz();
} else if (c && e) {
    quux();
} else {
    quuux();
}

if (n === 1) {
    foo();
} else if (n === 2) {
    bar();
} else if (n === 3) {
    baz();
} else if (n === 4) {
    quux();
} else if (n === 5) {
    quuux();
}
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31

这条规则还可以检测到一些情况，即条件不一致，但由于 || 和 && 运算符的逻辑，分支永远无法执行。

使用此规则的额外错误示例：

/*eslint no-dupe-else-if: "error"*/

if (a || b) {
    foo();
} else if (a) {
    bar();
}

if (a) {
    foo();
} else if (b) {
    bar();
} else if (a || b) {
    baz();
}

if (a) {
    foo();
} else if (a && b) {
    bar();
}

if (a && b) {
    foo();
} else if (a && b && c) {
    bar();
}

if (a || b) {
    foo();
} else if (b && c) {
    bar();
}

if (a) {
    foo();
} else if (b && c) {
    bar();
} else if (d && (c && e && b || a)) {
    baz();
}
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41

请注意，这条规则不比较链上的条件和语句中的条件，在以下情况下不会发出警告。

if (a) {
    if (a) {
        foo();
    }
}

if (a) {
    foo();
} else {
    if (a) {
        bar();
    }
}
1
2
3
4
5
6
7
8
9
10
11
12
13

何时不用

在极少数情况下，你确实需要在同一链中有相同的测试条件，这必然意味着链中的表达式会引起和依赖副作用，你将不得不关闭这一规则。

Related Rules

• no-duplicate-case
• no-lonely-if

Version

This rule was introduced in ESLint v6.7.0.

Resources

• Rule source
• Tests source