В чем ошибка. Почему так?

Question

[Андрей Андреев]

Команда .lb это лидер боард, показывает того у кого самое большое количество денег, то есть он на первом месте, но почему то показывает только бота с балансом 0. Почему?

@client.command(aliases= ['leaderboard', 'lb'])
async def __leaderboard(ctx):
    embed = discord.Embed(title= 'Топ 10 сервера')
    counter = 0

    for row in cursor.execute("SELECT name, cash FROM users WHERE server_id = {} ORDER BY cash DESC LIMIT 10".format(ctx.guild.id)):
        counter += 1
        embed.add_field(
            name = f'# {counter} | {row[0]}',
            value = f'Баланс: {row[1]}',
            inline = False
            )
        connection.commit()
        await ctx.send(embed= embed)


ТАБЛИЦА:
@client.event
async def on_ready():
    cursor.execute("""CREATE TABLE IF NOT EXISTS users (
        name TEXT,
        id INT,
        cash BIGINT,
        rep INT,
        lvl INT,
        server_id INT
    )""")
    connection.commit()

    cursor.execute("""CREATE TABLE IF NOT EXISTS shop(
        role_id INT,
        id INT,
        cost BIGINT
        )""")

Comments

[Rsa97]

А кроме бота хоть кто-нибудь в таблице записан?

[o5a]

Раз ничего более не выводит, значит в этой таблице users ничего больше и нет (для данной guild.id).

[Андрей Андреев]

Rsa97, а да никого и нету, а почему? в таблице users есть в shop нету?

[Андрей Андреев]

o5a, а почему оно не добавляет пользователей

[Rsa97]

Андрей Андреев, Вам виднее. Кода добавления вы не написали.

[o5a]

Андрей Андреев, может забыли commit сделать, может неправильно добавление делали в целом. Без кода неизвестно.

[Андрей Андреев]

o5a, какую часть кода вас предоставить?

[o5a]

Андрей Андреев, как минимум ту, где заносите данные в эту users.

[Андрей Андреев]

КОМАНДА

@client.command(aliases= ['leaderboard', 'lb'])
async def __leaderboard(ctx):
    embed = discord.Embed(title= 'Топ 10 сервера')
    counter = 0
    for row in cursor.execute("SELECT name, cash FROM users WHERE server_id = {} ORDER BY cash DESC LIMIT 10".format(ctx.guild.id)):
        counter += 1
        embed.add_field(
            name = f'# {counter} | {row[0]}',
            value = f'Баланс: {row[1]}',
            inline = False
            )
        connection.commit()
        await ctx.send(embed= embed)

ВОТ ГИЛДИЯ
    for guild in client.guilds:
        for member in guild.members:
            if cursor.execute(f"SELECT id FROM users WHERE id = {member.id}").fetchone() is None:
                cursor.execute(f"INSERT INTO users VALUES ('{member}',{member.id}, 0, 0, 1, {guild.id})")
                connection.commit()
            else:
                pass

Таблицы я уже предоставил, мне кажется это все что относится к .leaderboard

[Андрей Андреев]

o5a, если нет то вот код

connection = sqlite3.connect('server.db')
cursor = connection.cursor()


@client.event
async def on_ready():
    cursor.execute("""CREATE TABLE IF NOT EXISTS users (
        name TEXT,
        id INT,
        cash BIGINT,
        rep INT,
        lvl INT,
        server_id INT
    )""")
    cursor.execute("""CREATE TABLE IF NOT EXISTS shop(
        role_id INT,
        id INT,
        cost BIGINT
        )""")


    for guild in client.guilds:
        for member in guild.members:
            if cursor.execute(f"SELECT id FROM users WHERE id = {member.id}").fetchone() is None:
                cursor.execute(f"INSERT INTO users VALUES ('{member}',{member.id}, 0, 0, 1, {guild.id})")
                connection.commit()
            else:
                pass

