Почему заносит одного пользователя несколько раз?

Question

[nikto_211]

Почему заносит одного пользователя несколько раз? При этом других тоже заносит.

@bot.slash_command(description="Занести всех в бд")
async def db(interaction: nextcord.Interaction):
    for guild in bot.guilds:
        print(guild.id)
        for member in guild.members:
            print(member.name)
            cursor.execute(f"SELECT ID FROM Users WHERE ID={member.id}")
            if cursor.fetchone() is None:
                cursor.execute(f"INSERT INTO Users VALUES ({member.id}, '{member.name}',0, 0)")
                connection.commit()
                await interaction.send(f'Занес {member.name} в DataBase', ephemeral=True)
            else:
                pass

Comments

[Vindicar]

НИКОГДА не используй f-строки для конструирования SQL запросов. Освой уже параметризованные запросы.

Answer 1

[fenrir]

Потому что cursor.fetchone() is None всегда будет False.

Comments

[nikto_211]

И как мне исправить? Если не трудно

[fenrir]

nikto_211, лучшим способом было бы вообще не делать такую команду и не хранить мертвые души, а заносить людей по необходимости.

Так же стоит познакомиться с опцией ON CONFLICT, которая используется вместо подобных селект-инсертов.

Ну и самый простой, но кривой вариант посмотреть принтом, что лежит в fetchone и проверять, например длину.