noUselessElse

Este conteúdo não está disponível em sua língua ainda.

JavaScript (and super languages)

Summary

Rule available since: v1.3.0
Diagnostic Category: lint/style/noUselessElse
This rule has a safe fix.
The default severity of this rule is information.
Sources:
- Inspired from no-else-return
- Inspired from redundant_else

How to configure

1
{
2
  "linter": {
3
    "rules": {
4
      "style": {
5
        "noUselessElse": "error"
6
      }
7
    }
8
  }
9
}

Description

Disallow else block when the if block breaks early.

If an if block breaks early using a breaking statement (return, break, continue, or throw), then the else block becomes useless. Its contents can be placed outside of the block.

If an if block breaks early using a breaking statement (return, break, continue, or throw), then the else block becomes unnecessary. This is because the content of the else block will never be executed in conjunction with the if block, as the breaking statement ensures the control flow exits the if block immediately. Therefore, the else block is redundant, and its content can be placed outside of the block, reducing the indentation level by one.

Examples

Invalid

1
while (x > 0) {
2
    if (f(x)) {
3
        break;
4
    } else {
5
        x++
6
    }
7
}

code-block.js:4:7 lint/style/noUselessElse  FIXABLE  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  ℹ This else clause can be omitted because previous branches break early.
  
    2 │     if (f(x)) {
    3 │         break;
  > 4 │     } else {
      │       ^^^^^^
  > 5 │         x++
  > 6 │     }
      │     ^
    7 │ }
    8 │ 
  
  ℹ Safe fix: Omit the else clause.
  
    2 2 │       if (f(x)) {
    3 3 │           break;
    4   │ - ····}·else·{
      4 │ + ····}
    5 5 │           x++
    6   │ - ····}
    7 6 │   }
    8 7 │

1
function f(x) {
2
    if (x < 0) {
3
        return 0;
4
    } else {
5
        return x;
6
    }
7
}

code-block.js:4:7 lint/style/noUselessElse  FIXABLE  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  ℹ This else clause can be omitted because previous branches break early.
  
    2 │     if (x < 0) {
    3 │         return 0;
  > 4 │     } else {
      │       ^^^^^^
  > 5 │         return x;
  > 6 │     }
      │     ^
    7 │ }
    8 │ 
  
  ℹ Safe fix: Omit the else clause.
  
    2 2 │       if (x < 0) {
    3 3 │           return 0;
    4   │ - ····}·else·{
      4 │ + ····}
    5 5 │           return x;
    6   │ - ····}
    7 6 │   }
    8 7 │

1
function f(x) {
2
    if (x < 0) {
3
        throw new RangeError();
4
    } else {
5
        return x;
6
    }
7
}

code-block.js:4:7 lint/style/noUselessElse  FIXABLE  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  ℹ This else clause can be omitted because previous branches break early.
  
    2 │     if (x < 0) {
    3 │         throw new RangeError();
  > 4 │     } else {
      │       ^^^^^^
  > 5 │         return x;
  > 6 │     }
      │     ^
    7 │ }
    8 │ 
  
  ℹ Safe fix: Omit the else clause.
  
    2 2 │       if (x < 0) {
    3 3 │           throw new RangeError();
    4   │ - ····}·else·{
      4 │ + ····}
    5 5 │           return x;
    6   │ - ····}
    7 6 │   }
    8 7 │

Valid

1
function f(x) {
2
    if (x < 0) {
3
        return 0;
4
    }
5
    return x;
6
}

1
function f(x) {
2
    if (x < 0) {
3
        console.info("negative number");
4
    } else if (x > 0) {
5
        return x;
6
    } else {
7
        console.info("number 0");
8
    }
9
}