no-useless-return

Disallow redundant return statements

A return; statement with nothing after it is redundant, and has no effect on the runtime behavior of a function. This can be confusing, so it’s better to disallow these redundant statements.

Rule Details

This rule aims to report redundant return statements.

Examples of incorrect code for this rule:

/* eslint no-useless-return: "error" */ const foo = function() { return; } const bar = function() { doSomething(); return; } const baz = function() { if (condition) { qux(); return; } else { quux(); } } const item = function() { switch (bar) { case 1: doSomething(); default: doSomethingElse(); return; } } 
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28

Examples of correct code for this rule:

/* eslint no-useless-return: "error" */ const foo = function() { return 5; } const bar = function() { return doSomething(); } const baz = function() { if (condition) { qux(); return; } else { quux(); } qux(); } const item = function() { switch (bar) { case 1: doSomething(); return; default: doSomethingElse(); } } const func = function() { for (const foo of bar) { return; } } 
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34

When Not To Use It

If you don’t care about disallowing redundant return statements, you can turn off this rule.

Version

This rule was introduced in ESLint v3.9.0.

Resources

• Rule source
• Tests source