PYL-W0613 · Function contains unused argument

KOSASIH / pipfs-hub

Active

Function contains unused argument PYL-W0613

Anti-pattern

Major

4 months ago — 4 months old

Unused argument 'root_dir'

1from typing import List
2
3def list_packages(root_dir: str) -> List[str]:4    """Return a list of package names in the given root directory."""
5    # ...

Unused argument 'config'

natural_language_processing/text_processing.py

 1import nltk
 2
 3
 4def preprocess_text(text, config): 5    # Preprocess text using the specified configuration
 6    # For example, tokenize, remove stop words, or stem the words
 7

Unused argument 'client'

models/device_drivers/device_communication.py

26        def on_connect(client, userdata, flags, rc):
27            client.subscribe(self.topic)
28
29        def on_message(client, userdata, msg):30            callback(msg.payload)
31
32        client = mqtt.Client()

Unused argument 'userdata'

models/device_drivers/device_communication.py

26        def on_connect(client, userdata, flags, rc):
27            client.subscribe(self.topic)
28
29        def on_message(client, userdata, msg):30            callback(msg.payload)
31
32        client = mqtt.Client()

Unused argument 'rc'

models/device_drivers/device_communication.py

23
24    def mqtt_subscribe(self, callback):
25        # Subscribe to a topic using MQTT protocol
26        def on_connect(client, userdata, flags, rc):27            client.subscribe(self.topic)
28
29        def on_message(client, userdata, msg):

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