2015-05-29 14:07:34 +00:00
|
|
|
//! Checks for if expressions that contain only an if expression.
|
|
|
|
//!
|
|
|
|
//! For example, the lint would catch:
|
|
|
|
//!
|
2016-12-21 09:00:13 +00:00
|
|
|
//! ```rust,ignore
|
2015-05-29 14:07:34 +00:00
|
|
|
//! if x {
|
|
|
|
//! if y {
|
|
|
|
//! println!("Hello world");
|
|
|
|
//! }
|
|
|
|
//! }
|
|
|
|
//! ```
|
|
|
|
//!
|
|
|
|
//! This lint is **warn** by default
|
|
|
|
|
|
|
|
use rustc::lint::*;
|
2016-06-21 21:17:18 +00:00
|
|
|
use syntax::ast;
|
2015-08-16 06:54:43 +00:00
|
|
|
|
2016-06-29 19:26:26 +00:00
|
|
|
use utils::{in_macro, snippet_block, span_lint_and_then};
|
|
|
|
use utils::sugg::Sugg;
|
2015-05-29 14:07:34 +00:00
|
|
|
|
2016-08-06 07:55:04 +00:00
|
|
|
/// **What it does:** Checks for nested `if` statements which can be collapsed
|
|
|
|
/// by `&&`-combining their conditions and for `else { if ... }` expressions that
|
|
|
|
/// can be collapsed to `else if ...`.
|
2015-12-11 00:22:27 +00:00
|
|
|
///
|
2016-08-06 07:55:04 +00:00
|
|
|
/// **Why is this bad?** Each `if`-statement adds one level of nesting, which
|
|
|
|
/// makes code look more complex than it really is.
|
2015-12-11 00:22:27 +00:00
|
|
|
///
|
2016-08-06 07:55:04 +00:00
|
|
|
/// **Known problems:** None.
|
2015-12-11 00:22:27 +00:00
|
|
|
///
|
2016-07-15 22:25:44 +00:00
|
|
|
/// **Example:**
|
2016-12-21 09:00:13 +00:00
|
|
|
/// ```rust,ignore
|
2016-07-15 22:25:44 +00:00
|
|
|
/// if x {
|
|
|
|
/// if y {
|
|
|
|
/// …
|
|
|
|
/// }
|
|
|
|
/// }
|
|
|
|
///
|
|
|
|
/// // or
|
|
|
|
///
|
|
|
|
/// if x {
|
|
|
|
/// …
|
|
|
|
/// } else {
|
|
|
|
/// if y {
|
|
|
|
/// …
|
|
|
|
/// }
|
|
|
|
/// }
|
|
|
|
/// ```
|
|
|
|
///
|
|
|
|
/// Should be written:
|
|
|
|
///
|
2016-12-21 09:00:13 +00:00
|
|
|
/// ```rust.ignore
|
2016-07-15 22:25:44 +00:00
|
|
|
/// if x && y {
|
|
|
|
/// …
|
|
|
|
/// }
|
|
|
|
///
|
|
|
|
/// // or
|
|
|
|
///
|
|
|
|
/// if x {
|
|
|
|
/// …
|
|
|
|
/// } else if y {
|
|
|
|
/// …
|
|
|
|
/// }
|
|
|
|
/// ```
|
2015-05-29 14:07:34 +00:00
|
|
|
declare_lint! {
|
|
|
|
pub COLLAPSIBLE_IF,
|
|
|
|
Warn,
|
2016-07-17 11:29:34 +00:00
|
|
|
"`if`s that can be collapsed (e.g. `if x { if y { ... } }` and `else { if x { ... } }`)"
|
2015-05-29 14:07:34 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
#[derive(Copy,Clone)]
|
|
|
|
pub struct CollapsibleIf;
|
|
|
|
|
|
|
|
impl LintPass for CollapsibleIf {
|
|
|
|
fn get_lints(&self) -> LintArray {
|
|
|
|
lint_array!(COLLAPSIBLE_IF)
|
|
|
|
}
|
2015-09-19 02:53:04 +00:00
|
|
|
}
|
2015-08-11 18:22:20 +00:00
|
|
|
|
2016-06-21 21:17:18 +00:00
|
|
|
impl EarlyLintPass for CollapsibleIf {
|
|
|
|
fn check_expr(&mut self, cx: &EarlyContext, expr: &ast::Expr) {
|
2015-09-17 05:27:18 +00:00
|
|
|
if !in_macro(cx, expr.span) {
|
|
|
|
check_if(cx, expr)
|
|
|
|
}
|
2015-08-11 18:22:20 +00:00
|
|
|
}
|
2015-06-01 13:09:17 +00:00
|
|
|
}
|
|
|
|
|
2016-06-21 21:53:45 +00:00
|
|
|
fn check_if(cx: &EarlyContext, expr: &ast::Expr) {
|
|
|
|
match expr.node {
|
|
|
|
ast::ExprKind::If(ref check, ref then, ref else_) => {
|
|
|
|
if let Some(ref else_) = *else_ {
|
|
|
|
check_collapsible_maybe_if_let(cx, else_);
|
|
|
|
} else {
|
|
|
|
check_collapsible_no_if_let(cx, expr, check, then);
|
|
|
|
}
|
2016-12-20 17:21:30 +00:00
|
|
|
},
|
2016-06-21 21:53:45 +00:00
|
|
|
ast::ExprKind::IfLet(_, _, _, Some(ref else_)) => {
|
|
|
|
check_collapsible_maybe_if_let(cx, else_);
|
2016-12-20 17:21:30 +00:00
|
|
|
},
|
2016-06-21 21:53:45 +00:00
|
|
|
_ => (),
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
fn check_collapsible_maybe_if_let(cx: &EarlyContext, else_: &ast::Expr) {
|
|
|
|
if_let_chain! {[
|
|
|
|
let ast::ExprKind::Block(ref block) = else_.node,
|
2016-07-01 15:41:57 +00:00
|
|
|
let Some(ref else_) = expr_block(block),
|
|
|
|
!in_macro(cx, else_.span),
|
2016-06-21 21:53:45 +00:00
|
|
|
], {
|
|
|
|
match else_.node {
|
|
|
|
ast::ExprKind::If(..) | ast::ExprKind::IfLet(..) => {
|
2016-02-01 10:28:39 +00:00
|
|
|
span_lint_and_then(cx,
|
|
|
|
COLLAPSIBLE_IF,
|
|
|
|
block.span,
|
|
|
|
"this `else { if .. }` block can be collapsed", |db| {
|
2016-02-15 16:43:16 +00:00
|
|
|
db.span_suggestion(block.span, "try", snippet_block(cx, else_.span, "..").into_owned());
|
2016-02-01 10:28:39 +00:00
|
|
|
});
|
2015-08-11 18:22:20 +00:00
|
|
|
}
|
2016-06-21 21:53:45 +00:00
|
|
|
_ => (),
|
2016-01-04 04:26:12 +00:00
|
|
|
}
|
2016-06-21 21:53:45 +00:00
|
|
|
}}
|
|
|
|
}
|
|
|
|
|
2016-12-20 17:21:30 +00:00
|
|
|
fn check_collapsible_no_if_let(cx: &EarlyContext, expr: &ast::Expr, check: &ast::Expr, then: &ast::Block) {
|
2016-06-21 21:53:45 +00:00
|
|
|
if_let_chain! {[
|
2016-07-01 15:41:57 +00:00
|
|
|
let Some(inner) = expr_block(then),
|
2016-06-21 21:53:45 +00:00
|
|
|
let ast::ExprKind::If(ref check_inner, ref content, None) = inner.node,
|
|
|
|
], {
|
|
|
|
if expr.span.expn_id != inner.span.expn_id {
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
span_lint_and_then(cx, COLLAPSIBLE_IF, expr.span, "this if statement can be collapsed", |db| {
|
2016-06-29 19:26:26 +00:00
|
|
|
let lhs = Sugg::ast(cx, check, "..");
|
|
|
|
let rhs = Sugg::ast(cx, check_inner, "..");
|
2016-06-21 21:53:45 +00:00
|
|
|
db.span_suggestion(expr.span,
|
|
|
|
"try",
|
2016-06-29 19:26:26 +00:00
|
|
|
format!("if {} {}",
|
2016-07-01 17:30:38 +00:00
|
|
|
lhs.and(rhs),
|
2016-06-21 21:53:45 +00:00
|
|
|
snippet_block(cx, content.span, "..")));
|
|
|
|
});
|
|
|
|
}}
|
2015-05-29 14:07:34 +00:00
|
|
|
}
|
|
|
|
|
2016-12-21 09:00:13 +00:00
|
|
|
/// If the block contains only one expression, return it.
|
2016-07-01 15:41:57 +00:00
|
|
|
fn expr_block(block: &ast::Block) -> Option<&ast::Expr> {
|
|
|
|
let mut it = block.stmts.iter();
|
|
|
|
|
|
|
|
if let (Some(stmt), None) = (it.next(), it.next()) {
|
|
|
|
match stmt.node {
|
2016-12-20 17:21:30 +00:00
|
|
|
ast::StmtKind::Expr(ref expr) |
|
|
|
|
ast::StmtKind::Semi(ref expr) => Some(expr),
|
2016-07-01 15:41:57 +00:00
|
|
|
_ => None,
|
2016-01-04 04:26:12 +00:00
|
|
|
}
|
2016-01-13 17:32:55 +00:00
|
|
|
} else {
|
|
|
|
None
|
2015-05-29 14:07:34 +00:00
|
|
|
}
|
2015-06-01 13:09:17 +00:00
|
|
|
}
|