Почему бот отправляет результат действия мне, а не юзеру?

Question

[Михаил Евсеев]

Добрый день, проблема в том, что при использовании бота мной, все работает верно, но когда его пытаются использовать другие он отвечает им на все команды, но результат вычисления отправляет почему то мне, в чем дело?

@bot.message_handler(commands=['spam'])
def start_message(message):
    bot.send_message(message.chat.id, 'Пиши что угодно')
    @bot.message_handler(content_types=['text'])
    def get_text(message):
        text = message.text
        markup = telebot.types.InlineKeyboardMarkup()
        markup.add(telebot.types.InlineKeyboardButton(text='Десять', callback_data=10))
        markup.add(telebot.types.InlineKeyboardButton(text='Пятьдесят', callback_data=50))
        markup.add(telebot.types.InlineKeyboardButton(text='Сто', callback_data=100))
        bot.send_message(message.chat.id, text="Выбери сколько раз повторить", reply_markup=markup)

        @bot.callback_query_handler(func=lambda call: True)
        def query_handler(call):
            bot.answer_callback_query(callback_query_id=call.id, text='11111')
            so = []
            bot.send_message(message.chat.id, f'Пишем {call.data} раз')
            for i in range(int(call.data)):
                so.append(text)
                sos = '\n'.join(so)
            bot.send_message(message.chat.id, sos)

Понимаю, что вопрос легкий и ошибка, вероятно в том, что я где то не указал путь отправки, но я в упор не вижу ее(

Comments

[sinneren]

я не питонист и вообще мимокрокодил, но, кажется, надо исправить на call.message.chat.id

[Михаил Евсеев]

sinneren, Если вы говорите про def query_handler(call.message.chat.id) то такого варианта нет, python ругается(

[sinneren]

Михаил Евсеев, я про query_handler, но зачем в пропсе сразу call.message.chat.id, когда внутри функции, точно ли на это ругается. ибо с таким указанием уже должен был ругнуться на call.* вызовы. но, повторюсь, книжку по питону я читал 12 лет назад)) хотя имею опыт написания бота на js

Answer 1

[Михаил Кростелев]

Зачем вы сделали вложенную структуру функций? Они все должны быть на верхнем уровне

@bot.message_handler(commands=['spam'])
def start_message(message):
    bot.send_message(message.chat.id, 'Пиши что угодно')

@bot.message_handler(content_types=['text'])
def get_text(message):
    text = message.text
    markup = telebot.types.InlineKeyboardMarkup()
    markup.add(telebot.types.InlineKeyboardButton(text='Десять', callback_data=10))
    markup.add(telebot.types.InlineKeyboardButton(text='Пятьдесят', callback_data=50))
    markup.add(telebot.types.InlineKeyboardButton(text='Сто', callback_data=100))
    bot.send_message(message.chat.id, text="Выбери сколько раз повторить", reply_markup=markup)

@bot.callback_query_handler(func=lambda call: True)
def query_handler(call):
    bot.answer_callback_query(callback_query_id=call.id, text='11111')
    so = []
    bot.send_message(message.chat.id, f'Пишем {call.data} раз')
    for i in range(int(call.data)):
        so.append(text)
        sos = '\n'.join(so)
    bot.send_message(call.message.chat.id, sos)

И да, в последней строке обращаться к объекту message напрямую нельзя, через call только

Comments

[Михаил Евсеев]

Спасибо, все же python ругался на то что при предложенной вами структуре он не наследует message, text. А также появился странный баг что он выводит рандомный текст, который я когда то задал)

[Михаил Кростелев]

Михаил Евсеев, откуда вы вообще взяли инфу, что все хэндлеры нужно объявлять друг в друге?

[Михаил Евсеев]

Михаил, А как иначе сделать сложные условия. ТЕ мне нужно считывать любой текст только после введенной /spam, если расположить из не друг в друге все что угодно кроме /pass будет попадать в хендлер сразу, а не после ввода /spam

[Михаил Кростелев]

Михаил Евсеев, Для этого в telebot (насколько я понял вы им пользуетесь) есть register_next_step_handler. Гуглите, изучайте.

[shurshur]

Михаил Евсеев, нет, нельзя так делать "сложные условия". Для этого есть register_next_step_handler (посмотреть в документации примеры).

[Михаил Евсеев]

shurshur,Спасибо,может быть проблема решится благодаря этому. Не могли бы вы рассказать в чем проблема таких сложных условий?

[shurshur]

Михаил Евсеев, потому что обработчик обрабатывает ВСЕ сообщения, удовлетворяющие указанным в его определении условиям (тип контента / команда / функция), Метод register_next_step_handler означает "следующее сообщение именно от этого пользователя обработать вот тем другим обработчиком".

[Михаил Евсеев]

shurshur, Понял, спасибо. сейчас почитал про register_next_step_handler. Почти получилось его внедрить только кроме обычных хенделеров был один callback_query_handler. И когда я убрал его и заменил на register_next_step_handler все перестало работать.

@bot.message_handler(commands=['spam'])
def start_message(message):
    msg = bot.send_message(message.chat.id, 'Пиши что угодно')
    bot.register_next_step_handler(msg, get_text)


