rust-lang/rust-clippy
Voir sur GitHubif_same_then_else Should not warn for `else if` blocks.
Open
#3 770 ouverte le 16 févr. 2019
I-false-positiveS-needs-discussiongood first issue
Métriques du dépôt
- Stars
- (10 406 stars)
- Métriques de merge PR
- (Merge moyen 19j 22h) (113 PRs mergées en 30 j)
Description
if condition1() {
1
} else if condition2() {
1
} else if condition3() {
2
} else {
3
}
There are many cases where code like this shouldn't be warned on. While you could do condition1() || condition2() often times it is more clear of the intent if they are separate. Especially if the conditions are non-trivial compared to the value being returned.
I think the right time for this warning to warn is if the last if has the same condition as the else. For example the following two cases would warn.
if condition1() {
1
} else {
1
}
if condition1() {
1
} else if condition2() {
2
} else {
2
}