Как сделать полное выполнение условия if-elif Python?

Question

[1LIK]

Проблема вот здесь:

elif message.text == 'Оценить тебя':
        bot.send_message(message.chat.id, 'Отправьте свою фотографию')
        @bot.message_handler(content_types=['photo'])
        def get_photo(message):
            bot.send_message(message, 'Очень красивая фотография!')

Бот выполняет первый шаг условия, отправка сообщения, но дальше не выполняет саму "оценку"
Весь код ниже:

import telebot
from telebot import types

bot = telebot.TeleBot('TOKEN')
name = 'bot_name'

@bot.message_handler(commands=['start'])
def message_enter(message):
    markup = types.ReplyKeyboardMarkup()
    get_to_know_me = types.KeyboardButton('О боте')
    get_to_know_currency = types.KeyboardButton('Курс валюты')
    rate_you = types.KeyboardButton('Оценить тебя')
    markup.row(get_to_know_me)
    markup.row(get_to_know_currency, rate_you)
    bot.send_message(message.chat.id,
                     f'Привет, {message.from_user.first_name}! Что бы ты хотел сделать?', reply_markup=markup)
    bot.register_next_step_handler(message, on_click)

def on_click(message):
    if message.text == 'О боте':
        bot.send_message(message.chat.id,
                         f'Меня зовут {name}. Я написан для того, чтобы оценивать твои фотографии и давать тебе свежий курс валют')
    elif message.text == 'Курс валюты':
        markup = types.InlineKeyboardMarkup()
        eur = types.InlineKeyboardButton(text='EUR',
                                         url='')
        cny = types.InlineKeyboardButton(text='CNY',
                                         url='')
        usd = types.InlineKeyboardButton(text='USD',
                                         url='')
        markup.row(eur, cny)
        markup.row(usd)
        bot.send_message(message.chat.id, 'Курс какой валюты вы бы хотели посмотреть?',
                         reply_markup=markup)
    elif message.text == 'Оценить тебя':
        bot.send_message(message.chat.id, 'Отправьте свою фотографию')
        @bot.message_handler(content_types=['photo'])
        def get_photo(message):
            bot.send_message(message, 'Очень красивая фотография!')

    bot.register_next_step_handler(message, on_click)

bot.polling(non_stop=True)

Answer 1

[Vindicar]

Не надо описывать один обработчик внутри другого.
Это НИКОГДА не будет работать так, как ты этого ожидаешь.

Comments

[1LIK]

Возможно, но если напишу так:

@bot.callback_query_handler(func=lambda callback:True)
def response(function_call):
    if function_call.message:
        if function_call.data == 'tell':
            bot.send_message(function_call.message.chat.id, f'Меня зовут {name}. Я написан для того, чтобы оценивать твои фотографии и давать тебе свежий курс валют')
            bot.answer_callback_query(function_call.id)
        elif function_call.data == 'currency':
            markup = types.InlineKeyboardMarkup()
            eur = types.InlineKeyboardButton(text = 'EUR', url='')
            cny = types.InlineKeyboardButton(text = 'CNY', url='')
            usd = types.InlineKeyboardButton(text = 'USD', url='')
            markup.row(eur, cny)
            markup.row(usd)
            bot.send_message(function_call.message.chat.id, 'Курс какой валюты вы бы хотели посмотреть?', reply_markup=markup)
            bot.answer_callback_query(function_call.id)
        elif function_call.data == 'rate':
            bot.send_message(function_call.message.chat.id, 'Отправьте свою фотографию')
            @bot.message_handler(content_types=['photo'])
            def get_photo(message):
                bot.reply_to(message, 'Очень красивая фотография!')
            bot.answer_callback_query(function_call.id)

то все работает

[Vindicar]

1LIK, да? В том числе если послать несколько сообщений? В том числе с разных учёток?
В любом случае, мысль в том, что после того, как response() отработает, get_photo() останется и будет лезть в обработку фото, возможно, мешая другим частям бота. Используй FSM, опиши get_photo() отдельно, и проверяй, что пользователь выполнил нужные предыдущие шаги.

Не говоря уже о том, что для КАЖДОГО вызова response() питон будет создавать новую копию get_photo() и привязывать её к боту. Бот выберет одну из этих копий (зависит от реализации, первую или последнюю), но остальные-то никуда не денутся. Так что будет утечка памяти со временем.

[1LIK]

Попробую разобраться в том, что ты написал и применить это. Спасибо

Answer 2

[wbw1638]

Оно не отправляет оценку, потому что функцию get_photo() никто не вызывает