Как сделать вывод имени с SQL | Aiogram?

Question

[Alexndrev]

Делаю команду /dashboard. Столкнулся с тем что выводит только ID, но не имя.
Код в main.py

@dp.message_handler(commands = ["dashboard", "топ", "дешбоард", "дешборд"], commands_prefix='./')
async def top(message):
    db_rep.cursor.execute(f"SELECT id, rep FROM users ORDER BY rep DESC LIMIT 10")
    leadermsg = "*Топ експертов*:\n\n"
    fetchleader = db_rep.cursor.fetchall()
    for i in fetchleader:
        leadermsg += f"{fetchleader.index(i) + 1}| {i[0]}:  {i[1]} exp.\n"
    await message.reply(leadermsg)

Код из db_rep.py

import sqlite3

con = sqlite3.connect('reputation.db')
cursor = con.cursor()
def CreateDB():
    cursor = con.cursor()
    cursor.execute("CREATE TABLE IF NOT EXISTS users(name TEXT, id INT, rep INT)")
    con.commit()
def UpdateValue(val_name, new_val, id):
    for row in cursor.execute(f"SELECT {val_name} FROM users where id={id}"):
        new = row[0]+new_val
        cursor.execute(f"UPDATE users SET {val_name}={new} where id={id}")
        con.commit()

def InsertValue(name, id):
    cursor.execute(f'INSERT INTO users VALUES ("{name}", {id}, 0)')
    con.commit()

Answer 1

[Александр Нестеров]

А что, по вашему, должен выводить составленный вами запрос? Вы сами указали, какие столбцы нужно выводить:

db_rep.cursor.execute(f"SELECT id, rep FROM users ORDER BY rep DESC LIMIT 10")

И да, использование f-строк для sql-запросов недопустимо. Используйте корректные методы.

Comments

[Alexndrev]

нюанс в том, что если я ввожу в запросе вместо id, name - выводит только одно имя, а не все

[Александр Нестеров]

Alexndrev, в таком случае проверьте - насколько правильно вы формируете запрос.
В примере создал точно-такую-же как у вас таблицу и запрос, но с именем:

пример

import sqlite3
with sqlite3.connect('db.sqlite3') as conn:
	conn.execute("CREATE TABLE IF NOT EXISTS users(name TEXT, id INT, rep INT)")
	conn.execute("INSERT INTO users VALUES('username0',1,2323)")
	conn.execute("INSERT INTO users VALUES('username1',2,324)")
	conn.execute("INSERT INTO users VALUES('username2',3,23443)")
	conn.execute("INSERT INTO users VALUES('username3',4,43521)")
	conn.commit()
	cur = conn.cursor()
	result = cur.execute("SELECT id, name, rep FROM users ORDER BY rep DESC LIMIT 10").fetchall()
	result1 = cur.execute("SELECT * FROM users ORDER BY rep DESC LIMIT 10").fetchall()
	cur.close()
	print(result)
	print(result1)

Все работает отлично:

[(4, 'username3', 43521), (3, 'username2', 23443), (1, 'username0', 2323), (2, 'username1', 324)]
[('username3', 4, 43521), ('username2', 3, 23443), ('username0', 1, 2323), ('username1', 2, 324)]

И да, как видите можно выбирать все значения с помощью *

[Alexndrev]

Александр Нестеров, вижу, но всё также. Вот скрины:

[Александр Нестеров]

Alexndrev, ну и? Скрины это хорошо, но без кода, результатом выполнения которого являются эти скрины и описания того что так/не так, не очень понятно, чего вы ожидаете.
Тем ни менее:
На первом скрине видно, что вы пытаетесь вы печатаете cursor.execute() или какой-то другой его метод. В общем случае, когда вы выполняете запрос-выборку, по нему можно итерироваться, как и по итераторам/генераторам, но как и в случае с последними - напрямую получить данные из него нельзя. Нужно вызывать метод .fetchall()/.fetchone() Как в примере.
По второму скрину - вообще ничего конкретного сказать нельзя. Ну, есть список экспертов, он выводится.
Возможно, проблема в том, что у всех экспертов одинаковое имя. Тогда проблема в том, какие ограничения установлены на уровне базы для отсеивания дублирующихся имен (скажу сразу - из нет) и в том, каким способом вы проверяете при добавлении/обновлении значений, есть ли уже юзер с таким именем в базе (опять же, в приведенном коде таких проверок нет).

[Alexndrev]

Александр Нестеров, дело в том что юзеров несколько и всем присваивается одно имя. использую .fetchall()

[Александр Нестеров]

Alexndrev,
В таком случае - это комплексная проблема, требующая, по-факту реализации схемы снуля.
Прежде всего, нужно идентифицировать пользователя исключительно по неизменяемым данным, т.е. по его telegram-id (либо его id внутри группы, как в дискорде. Честно говоря, с ТГ я почти не работал, поэтому не могу сказать, как здесь это реализовано). В целом, вы делали шаги в эту сторону, однако есть несколько "но":

• Поле id должно быть уникальным. В доках sqlite есть пример как сделать поле таковым. Добавить одного и того-же пользователя два раза уже не получится.
  
• Создавать запись о юзере при его первом добавлении в группу. Тут два варианта - предварительно проверять, есть ли соответсвующие данние и если да, то ничего не делать. Альтернативным вариантом будет обработка исключений (напомню, id - уникален и при попытке повторно добавить идентичное значение, драйвер sqlite выдаст ошибку).
  
• То, как вы обновляете - неверно и крайне затратно по ресурсам. Обновлять значения можно одним запросом:
  

UPDATE users SET rep = rep * 2 WHERE id = 4 -- rep = rep * 2 здесь можно производить любые базовые мат.действия (сложение, вычитание, умножение и деление).

Ники можно хранить как в базе, так и получать из ТГ на основе id

Ну и как дополнение - нормализация баз данных. К сабжу отношения не имеет, но в целом полезная вещь.