Exception might be lost PYL-W0150
Bug risk
Major
a month ago
—
a year old
return statement in finally block may swallow exception
70 except discord.HTTPException:
71 pass
72 finally:
73 return 74
75 if gen_data.defaultRole:
76 default_role = member.guild.get_role(int(gen_data.defaultRole))return statement in finally block may swallow exception
51 except discord.HTTPException:
52 pass
53 finally:
54 return 55
56 count = len(gen_data.openChannels) if gen_data.openChannels else 0
57 editable = VoiceGeneratorOption.EDITABLE in gen_data.defaultOptions
Description
The finally block may swallow any exceptions raised in the try block. This can pose a bug risk.
Bad practice
def some_function():
try:
with open('myfile.txt') as file:
data = file.read()
except FileNotFoundError:
print(fnf_error)
finally:
return data
Recommended
def some_function():
try:
with open('myfile.txt') as file:
data = file.read()
except FileNotFoundError:
print(fnf_error)
return data