Clippy lints / Restriction
Clippy lint · Restriction · checked with Clippy 0.1.99
clippy::else_if_without_else: `if` expression with an `else if`, but without a final `else`
What it meansChecks for usage of if expressions with an else if branch, but without a final else branch. It is in the restriction group and off by default.
What you see
warning: `if` expression with an `else if`, but without a final `else` --> main.rs:5:8 | 5 | } else if x.is_negative() { | ________^ 6 | | b(); 7 | | } | |_^ | = help: add an `else` block here = help: for further information visit https://rust-lang.github.io/rust-clippy/stable/index.html#else_if_without_else = note: requested on the command line with `-W clippy::else-if-without-else`
Real output of Clippy 0.1.99 (Rust 1.99) on the example below. Names and line numbers in yours will differ.
Does it matter?
Some coding guidelines require this (e.g., MISRA-C:2004 Rule 14.10).
Lints that forbid language features for specific codebases (no unwrap, no indexing...). Off by default; pick them one by one, never the whole group.
How to fix it
Clippy does not fix this one automatically. Change the code by hand:
Flagged
if x.is_positive() {
a();
} else if x.is_negative() {
b();
}
Write this instead
if x.is_positive() {
a();
} else if x.is_negative() {
b();
} else {
// We don't care about zero.
}
Turn it off, or on
It is off by default. To use it, set it to warn or deny:
Whole crate (Cargo.toml)
[lints.clippy]
else_if_without_else = "warn"In code
#![warn(clippy::else_if_without_else)]One run
cargo clippy -- -W clippy::else-if-without-elseThe [lints] table needs Rust 1.74 or newer. In a workspace, put it under [workspace.lints.clippy] and add [lints] workspace = true to each member.
Clippy's own documentation
What it does
Checks for usage of if expressions with an else if branch,
but without a final else branch.
Why restrict this?
Some coding guidelines require this (e.g., MISRA-C:2004 Rule 14.10).
Example
if x.is_positive() {
a();
} else if x.is_negative() {
b();
}
Use instead:
if x.is_positive() {
a();
} else if x.is_negative() {
b();
} else {
// We don't care about zero.
}
From the Clippy lint list (0.1.99), part of the Rust project, MIT or Apache-2.0. Official entry.
Related lints
- unwrap_usedChecks for .unwrap() or .unwrap_err() calls on Results and .unwrap() call on Options
- module_name_repetitionsDetects public item names that are prefixed or suffixed by the containing public module’s name
- expect_usedChecks for .expect() or .expect_err() calls on Results and .expect() call on Options
- disallowed_script_identsidentifier `счётчик` has a Unicode script that is not allowed by configuration: Cyrillic
- doc_include_without_cfgChecks if included files in doc comments are included only for cfg(doc)
- doc_paragraphs_missing_punctuationChecks for doc comments whose paragraphs do not end with a period or another punctuation mark. Various Markdowns constructs are taken into account to avoid false positives
- empty_dropempty drop implementation
- empty_enum_variants_with_bracketsenum variant has empty brackets
- empty_structs_with_bracketsfound empty brackets on struct declaration
All 847 Clippy lints · Rust compiler errors · Cargo.toml checker
Up next
Related lintclippy::unwrap_usedChecks for .unwrap() or .unwrap_err() calls on Results and .unwrap() call on OptionsRead it next →Did this answer your question?