Remove Unnecessary Else¶
Sourcery refactoring id:
else after guard condition
def f(a=None): if a is None: return 42 else: # some long calculations var = (i 2 for i in range(a)) return sum(var)
def f(a=None): if a is None: return 42 # some long calculations var = (i 2 for i in range(a)) return sum(var)
A common code pattern is to have some clauses at the start of a function, to check whether certain conditions have been fulfilled and return early or raise an exception if not.
While this is perfectly valid code, it can run into problems with excessive nesting, particularly if the rest of the function is fairly long.
Here we can take advantage of the fact that we don't need the
else if the main
body of the
if breaks the control flow by ending with
Rewriting the function as shown here is logically equivalent.
Using a guard condition, or multiple guard conditions, in this way now doesn't cause the rest of the function to be indented. In general the less we have to deal with indents the easier the code is to understand.