Aogram Почему бот не отвечает на сообщение?

Question

[VeinKoor]

Бот не отвечает на сообщение, я думаю что это из за того, что обработчик в коде находится в самом конце, можно как то это исправить?

from aiogram import types, executor, Dispatcher, Bot
import sqlite3

token = ''

bot = Bot(token=token)
dp = Dispatcher(bot)

with sqlite3.connect("data.db") as db:
    cursor = db.cursor()

    cursor.execute("""CREATE TABLE IF NOT EXISTS users(
        chat_id TEXT not null ,
        username TEXT not null ,
        balance BIGINT
        
    )""")


@dp.message_handler(commands=['start'])
async def start(message: types.Message):
    cursor.execute(f"SELECT chat_id FROM users WHERE chat_id = '{message.chat.id}'") 

    if cursor.fetchone() is None:  
        cursor.execute(f"INSERT INTO users VALUES(?,?,?)", (message.chat.id, message.from_user.username, 0))  #
        await message.answer("База данных обновлена!")
        print("Данные занесены в таблицу")
        db.commit()




@dp.message_handler(content_types=['text'])
async def profile(msg: types.Message):
    if msg.text.lower() == "профиль":
        cursor.execute("SELECT * FROM users WHERE chat_id = '{msg.chat.id}'")
        for value in cursor.execute(f"SELECT * FROM users WHERE chat_id = '{msg.chat.id}'"):
            print(value[0], value[1], value[2])
            await msg.answer(f"id: " + value[0] + "\n" + "Имя: " + value[1] + "\n" + "Баланс: " + str(value[2]))





@dp.message_handler(content_types=['text'])
async def balance(msg: types.Message):
    if msg.text.lower() == "баланс":
        for value in cursor.execute(f"SELECT balance FROM users WHERE chat_id = {msg.chat.id}"):
            await msg.answer(f"Твой баланс: {value[0]}")




executor.start_polling(dp, skip_updates=True)

проблема с сообщением "баланс"

Comments

[Рамис]

Вспоминаем азы которых вы не знаете.

def hello():
    if msg == 'hello':
        ...
    elif msg == 'привет':
        ...

Answer 1

[shurshur]

Да, это потому что "в конце". Обработчики перебираются последовательно, и как находится подходящий, остальные не используются. А тут два обработчика с одинаковым декоратором, естественно, что первый будет применяться, а второй уже нет.

Надо или обрабатывать обе команды в одной функции, или отфильтровать в декораторе так, чтобы он применялся только к конкретным сообщениям

Условие commands=["баланс"] в декораторе позволит использовать команду как /баланс

А вот такой вариант:

@dp.message_handler(lambda message: message.text and (message.text.lower() == "баланс"))

позволит сделать то же самое без / перед словом "баланс". И вообще, так можно делать более сложные проверки.

Comments

[VeinKoor]

а func=lambda message: message.text.lower() нужно писать перед всеми сообщениями или только тут?

[shurshur]

VeinKoor, это условие в декораторе dp.message_handler

[VeinKoor]

shurshur, можете показать куда это вставить в коде? а то я чёт не понимаю :/

[shurshur]

VeinKoor, примерно так:

@dp.message_handler(commands=["профиль"])
async def profile(msg: types.Message):
    #if msg.text.lower() == "профиль": # это уже не нужно
        cursor.execute("SELECT * FROM users WHERE chat_id = '{msg.chat.id}'")
        for value in cursor.execute(f"SELECT * FROM users WHERE chat_id = '{msg.chat.id}'"):
            print(value[0], value[1], value[2])
            await msg.answer(f"id: " + value[0] + "\n" + "Имя: " + value[1] + "\n" + "Баланс: " + str(value[2]))

@dp.message_handler(func=lambda message: message.text.lower() == "баланс")
async def balance(msg: types.Message):
    #if msg.text.lower() == "баланс": # и это не нужно
        for value in cursor.execute(f"SELECT balance FROM users WHERE chat_id = {msg.chat.id}"):
            await msg.answer(f"Твой баланс: {value[0]}")

[VeinKoor]

shurshur, а если выдает ошибку NameError: Invalid filter name(s): 'func' ?

[shurshur]

VeinKoor, хм... Похоже это немного иначе делается (пример из доков):

@dp.message_handler(lambda message: message.text and 'hello' in message.text.lower())
async def text_handler(message: types.Message):

Параметр func есть у метода register_message_handler, который можно использовать вместо декоратора (менее наглядно).

Кстати, советую обратить внимание на этот пример. Если сообщение не текстовое, то message.text будет None и взять lower() от него будет нельзя.

[VeinKoor]

shurshur, а такой вариант может быть правильным?

@dp.message_handler(text = "профиль", content_types=['text'])


@dp.message_handler(text = "баланс", content_types=['text'])

[shurshur]

VeinKoor, судя по коду, там нет фильтра по тексту, но есть по регулярке, так что должно сработать:

regexp="^профиль$"
regexp="^баланс$"

Answer 2

[Totem]

Все хорошо, но почему используется асинхронный aiogram, и неасинхронный sqlite3?