Бот должен при команде /photo сохранять присланное после команды фото в одну папку, а при /photo1 в другую?

Question

[KhinevichVladimir]

Вот что я написал:

import telebot
from telebot import types                                               # для указание типов
import config
bot = telebot.TeleBot('5968574660:AAFx_YTvm31qsHI9lLSst3OQJozm0KXQ3XU')
@bot.message_handler(commands=['photo'])
def start(message):
    bot.send_message(message.chat.id, "отправьте фото")
    @bot.message_handler(content_types=['photo'])
    def photo(message):   
        bot.send_message(message.chat.id, "0")
        fileID = message.photo[-1].file_id   
        file_info = bot.get_file(fileID)
        downloaded_file = bot.download_file(file_info.file_path)
        with open("E:/python/img0.jpg", 'wb') as new_file:
            new_file.write(downloaded_file)
            bot.send_message(message.chat.id, file_info)

@bot.message_handler(commands=['photo1'])
def start(message):
    bot.send_message(message.chat.id, "отправьте фото")
    @bot.message_handler(content_types=['photo'])
    def photo2(message):   
        bot.send_message(message.chat.id, "1")
        fileID = message.photo[-1].file_id   
        file_info = bot.get_file(fileID)
        downloaded_file = bot.download_file(file_info.file_path)
        with open("E:/python/img1.jpg", 'wb') as new_file:
            new_file.write(downloaded_file)
            bot.send_message(message.chat.id, file_info)

bot.polling(none_stop=True)

Comments

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

Доброго времени суток. Пайтон бот должен при команде /photo сохранять присланное после команды фото в одну папку, а при /photo1 в другую?

Нет, не должен.

[Алан Гибизов]

Бот должен при команде /photo сохранять присланное после команды фото в одну папку, а при /photo1 в другую?

Да, должен.

[EugeneVKruglov]

А почему хэндлеры находятся один внутри другого?

[Сергей П]

Доброго времени суток. Пайтон бот должен при команде /photo сохранять присланное после команды фото в одну папку, а при /photo1 в другую?

Тут не всё так однозначно, коллеги...

[Константин Нагибович]

EugeneVKruglov, потому что кто-то решил делать ботов раньше чем выучил язык.

Answer 1

[shurshur]

Не надо внутри одной функции объявлять другую. Это не будет работать, вообще никак! Результатом будет дописывание в список хендлеров новых обработчиков, а так как условия в них уже описаны в других обработчиках, то эти никогда вызваны не будут.

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

В данном случае для решения задачи нужно использовать FSM. В telebot есть очень простая в использовании реализация FSM на основе вызовов register_next_step_handler пример.

Comments

[KhinevichVladimir]

Вы не могли бы прислать решение? Я не понял.

[shurshur]

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

Я дал ссылку на официальный пример.

[o5a]

KhinevichVladimir, почти то же самое, просто считайте это как следующий шаг. При срабатывании хэндлера и функции start вы указываете на следующий шаг - функцию photo, которая уже сохранит эту картинку.

@bot.message_handler(commands=['photo'])
def start(message):
    bot.send_message(message.chat.id, "отправьте фото")
    register_next_step_handler(message, НАЗВАНИЕ_ФУНКЦИИ)

Аналогично для второго хэндлера.
Можете даже на этом сайте поискать по "register_next_step_handler", много примеров было.

[KhinevichVladimir]

shurshur.Спасибо за помощь.

[KhinevichVladimir]

o5a, спасибо, теперь понял. Все работает

[shurshur]

KhinevichVladimir, FSM - это конечный автомат, система из состояний и переходов. Бот находится в базовом состоянии и ожидает команды, когда получает команду от пользователя - меняет состояние, благодаря чему следующим обработчиком выбирается конкретный. При этом в telebot понятие состояния ярко не выражено, а вот в других библиотеках (aiogram) состояния описываются более явно (обработчику можно задавать фильтр по состоянию, а вместо вызовов register_next_step_handler с указанием конкретного обработчика делается установка нового состояния).

Например, если мы спрашиваем у пользователя его возраст, то хотим, чтобы его ответ обработал не стандартный обработчик входящих текстовых сообщений, а какой-то отдельный, но только один раз и только для этого пользователя. В официальном примере как раз среди вопросов запрашивается возраст. Таким образом, можно делать весьма разветвлённые системы вопросов-ответов.

В случае с описанной задачей с фото обработчик должен по-разному отреагировать на один и тот же вид контента (фото) в зависимости от предыдущего действия.