What is wrong in this Python code (discord.py)?

Question

@client.command(name="delete_role", pass_context=True)
async def delete_role(ctx):
    for role in ctx.guild.roles:
        if role.name != "Team" or "Team(OLD)" or "Admin" or "Админ" or "Rythm" or "Toxic Bot" or "@everyone":
            await role.delete()
            await ctx.send(f"deleted {role}")

I was trying to delete all roles on my server, but those which I choose.

Are you getting an error? or just didn't behave as expected? — Joac
– Joac, Commented Nov 28, 2020 at 16:42

Joac · Accepted Answer · 2020-11-28 16:47:31Z

3

Given you actual code, the expresion that you have on the if statement will always evaluate to True, because all non empty strings are evaluated to True in python.

>>> bool("")
False
>>> bool("a")
True

You can change your code to something like this

PRESERVE_USERS = [
    "Team", 
    "Team(OLD)", 
    "Admin", 
    "Админ", 
    "Rythm", 
    "Toxic Bot", 
    "@everyone",
]


@client.command(name="delete_role", pass_context=True)
async def delete_role(ctx):
    for role in ctx.guild.roles:
        if role.name not in PRESERVE_USERS:  
            await role.delete()
            await ctx.send(f"deleted {role}")

answered Nov 28, 2020 at 16:47

Joac

4903 silver badges12 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

What is wrong in this Python code (discord.py)?

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related