Function contains unused argument PYL-W0613
Anti-pattern
Major
3 months ago
—
3 months old
Unused argument 'uniqueid'
163 return res
164
165
166def process7(uniqueid, fn):167 """Kellogg"""
168 df = pd.read_excel(fn)
169 x = {}Unused argument 'uniqueid'
110 return {}
111
112
113def process6(uniqueid, dirname):114 """HICKS"""
115 os.chdir(dirname)
116 files = glob.glob("*2015.xlsx")Unused argument 'environ'
29
30
31@iemapp()
32def application(environ, start_response): 33 """Go Main"""
34 start_response("200 OK", [("Content-type", "text/html")])
35 cursor.execute(
Description
An unused argument can lead to confusions. It should be removed. If this variable is necessary, name the variable _ or start the name with unused or _unused.
Bad practice
def square(x, y=1):
return x * x
class MySubClass(MyClass):
def __init__(self, number):
self.value = 42 # argument `number` remains unused
Preferred:
def square(x):
return x * x
class MySubClass(MyClass):
def __init__(self, _):
self.value = 42