pylint-errors

R1705 (no-else-return)

Problematic code:

def foo(x):
    if x:
        return 1
    else:
        return 2

Correct code:

def foo(x):
    if x:
        return 1
    return 2

Rationale:

Unnecessary else after return.

Used in order to highlight an unnecessary block of code following an if containing a return statement. As such, it will warn when it encounters an else following a chain of ifs, all of them containing a return statement.

Related resources:

• Testcases
• Issue Tracker
• StackOverflow (It is more efficient to use if-return-return or if-else-return?)

This site is open source. Improve this page.