Last if statement guard
Sourcery refactoring id:
Convert the final conditional into a guard clause
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
breaks the control flow by ending with
raise. 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.