Function contains unused argument PYL-W0613
Anti-pattern
Major
5 months ago
—
4 years old
Unused argument 'bot'
84
85
86@UserBot.on_message(filters.command("shorten", ".") & filters.me)
87async def shorten(bot: UserBot, message: Message): 88 keyword = None
89
90 if message.reply_to_message:Unused argument 'bot'
62
63
64@UserBot.on_message(filters.command("expand", ".") & filters.me)
65async def expand(bot: UserBot, message: Message): 66 if message.reply_to_message:
67 url = message.reply_to_message.text or message.reply_to_message.caption
68 elif len(message.command) > 1:Unused argument 'bot'
53
54
55@UserBot.on_message(filters.command("ping", ".") & filters.me)
56async def ping_me(bot: UserBot, message: Message): 57 """Ping the assistant"""
58 start = time.time()
59 reply = await message.reply_text("...")Unused argument 'bot'
15
16
17@UserBot.on_message(filters.command(["speed", "speedtest"], ".") & filters.me)
18async def speed_test(bot: UserBot, message: Message): 19 new_msg = await message.edit("`Running speed test . . .`")
20 spd = speedtest.Speedtest()
21Unused argument 'bot'
10from pyrogram import enums
11
12@UserBot.on_message(filters.command(["weather", "w"], ".") & filters.me)
13async def get_weather(bot: UserBot, message: Message):14 if len(message.command) == 1:
15 await message.edit("Usage: `.weather Maldives`")
16 await asyncio.sleep(3)
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