Re-definition found for builtin function PYL-W0622
Anti-pattern
4 months ago
—
5 months old
Redefining built-in 'set'
12
13
14@app.route('/set/<pin:int>/<state:int>')
15def set(pin, state):16 if pin not in pins:
17 abort(code=404, text="pin useless")
18 if state not in (1, 0):
Description
Defining a local variable or function with the same name as a built-in object makes the built-in object unusable within the current scope and makes the code prone to bugs.
Bad practice
dict = {} # dict is a Python builtin!
for idx, student in enumerate(students):
dict[student] = idx
Preferred:
roll_number = {}
for idx, student in enumerate(students):
roll_number[student] = idx