ssshmyaks / rubtousd Goto Github PK

Описание:

В текущей реализации бота используется цикл while True, который отправляет запрос каждые 5 секунд. Это может привести к проблемам с использованием ресурсов, особенно при массовом наплыве пользователей. В случае массового использования, бот будет часто сталкиваться с ошибкой 429 ("Too Many Requests"), что может привести к нежелательным проблемам.

Предложение:

1. Оценить возможность использования асинхронных методов для отправки сообщений, чтобы уменьшить нагрузку на сервер и избежать чрезмерного использования ресурсов.
2. Реализовать механизм обработки ошибок, связанных с получением ошибки 429 ("Too Many Requests"), включая повторную попытку отправки сообщения через указанный интервал времени (параметр retry_after).
3. Предложить пользователю опцию настройки интервала между запросами, чтобы избежать возможных проблем с ограничениями на частоту запросов.

Дополнительные замечания:

При проектировании и реализации бота необходимо также учитывать возможные ограничения и правила платформы Telegram, такие как "slow_mode", который может ограничить частоту отправки сообщений в группах.

Предложение:

1. Используйте асинхронные методы aiogram для отправки сообщений, чтобы уменьшить нагрузку на сервер и избежать чрезмерного использования ресурсов.
2. Реализуйте механизм обработки ошибок, связанных с получением ошибки 429 ("Too Many Requests"), включая повторную попытку отправки сообщения через указанный интервал времени.
3. Добавьте максимальное количество комментариев в код для обеспечения понятности и поддерживаемости.
4. Обеспечьте корректную обработку всех возможных исключений при отправке сообщений и получении данных от ЦБ РФ.

Дополнительные замечания:

Пожалуйста, обратите внимание на то, что даже с обработкой ошибки 429 ("Too Many Requests"), частая отправка сообщений может привести к негативному воздействию на вашего бота и Telegram API. Рекомендую Вам установить разумные интервалы для отправки сообщений и обеспечить возможность настройки этих интервалов пользователем.

import asyncio
import requests
import aiogram
from aiogram import Bot, types
from aiogram.utils import executor
import config

bot = Bot(token=config.TOKEN)
dp = aiogram.Dispatcher(bot)

async def send_currency_rate(chat_id):
    try:
        # отправляем запрос к цб для получения текущего курса доллара
        data = requests.get('https://www.cbr-xml-daily.ru/daily_json.js').json()
        # отправляем сообщение с текущим курсом
        await bot.send_message(chat_id, data['Valute']['USD']['Value'])
    except requests.RequestException as e:
        # лбрабатываем ошибку при отправке запроса к цб
        print("Ошибка при отправке запроса к Центробанку:", e)

@dp.message_handler(commands=['start'])
async def start(message: types.Message):
    while True:
        try:
            # отправляет текущий курс
            await send_currency_rate(message.chat.id)
            # ждём 5 секк перед отправкой следующего сообщения
            await asyncio.sleep(5)
        except aiogram.utils.exceptions.RetryAfter as e:
            # ловим 429 ("Too Many Requests")
            retry_after = e.timeout
            print(f"Слишком много запросов. Повторная попытка через {retry_after} сек.")
            await asyncio.sleep(retry_after)
        except Exception as e:
            # орабатываем другие возможные ошибки
            print("Ошибка при отправке сообщения:", e)

if __name__ == '__main__':
    # Запускаем бота
    executor.start_polling(dp, skip_updates=True)