Как создать клавиши с названиями из списка?

Question

[Corn385]

библиотека aiogam.
У меня есть список, и нужно чтобы создались кнопки типа KeyboardButton с названиями из этого списка. Т.е. допустим список такой [('print',), ('enter',), ('delete',), ('add',), ('home',)] (этот список получается из запроса в бд). И нужно чтобы создалось 5 кнопок(именно столько элементов в списке) типа KeyboardButton(клавиатура грубо говоря), с соответствующими названиями.

async def choice_cat(message):
    CATs = cur.execute(f'SELECT name FROM cathlete WHERE id = "{message.from_user.id}"').fetchall()
    for fds in range(len(CATs)):
        astt = ReplyKeyboardMarkup(resize_keyboard=True).add(KeyboardButton(f"CATs[{fds}]"))

То что начинается с "for", к осмыслению не принимать, моя больная фантазия! Пытаюсь методом тыка найти способ, но безуспешно(

Comments

[Сергей Горностаев]

Офигеть, ошибка буквально в каждой строке! В некоторых строках даже по две ошибки!

Answer 1

[Vindicar]

async def choice_cat(message):
    CATs = cur.execute(f'SELECT name FROM cathlete WHERE id = "{message.from_user.id}"').fetchall()
    for fds in range(len(CATs)):
        astt = ReplyKeyboardMarkup(resize_keyboard=True).add(KeyboardButton(f"CATs[{fds}]"))

Код, конечно, жесть. По пунктам:
1. НИКОГДА не генерируй SQL запрос через форматирование строк. Легко словить ошибку, а если очень не повезёт - SQL-инъекцию. Используй подстановку параметров (ссылка для sqlite, если у тебя другая БД - ищи доки на неё).
2. Как насчёт проверить, а вернула ли БД хоть что-то? Это не гарантируется. Всегда ожидай подставы.
3. Зачем итерироваться по индексу списка, если ты индекс не используешь? Почему не итерироваться по самому списку? И да, это абсолютные основы питона, которые НАДО знать задолго до того, как браться за ботов.
4. Зачем ты создаёшь новую клавиатуру на каждой итерации цикла? Тут уже даже не знания языка, тут простая логика. Создал клавиатуру, добавил кнопку - на следующей итерации выбросил старую клавиатуру, создал новую добавил другую кнопку. Ну бессмыслица же! Создай клавиатуру один раз, до цикла, и добавляй кнопки к ней.
5. Ну тот факт, что ты не посылаешь сообщение, к которому будет прикреплена клавиатура (а сама по себе она не бывает!), это уже мелочи. Видимо, не дошёл ещё до этого этапа.

kbd = ReplyKeyboardMarkup(resize_keyboard=True)
btns = cur.execute(f'SELECT name FROM cathlete WHERE id = ?', (message.from_user.id,)).fetchall()
# btns - список списков. 
if not btns:
    # кнопки не предусмотрены, как-то даём пользователю знать об этом
    return  # если это фатально, останавливаемся тут
    # иначе создаём какое-то значение btns по умолчанию и  продолжаем
    # тут уж сам решай
# делаем клавиатуру
for btn in btns:
    kbd.add(KeyboardButton(btn[0]))
# ну и отправляем её вместе с сообщением

Comments

[Corn385]

только вот кнопки не создаются.
если вывести что делает цикл в консоль, то получается такая картина:
{"keyboard": [[{"text": ["print"]}], [{"text": ["print"]}]], "resize_keyboard": true}
{"keyboard": [[{"text": ["print"]}], [{"text": ["print"]}], [{"text": ["enter"]}], [{"text": ["enter"]}]], "resize_keyboard": true}
{"keyboard": [[{"text": ["print"]}], [{"text": ["print"]}], [{"text": ["enter"]}], [{"text": ["enter"]}], [{"text": ["delete"]}], [{"text": ["delete"]}]], "resize_keyboard": true}
{"keyboard": [[{"text": ["print"]}], [{"text": ["print"]}], [{"text": ["enter"]}], [{"text": ["enter"]}], [{"text": ["delete"]}], [{"text": ["delete"]}], [{"text": ["add"]}], [{"text": ["add"]}]], "resize_keyboard": true}
{"keyboard": [[{"text": ["print"]}], [{"text": ["print"]}], [{"text": ["enter"]}], [{"text": ["enter"]}], [{"text": ["delete"]}], [{"text": ["delete"]}], [{"text": ["add"]}], [{"text": ["add"]}], [{"text": ["home"]}], [{"text": ["home"]}]], "resize_keyboard": true}

[Vindicar]

Corn385, поправил.

[Corn385]

Vindicar, очень странно...
Выдает тоже самое

[Vindicar]

Corn385, полный код в студию. (Ну или полный фрагмент, воспроизводящий проблему.)

[Corn385]

Vindicar,

async def choice_cat(message):
	CATs = cur.execute(f'SELECT name FROM cathlete WHERE id = ?', (message.from_user.id, )).fetchall()
	astt = ReplyKeyboardMarkup(resize_keyboard=True)
	for i in CATs:
		astt.add(KeyboardButton(i[0]))
		fsa = astt.add(KeyboardButton(i[0]))
		print(fsa)

В тг ничего не меняется. В консоль выводит:

{"keyboard": [[{"text": "print"}], [{"text": "print"}]], "resize_keyboard": true}
{"keyboard": [[{"text": "print"}], [{"text": "print"}], [{"text": "enter"}], [{"text": "enter"}]], "resize_keyboard": true}
{"keyboard": [[{"text": "print"}], [{"text": "print"}], [{"text": "enter"}], [{"text": "enter"}], [{"text": "delete"}], [{"text": "delete"}]], "resize_keyboard": true}
{"keyboard": [[{"text": "print"}], [{"text": "print"}], [{"text": "enter"}], [{"text": "enter"}], [{"text": "delete"}], [{"text": "delete"}], [{"text": "add"}], [{"text": "add"}]], "resize_keyboard": true}
{"keyboard": [[{"text": "print"}], [{"text": "print"}], [{"text": "enter"}], [{"text": "enter"}], [{"text": "delete"}], [{"text": "delete"}], [{"text": "add"}], [{"text": "add"}], [{"text": "home"}], [{"text": "home"}]], "resize_keyboard": true}

[Vindicar]

Corn385, во-первых, зачем ты добавляешь кнопку дважды?
Во-вторых, читай мой ответ

Ну тот факт, что ты не посылаешь сообщение, к которому будет прикреплена клавиатура (а сама по себе она не бывает!), это уже мелочи.

От того, что ты создал клавиатуру, ничего не изменится. Тебе нужно отправить пользователю сообщение, и прикрепить клавиатуру к нему.
Смотри пример на гитхабе.

[Corn385]

Vindicar, Спасибо за помощь, но не мог бы ты посмотреть мой последний вопрос про машину состояния?