@client.event
async def on_member_join(member):
            if cursor.execute(f"SELECT id FROM users WHERE id = {member.id}").fetchone() is None:
                cursor.execute(f"INSERT INTO users VALUES ('{member}',{member.id}, 0, 0, 1, {member.guild.id})")
                connection.commit()
            else:
                pass

@client.command(aliases= ['balance', 'cash'])
async def __balance(ctx, member: discord.Member = None):
    if member is None: 
        if fetch := cursor.execute("SELECT cash FROM users WHERE id = {}".format(ctx.author.id)).fetchone():
            message = f"""Баланс пользователя **{ctx.author}** составляет **{fetch[0]} :small_orange_diamond:  **"""
        else:
            # тут создаешь пользователя
            message = f"""Баланс пользователя **{ctx.author}** составляет **0:small_orange_diamond: **"""
    else:
        if fetch := cursor.execute("SELECT cash FROM users WHERE id = {}".format(member.id)).fetchone():
            message = f"""Баланс пользователя **{member}** составляет **{fetch[0]} :small_orange_diamond:  **"""
        else:
            # тут создаешь пользователя
            message = f"""Баланс пользователя **{member}** составляет **0:small_orange_diamond: **"""

    await ctx.send(embed = discord.Embed(color=0xEC058E,
      description = message
      ))

@client.command(aliases= ['award'])
@commands.has_any_role('️ Модератор', 'Создатель')
async def __award(ctx, member: discord.Member = None, amount: int = None):
    if member is None:
        await ctx.send(f"**{ctx.author}**, укажите пользователя, которому желаете выдать определенную сумму")
    else:
        if amount is None:
            await ctx.send(f"**{ctx.author}**, укажите сумму, которую желаете начислить на счет пользователя")
        # elif amount <1:
        #     await ctx.send(f"**{ctx.author}**, укажите сумму большо 1 :small_orange_diamond: ")
        else:
            if cursor.execute(f"SELECT id FROM users WHERE id = {member.id}").fetchone() is None:
                cursor.execute(f"INSERT INTO users VALUES ('{member}',{member.id}, 0, 0, 1, {member.id})")
                cursor.execute("UPDATE users SET cash = cash + ? WHERE id = ?",(amount, member.id))
                connection.commit()
            else: 
                cursor.execute("UPDATE users SET cash = cash + ? WHERE id = ?",(amount, member.id))
                connection.commit()

            await ctx.message.add_reaction('✅')

@client.command(aliases = ['add-shop'])
@commands.has_any_role('️ Модератор', 'Создатель')
async def __add_shop(ctx, role: discord.Role = None, cost: int = None):
    if role is None:
        await ctx.send(f"**{ctx.author}**, укажите роль, которую вы желаете внести в магазин")
    else:
        if cost is None:
            await ctx.send(f"**{ctx.author}**, укажите стоимость для данной роли")
        elif cost < 0:
            await ctx.send(f"**{ctx.author}**, стоимость роли не может быть такой маленькой")
        else:
            cursor.execute("INSERT INTO shop VALUES ({}, {}, {})".format(role.id, ctx.guild.id, cost))
            connection.commit()

            await ctx.message.add_reaction('✅')


@client.command(aliases = ['remove-shop'])
@commands.has_any_role('️ Модератор', 'Создатель')
async def __remove_shop(ctx, role: discord.Role = None):
    if role is None:
        await ctx.send(f"**{ctx.author}**, укажите роль, которую вы удалить из магазина")
    else:        
        if fetch := cursor.execute("DELETE FROM shop WHERE role_id = {}".format(role.id)):
            connection.commit()

        await ctx.message.add_reaction('✅')


@client.command(aliases = ['shop'])
async def __shop(ctx):
    embed = discord.Embed(color = 0xDB952B, title = 'Магазин ролей')

    for row in cursor.execute("SELECT role_id, cost FROM shop WHERE id = {}".format(ctx.guild.id)):
        if ctx.guild.get_role(row[0]) != None:
            embed.add_field(
                name = f"Стоимость {row[1]}",
                value = f"Вы приобрете роль {ctx.guild.get_role(row[0]).mention}",
                inline = False
                )
        else:
            pass

    await ctx.send(embed = embed)