def get_text(message):
    text = message.text
    markup = telebot.types.InlineKeyboardMarkup()
    markup.add(telebot.types.InlineKeyboardButton(text='Десять', callback_data=10))
    markup.add(telebot.types.InlineKeyboardButton(text='Пятьдесят', callback_data=50))
    markup.add(telebot.types.InlineKeyboardButton(text='Сто', callback_data=100))
    msg = bot.send_message(message.chat.id, text="Выбери сколько раз повторить", reply_markup=markup)
    bot.register_next_step_handler(msg, query_handler)


def query_handler(call, text):
    bot.answer_callback_query(callback_query_id=call.id, text='11111')
    so = []
    bot.send_message(call.message.chat.id, f'Пишем {call.data} раз')
    for i in range(int(call.data)):
        so.append(text)
        sos = '\n'.join(so)
    bot.send_message(call.message.chat.id, sos)

[Михаил Кростелев]

Да, проблема в том, что register_next_step_handler не умеет в работу с инлайн кнопками. Ну или в новых версиях telebot уже что-то можно сделать, давно не изучал вопрос.
Решения как минимум 2:
1. Переписать все на aiogram и использовать FSM (машина состояний или конечный автомат)
2. Если сначала идет сбор каких-то данных от пользователя и в конце ему надо нажать на кнопку, чтобы та запустила функцию с ранее введенными параметрами, то просто в кнопку в callback_data записывай эти самые параметры, например json-строкой. А при обработке нажатия вытаскивай все эти данные из кнопки. Проблема может быть в том, что эти самые данные могут быть большого объема и сама API телеги отболтит этот запрос.

[Михаил Евсеев]

Михаил Евсеев, Проблема именно в query_handler. Сейчас я подредачил, выглядит лучше, но в строке 'so.append(text)'; text подсвечивается красным тк его нет в функции, а когда я пытаюсь его туда ввести в строке def query_handler(call, text) Он пишет что нельзя туда вписать text.

@bot.callback_query_handler(func=lambda call: True)
def query_handler(call):
    bot.answer_callback_query(callback_query_id=call.id, text='11111')
    so = []
    bot.send_message(call.message.chat.id, f'Пишем {call.data} раз')
    for i in range(int(call.data)):
        so.append(text)
        sos = '\n'.join(so)
    bot.send_message(call.message.chat.id, sos)

[shurshur]

Михаил Евсеев, callback_query_handler - это обработчик нажатия на inline-кнопку. А тут надо обрабатывать следующее именно сообщение, а не нажатие на кнопку.

Обычные (не inline) кнопки генерируют обычные сообщения, inline-кнопки генерируют inline callback запросы.

Так что в приведённом примере какая-то фигня написана. Нажатия на кнопки надо обрабатывать именно callback_query_handler, а вот непосредственно ввод текста - через next step handler.

Проще всего сделать так, что callback_data кнопки полностью идентифицировал смысл её глобально в пределах бота (чтобы можно было делать и другие кнопки, в том числе с числами в аргументе):

... callback_data='repeat:10'
... callback_data='repeat:50'
... callback_data='repeat:100'

@bot.callback_query_handler(func=lambda call: call.data.startswith('repeat:')
def repeat_query_handler(call):
    global user_repeat_texts
    repeat_count = int(call.data.replace('repeat:',''))
    ...

@bot.callback_query_handler(func=lambda call: call.data == 'another_button'):
def another_handler(call):
    # тут действия по обработке другой кнопки

[Михаил Евсеев]

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

def get_text(message):
    text = message.text
    markup = telebot.types.InlineKeyboardMarkup()
    markup.add(telebot.types.InlineKeyboardButton(text='Десять', callback_data=10))
    markup.add(telebot.types.InlineKeyboardButton(text='Пятьдесят', callback_data=50))
    markup.add(telebot.types.InlineKeyboardButton(text='Сто', callback_data=100))
    msg = bot.send_message(message.chat.id, text="Выбери сколько раз повторить", reply_markup=markup)
    bot.register_next_step_handler(msg, create_spam)


@bot.callback_query_handler(func=lambda call: True)
def take_digit(call):
    bot.send_message(call.message.chat.id, f'Вы выбрали {call.data} раз')


def create_spam(text, call):
    so = []
    for i in range(int(call.data)):
        so.append(text)
        sos = '\n'.join(so)
    bot.send_message(call.message.chat.id, sos)

[Михаил Кростелев]

Короче. Если запрашиваемый вначале текст у вас небольшой, можете сделать примерно так:

@bot.message_handler(commands=['spam'])
def start_message(message):
    msg = bot.send_message(message.chat.id, 'Пиши что угодно')
    bot.register_next_step_handler(msg, get_text)


def get_text(message):
    text = message.text
    markup = telebot.types.InlineKeyboardMarkup()
    markup.add(telebot.types.InlineKeyboardButton(text='Десять', callback_data=f'10|{text}'))
    markup.add(telebot.types.InlineKeyboardButton(text='Пятьдесят', callback_data=f'50|{text}'))
    markup.add(telebot.types.InlineKeyboardButton(text='Сто', callback_data=f'100|{text}'))
    bot.send_message(message.chat.id, text="Выбери сколько раз повторить", reply_markup=markup)

@bot.callback_query_handler(func=lambda call: True)
def query_handler(call):
    count, text = call.data.split('|')
    count = int(count)
    bot.answer_callback_query(callback_query_id=call.id, text='11111')
    so = [text]*count
    bot.send_message(message.chat.id, f'Пишем {call.data} раз')
    sos = '\n'.join(so)
    bot.send_message(call.message.chat.id, sos)

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

[Михаил Евсеев]

Огромное спасибо, работает!