Используя библиотеку PyTelegrammBotApi столкнулся с проблемой бесконечного вывода сообщений из текстового файла, как это можно исправить?

Question

[D3dyvka]

@bot.message_handler(content_types=['int', 'text'])
def work(message):
    if message.text == '1 курс':
        for y in x:
            if y.replace("\n", "") == time.strftime("%d %b", time.localtime(message.date)):
                for y1 in x:
                    if y1 != '_\n':
                        bot.send_message(message.chat.id, y1)
                        time.sleep(1)
                    elif y1 == '_\n':
                        bot.stop_polling()

нужно сделать так, чтобы сам бот не прекращал свою работу, а выводим текст переставал появляться

Comments

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

Это можно исправить, если разделить функцию на отдельные маленькие функции, каждая должна делать только одно действие.
Основная должна вызывать остальные, передавая им параметры и принимая от них результат, если надо.
Тогда можно отладить каждую по-отдельности и понять, где проблема.
Разбираться в этой лапше неохота.

[teenager_python]

Проблема, которую вы описали, возникает из-за того, что цикл for в вашей функции work() не имеет условия выхода и продолжает выводить сообщения из файла, пока он не будет полностью прочитан.

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

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

@bot.message_handler(content_types=['int', 'text'])
def work(message):
    if message.text == '1 курс':
        message_count = 0
        for y in x:
            if y.replace("\n", "") == time.strftime("%d %b", time.localtime(message.date)):
                for y1 in x:
                    if y1 != '_\n':
                        bot.send_message(message.chat.id, y1)
                        time.sleep(1)
                        message_count += 1
                        if message_count >= max_message_count:  # условие выхода из цикла
                            break
                    elif y1 == '_\n':
                        bot.send_message(message.chat.id, "Все сообщения отправлены")  # сообщение о завершении
                        return  # выход из функции

Здесь max_message_count - это максимальное количество сообщений, которые вы хотите отправить. Когда количество отправленных сообщений достигает этого значения, цикл for прерывается и функция work() завершается.

А вообще лучше как вам сказали выше, разбить все на маленькие функции, ибо в дальнейшем это принесет вам намного больше пользы

[D3dyvka]

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