Function contains unused argument PYL-W0613
Anti-pattern
5 months ago
—
5 months old
Unused argument 'corners_by_contour'
104
105 # @TODO: Implement this method and return the average corners.
106 def validate_corners(
107 self, corners_by_harris: list, corners_by_contour: list108 ) -> list:
109 self.debug.info("validate_corners(): Function started.")
110 self.debug.error("NOT IMPLEMENTED!")
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