Дискорд бот. В чем проблема?

Question

[NekIT]

Делаю бота который будет банить людей по команде с кд 1 час и банить могут только участники с определенной ролью.

@commands.has_role('1')
async def ban(ctx, member: discord.Member, days=1):
    if member is None:
        await ctx.send(f'{error_emoji} **Пожалуйста, укажите пользователя!** {error_emoji}')
    else:
        try:
            await Bot.ban(member, delete_message_days=0)
            await ctx.send('User banned for **' + str(days) + ' day(s)**')
            ban_list.append(member)
            day_list.append(days * 24 * 60 * 60)
            server_list.append(ctx.message.server)
        except:
            await ctx.send('Error! User not active')
        await ctx.send('You do not have permission to ban users!')

Вместо бана на команду -ban @iSovBot 2 отвечает вот этим:


В чем ошибка? Роль я себе выдал, роль бота поставил выше всех и всё равно...

Comments

[soremix]

В чем ошибка?

По-моему, нужно убрать бесполезный try... except, чтобы понять в чем ошибка

[NekIT]

soremix, говорит что не знает такой команды :/

discord.ext.commands.errors.CommandInvokeError: Command raised an exception: AttributeError: 'Bot' object has no attribute 'ban'

[Shandy]

NekIT, не "не знает такой команды", а нет метода. https://discordpy.readthedocs.io/en/latest/api.htm...

[soremix]

Никита, потому что нет такого метода
https://discordpy.readthedocs.io/en/stable/ext/com...

Метод ban есть у класса Discord.Member
https://discordpy.readthedocs.io/en/stable/api.htm...
и у Guild - https://discordpy.readthedocs.io/en/stable/api.htm...

Так что member.ban()

[NekIT]

soremix,

Ignoring exception in command ban:
Traceback (most recent call last):
  File "C:\Users\User\AppData\Local\Programs\Python\Python38-32\lib\site-packages\discord\ext\commands\core.py", line 85, in wrapped
    ret = await coro(*args, **kwargs)
  File "C:/Users/User/PycharmProjects/Discord_Bots/revoluton bot.py", line 28, in ban
    await member.ban(member, delete_message_days=0)
TypeError: ban() takes 1 positional argument but 2 were given

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "C:\Users\User\AppData\Local\Programs\Python\Python38-32\lib\site-packages\discord\ext\commands\bot.py", line 903, in invoke
    await ctx.command.invoke(ctx)
  File "C:\Users\User\AppData\Local\Programs\Python\Python38-32\lib\site-packages\discord\ext\commands\core.py", line 855, in invoke
    await injected(*ctx.args, **ctx.kwargs)
  File "C:\Users\User\AppData\Local\Programs\Python\Python38-32\lib\site-packages\discord\ext\commands\core.py", line 94, in wrapped
    raise CommandInvokeError(exc) from exc
discord.ext.commands.errors.CommandInvokeError: Command raised an exception: TypeError: ban() takes 1 positional argument but 2 were given

Даже если оставить аргумент member, а все остальные удалить, ничего не меняется.

[Vindicar]

NekIT, включи логику, а?
await member.ban(member, delete_message_days=0)

Ты вызываешь бан на объекте member, тем самым указываешь, кого банишь.
Вот на кой тебе еще раз передавать member в метод ban?

[NekIT]

Vindicar, а как должна выглядеть эта команда? Я первый раз ее пишу, и первый раз сталкиваюсь с этой проблемой

[Shandy]

NekIT, тебе скинули ссылки на документацию, читай

[Vindicar]

NekIT, тебя ошибка прямо говорит: ban() takes 1 positional argument but 2 were given
Нужно иметь ввиду, что у методов объектов первый аргумент - всегда self, объект, у которого этот метод вызывается. Так что по факту это сообщение следует понимать так: ban() принимает 1-1 = 0 позиционных аргументов, а получил при вызове 2-1 = 1 аргумент. Т.е. у тебя один лишний позиционный аргумент.

Смотрим документацию:
await ban(**kwargs)
В сигнатуре действительно нет позиционных аргументов, только **kwargs, т.е. некий набор keyword аргументов.

Смотрим еще раз твой вызов:
await member.ban(member, delete_message_days=0)
У тебя есть 1 позиционный аргумент member, и один keyword аргумент delete_message_days.

При этом мало того, что документация говорит "позиционных аргументов быть не должно", но и просто по логике тебе незачем иметь этот аргумент. Он указывает, кого банить - но так как мы вызываем метод на объекте Member, т.е. на участнике, значит, по логике, вызов метода ban() должен банить именно этого участника.

Вот если бы ты вызывал метод ban() на объекте Guild, там набор параметров другой, и указывать member необходимо. Документация это подтверждает.

Если есть трудности с построением таких логических цепочек, отложи бота, и потренируйся на более простых примерах по Питону в целом и ООП в частности.

Answer 1

[Hawchik_dev]

Вы не правильно указали условие наличия роли.

Попробуйте вот это:

@commands.has_permissions(ban_members=True)
async def ban(ctx, member: discord.Member, days=1):
    if member is None:
        await ctx.send(f'{error_emoji} **Пожалуйста, укажите пользователя!** {error_emoji}')
    else:
        try:
            await ctx.guild.ban(member, delete_message_days=0)
            await ctx.send('User banned for **' + str(days) + ' day(s)**')
            ban_list.append(member)
            day_list.append(days * 24 * 60 * 60)
            server_list.append(ctx.message.server)
        except:
            await ctx.send('Error! User not active')