Useless
return detected PYL-R1711
Anti-pattern
Minor
5 months ago
—
4 years old
Useless return at end of function or method
186 return
187
188
189def _upload_media(task_handle, event_id, base_path):190 """
191 Actually uploads the resources
192 """
Description
A single return or return None statement is found at the end of the function or method definition. This statement can safely be removed to improve the readability because Python will implicitly return None.
Bad practice
def log_data(data):
'''Sends the data to our logging API.'''
api = get_api()
api.send(data)
return # Unnecessary return
def print_pattern():
print('*')
print('**')
print('***')
return None # Unnecessary return
Recommended
def log_data(data):
'''Sends the data to our logging API.'''
api = get_api()
api.send(data)
def print_pattern():
print('*')
print('**')
print('***')