@client.command(aliases= ['buy', 'buy-role'])
async def __buy(ctx, role: discord.Role = None):
    if role is None:
        await ctx.send(f"**{ctx.author}**, укажите роль, которую вы желаете приобрести")
    else:
        if role in ctx.author.roles:
            await ctx.send(f"**{ctx.author}**, у вас уже имеется данная роль")
        elif fetch := cursor.execute("SELECT cost FROM shop WHERE role_id = {}".format(role.id)).fetchone()[0] > cursor.execute("SELECT cash FROM users WHERE id = {}".format(ctx.author.id)).fetchone()[0]:
            await ctx.send(f"**{ctx.author}**, у вас недостаточно средств для покупки данной роли")
        else:
            await ctx.author.add_roles(role)
            cursor.execute("UPDATE users SET cash = cash - {0} WHERE id = {1}".format(cursor.execute("SELECT cost FROM shop WHERE role_id = {}".format(role.id)).fetchone()[0], ctx.author.id))
            connection.commit()

            await ctx.message.add_reaction('✅')


@client.command(aliases = ['rep', '+rep'])
async def __rep(ctx, member: discord.Member = None):
    if member is None:
        await ctx.send(f"**{ctx.author}**, укажите участинка сервера")
    else:
        if member.id == ctx.author.id:
            await ctx.send(f"**{ctx.author}**, вы не можете указать самого себя")
        else:
            cursor.execute("UPDATE users SET rep = rep + {} WHERE id = {}".format(1, member.id))
            connection.commit()

            await ctx.message.add_reaction('✅')

@client.command(aliases= ['leaderboard', 'lb'])
async def __leaderboard(ctx):
    embed = discord.Embed(title= 'Топ 10 сервера')
    counter = 0
    for row in cursor.execute("SELECT name, cash FROM users WHERE server_id = {} ORDER BY cash DESC LIMIT 10".format(ctx.guild.id)):
        counter += 1
        embed.add_field(
            name = f'# {counter} | {row[0]}',
            value = f'Баланс: {row[1]}',
            inline = False
            )
        connection.commit()
        await ctx.send(embed= embed)

[o5a]

Андрей Андреев, я не особо знаю работу ботов для дискорда, не в курсе особенностей привязки гильдий и т.п.
В целом занесение в users адекватное. Только вижу моменты:

1. Все запросы лучше исполнять не текстовым форматирование, а передачей параметров, когда вторым параметром execute передается кортеж с значениями для подстановки в запрос, т.е. для данного запроса

if cursor.execute(f"SELECT id FROM users WHERE id = {member.id}").fetchone() is None:

правильнее делать так:

if cursor.execute("SELECT id FROM users WHERE id = ?", (member.id, )).fetchone() is None:

аналогично для всех запросов лучше сделать
2. Если в таблице users пользователи хранятся с привязкой к серверу (guild.id), то и проверять стоит по обоим критериям.
не

if cursor.execute("SELECT id FROM users WHERE id = ?", (member.id, )).fetchone() is None:

а

if cursor.execute("SELECT id FROM users WHERE id = ? AND server_id = ?", (member.id, guild.id)).fetchone() is None:

Потому что в первом варианте если пользователь занесен в users, но при этом с привязкой к другому серверу, то запрос посчитает, что такая запись уже есть и добавлять не будет.
Опять же, я не в курсе структуры discord, может быть это нормально, но по логике программы скорее всего нужно проверять именно по обоим параметрам.

Так что если on_member_join(member) у Вас срабатывает, то пользователи в базу должны заноситься (с учетом упомянутого в п.2). Если и при этом не заносятся, добавьте логирование или хотя бы просто print для проверки что за данные попадают там на вставку.