noUselessReturn

Ce contenu n’est pas encore disponible dans votre langue.

JavaScript (and super languages)

Summary

Rule available since: v2.3.15
Diagnostic Category: lint/nursery/noUselessReturn
This rule has a safe fix.
The default severity of this rule is information.
Sources:
- Inspired from no-useless-return

How to configure

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

Description

Disallow redundant return statements.

A return; statement with nothing after it is redundant when it is the last reachable statement in a function body. Removing it does not change the function’s behavior, as execution naturally falls through to the end.

Examples

Invalid

1
function foo() {
2
    return;
3
}

code-block.js:2:5 lint/nursery/noUselessReturn  FIXABLE  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  ℹ This return statement is unnecessary.
  
    1 │ function foo() {
  > 2 │     return;
      │     ^^^^^^^
    3 │ }
    4 │ 
  
  ℹ Removing this statement does not change the control flow of the function.
  
  ℹ This rule belongs to the nursery group, which means it is not yet stable and may change in the future. Visit https://biomejs.dev/linter/#nursery for more information.
  
  ℹ Safe fix: Remove the unnecessary return statement.
  
    1 1 │   function foo() {
    2   │ - ····return;
    3 2 │   }
    4 3 │

1
function foo() {
2
    doSomething();
3
    return;
4
}

code-block.js:3:5 lint/nursery/noUselessReturn  FIXABLE  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  ℹ This return statement is unnecessary.
  
    1 │ function foo() {
    2 │     doSomething();
  > 3 │     return;
      │     ^^^^^^^
    4 │ }
    5 │ 
  
  ℹ Removing this statement does not change the control flow of the function.
  
  ℹ This rule belongs to the nursery group, which means it is not yet stable and may change in the future. Visit https://biomejs.dev/linter/#nursery for more information.
  
  ℹ Safe fix: Remove the unnecessary return statement.
  
    1 1 │   function foo() {
    2 2 │       doSomething();
    3   │ - ····return;
    4 3 │   }
    5 4 │

1
function foo() {
2
    if (condition) {
3
        bar();
4
        return;
5
    }
6
}

code-block.js:4:9 lint/nursery/noUselessReturn  FIXABLE  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  ℹ This return statement is unnecessary.
  
    2 │     if (condition) {
    3 │         bar();
  > 4 │         return;
      │         ^^^^^^^
    5 │     }
    6 │ }
  
  ℹ Removing this statement does not change the control flow of the function.
  
  ℹ This rule belongs to the nursery group, which means it is not yet stable and may change in the future. Visit https://biomejs.dev/linter/#nursery for more information.
  
  ℹ Safe fix: Remove the unnecessary return statement.
  
    2 2 │       if (condition) {
    3 3 │           bar();
    4   │ - ········return;
    5 4 │       }
    6 5 │   }

Valid

1
function foo() {
2
    return 5;
3
}

1
function foo() {
2
    if (condition) {
3
        return;
4
    }
5
    bar();
6
}

1
function foo() {
2
    for (const x of xs) {
3
        return;
4
    }
5
}