Aiogram Message is not modified как исправить?

Question

[temjiu]

Пишу простого бота для телеграмма, выходит ошибка:

RuntimeWarning: Enable tracemalloc to get the object allocation traceback
Task exception was never retrieved
future: <Task finished name='Task-9' coro=<Dispatcher._process_polling_updates() done, defined at C:\Users\achub\AppData\Local\Programs\Python\Python39\lib\site-packages\aiogram\dispatcher\dispatcher.py:407> exception=MessageNotModi
fied('Message is not modified: specified new message content and reply markup are exactly the same as a current content and reply markup of the message')>
Traceback (most recent call last):
  File "C:\Users\achub\AppData\Local\Programs\Python\Python39\lib\site-packages\aiogram\dispatcher\dispatcher.py", line 415, in _process_polling_updates
    for responses in itertools.chain.from_iterable(await self.process_updates(updates, fast)):
  File "C:\Users\achub\AppData\Local\Programs\Python\Python39\lib\site-packages\aiogram\dispatcher\dispatcher.py", line 235, in process_updates
    return await asyncio.gather(*tasks)
  File "C:\Users\achub\AppData\Local\Programs\Python\Python39\lib\site-packages\aiogram\dispatcher\handler.py", line 117, in notify
    response = await handler_obj.handler(*args, **partial_data)
  File "C:\Users\achub\AppData\Local\Programs\Python\Python39\lib\site-packages\aiogram\dispatcher\dispatcher.py", line 256, in process_update
    return await self.message_handlers.notify(update.message)
  File "C:\Users\achub\AppData\Local\Programs\Python\Python39\lib\site-packages\aiogram\dispatcher\handler.py", line 117, in notify
    response = await handler_obj.handler(*args, **partial_data)
  File "C:\Users\achub\OneDrive\Рабочий стол\Python\telegram-bot\main.py", line 35, in start_handler
    await msg.edit_text(f'{name}: <b>{price}</b> ' + '\n' + f'Кол-во обновлений было: {r+1}')
  File "C:\Users\achub\AppData\Local\Programs\Python\Python39\lib\site-packages\aiogram\types\message.py", line 2892, in edit_text
    return await self.bot.edit_message_text(
  File "C:\Users\achub\AppData\Local\Programs\Python\Python39\lib\site-packages\aiogram\bot\bot.py", line 2990, in edit_message_text
    result = await self.request(api.Methods.EDIT_MESSAGE_TEXT, payload)
  File "C:\Users\achub\AppData\Local\Programs\Python\Python39\lib\site-packages\aiogram\bot\base.py", line 236, in request
    return await api.make_request(await self.get_session(), self.server, self.__token, method, data, files,
  File "C:\Users\achub\AppData\Local\Programs\Python\Python39\lib\site-packages\aiogram\bot\api.py", line 140, in make_request
    return check_result(method, response.content_type, response.status, await response.text())
  File "C:\Users\achub\AppData\Local\Programs\Python\Python39\lib\site-packages\aiogram\bot\api.py", line 115, in check_result
    exceptions.BadRequest.detect(description)
  File "C:\Users\achub\AppData\Local\Programs\Python\Python39\lib\site-packages\aiogram\utils\exceptions.py", line 140, in detect
    raise err(cls.text or description)
aiogram.utils.exceptions.MessageNotModified: Message is not modified: specified new message content and reply markup are exactly the same as a current content and reply markup of the message

Код бота:

import fake_useragent
from aiogram import Bot, Dispatcher, executor, types
import asyncio
import requests
from bs4 import BeautifulSoup as bs
import time

TOKEN = '5856432472:AAHbqiT-jVHjNarbFHFlsmuqxCP0KINJvCc'

bot = Bot(token=TOKEN, parse_mode="HTML")
dp = Dispatcher(bot=bot)

user = fake_useragent.UserAgent().random
headers = {'useragent': user}

url = 'https://coinmarketcap.com/currencies/bitcoin/'
response = requests.get(url, headers=headers)
soup = bs(response.text, 'lxml')
print(response.status_code)
name = soup.find('h1', class_='priceHeading').text
price = soup.find('div', class_='priceValue').text.replace('$', '').replace(',', '')

print(name + ': ' + price)

i = 0

@dp.message_handler(commands=['start'])
async def start_handler(message: types.Message):
    user_name = message.from_user.first_name
    await bot.send_message(message.from_user.id, f'Привет, <b>{user_name}</b>')
    msg = await bot.send_message(message.from_user.id, f'{name}: <b>{float(price)}</b>')

    while True:
        await msg.edit_text(f'{name}: <b>{price}</b> ' + '\n' + f'Кол-во обновлений было: {i+1}')
        asyncio.sleep(15)


if __name__ == '__main__':
    executor.start_polling(dp, skip_updates=True)

Comments

[Aragorn]

В трейсбэке всё написано

[temjiu]

?

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

temjiu, вот
Message is not modified: specified new message content and reply markup are exactly the same as a current content and reply markup of the message

[Aragorn]

Кстати, показывать свой токен - это глупо

[temjiu]

Lord_of_Rings, я сменил его

[temjiu]

Константин Нагибович, по подробнее, я понял что ''указанное новое содержимое сообщения и разметка ответа точно такие же, как текущее содержимое и разметка ответа сообщения'', а что сделать нужно

Answer 1

[Сергей П]

Вы пытаетесь в цикле поменять текст сообщения на тот же самый текст.
Протокол телеграмма не позволяет этого делать, потому что это бессмысленно.
Вы можете:
1) помнить предыдущий текст и сравнивать новый с ним, чтобы вызыватьредактирование только если текст изменился.
2) Перехватывать и игнорировать MessageNotModified, если считаете, что эта ситуация будет редкой и лишний трафик на сервера телеги вас не беспокоят.
3) Вы можете чисто логически не редактировать сообение, если исходные параметры его не поменялись.
4) вы можете гарантированно менять исходные параметры сообщения, чтобы оно гарантированно изменило текст.

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

Comments

[temjiu]

Если рассматривать 1 пункт, как можно реализовать сохранение старой цены и сравнивать ее с новой, мне на ум приходит только БД, но если долго пользоваться ботом БД будет очень емкой и будет много весить, есть еще способы какие-то? Я просто новичок-новичок

[Сергей П]

temjiu, видно что новичок. Просто перехватывайте и игнорируйте ошибку для начала.

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

Интереснее будет, если вам надо, чтобы при перезапуске бот не создавал новое сообщение, а редактировал одно из последних. Тогда первое значение можно вычитать из сообщение, или первое редактирование сделать в любом случае, но проигнорировав ошибку. При следующих редактированиях лучше всё же до ошибки не доводить. То есть не делать попытку редактирования. если текст не изменился. Это сэкономит вызовы к АПИ телеги и повысит производительность кода немного.

Не знаю поймёте ли что я тут написал, но попробуйте.

[temjiu]

Сергей Паньков, спасибочки, конечно мало что понял, но идейка есть одна что то типо того:

price = soup.find('div', class_='priceValue').text.replace('$', '').replace(',', '')

@dp.message_handler(commands=['start'])
async def start_handler(message: types.Message):
    user_name = message.from_user.first_name
    await bot.send_message(message.from_user.id, f'Привет, <b>{user_name}</b>')
    msg = await bot.send_message(message.from_user.id, f'{name}: <b>{float(price)}</b>')

I=0

    while True:
        oldprice = soup.find('div', class_='priceValue').text.replace('$', '').replace(',', '')
        asincio.sleep(15)
        If oldprice == price:
               print(“изменений нет”)
        else:
               await msg.edit_text(f'{name}: <b>{price}</b> ' + '\n' + f'Кол-во обновлений было: {i+1}')

[Сергей П]

temjiu, кошмар. в отступах ошибка, в синтаксисе ошибка, старую цену зачем-то по новой парсите, если можно было сохранить в переменной. Да и целиком сообщение можно сохранить. В вашем случае price вообще не может поменяться за время работы цикла. Короче, всё не правильно.

[temjiu]

Сергей Паньков, я с телефона писал, нету щас доступа к компьютеру

[temjiu]

name = soup.find('h1', class_='priceHeading').text
price = soup.find('div', class_='priceValue').text.replace('$', '').replace(',', '')

print(name + ': ' + price)

@dp.message_handler(commands=['start'])
async def start_handler(message: types.Message):
    user_name = message.from_user.first_name
    await bot.send_message(message.from_user.id, f'Привет, <b>{user_name}</b>')
    msg = await bot.send_message(message.from_user.id, f'{name}: <b>{float(price)}</b>')

    i = 0
    a = f'{float(price)}'
    while True:
        if a != float(price):
            asyncio.sleep(15)
            await msg.edit_text(f'{name}: <b>{price}</b> ' + '\n' + f'Кол-во обновлений было: {i+1}')
        else:
            print('Изменений нет')

Так лучшее? ну он всеже выдает ошибку:

C:\Users\achub\OneDrive\Рабочий стол\Python\telegram-bot\main.py:35: RuntimeWarning: coroutine 'sleep' was never awaited
asyncio.sleep(15)
RuntimeWarning: Enable tracemalloc to get the object allocation traceback
Task exception was never retrieved
future: exception=MessageNotModi
fied('Message is not modified: specified new message content and reply markup are exactly the same as a current content and reply markup of the message')>
Traceback (most recent call last):
File "C:\Users\achub\AppData\Local\Programs\Python\Python39\lib\site-packages\aiogram\dispatcher\dispatcher.py", line 415, in _process_polling_updates
for responses in itertools.chain.from_iterable(await self.process_updates(updates, fast)):
File "C:\Users\achub\AppData\Local\Programs\Python\Python39\lib\site-packages\aiogram\dispatcher\dispatcher.py", line 235, in process_updates
return await asyncio.gather(*tasks)
File "C:\Users\achub\AppData\Local\Programs\Python\Python39\lib\site-packages\aiogram\dispatcher\handler.py", line 117, in notify
response = await handler_obj.handler(*args, **partial_data)
File "C:\Users\achub\AppData\Local\Programs\Python\Python39\lib\site-packages\aiogram\dispatcher\dispatcher.py", line 256, in process_update
return await self.message_handlers.notify(update.message)
File "C:\Users\achub\AppData\Local\Programs\Python\Python39\lib\site-packages\aiogram\dispatcher\handler.py", line 117, in notify
response = await handler_obj.handler(*args, **partial_data)
File "C:\Users\achub\OneDrive\Рабочий стол\Python\telegram-bot\main.py", line 36, in start_handler
await msg.edit_text(f'{name}: {price} ' + '\n' + f'Кол-во обновлений было: {i+1}')
File "C:\Users\achub\AppData\Local\Programs\Python\Python39\lib\site-packages\aiogram\types\message.py", line 2892, in edit_text
return await self.bot.edit_message_text(
File "C:\Users\achub\AppData\Local\Programs\Python\Python39\lib\site-packages\aiogram\bot\bot.py", line 2990, in edit_message_text
result = await self.request(api.Methods.EDIT_MESSAGE_TEXT, payload)
File "C:\Users\achub\AppData\Local\Programs\Python\Python39\lib\site-packages\aiogram\bot\base.py", line 236, in request
return await api.make_request(await self.get_session(), self.server, self.__token, method, data, files,
File "C:\Users\achub\AppData\Local\Programs\Python\Python39\lib\site-packages\aiogram\bot\api.py", line 140, in make_request
return check_result(method, response.content_type, response.status, await response.text())
File "C:\Users\achub\AppData\Local\Programs\Python\Python39\lib\site-packages\aiogram\bot\api.py", line 115, in check_result
exceptions.BadRequest.detect(description)
File "C:\Users\achub\AppData\Local\Programs\Python\Python39\lib\site-packages\aiogram\utils\exceptions.py", line 140, in detect
raise err(cls.text or description)
aiogram.utils.exceptions.MessageNotModified: Message is not modified: specified new message content and reply markup are exactly the same as a current content and reply markup of the message

[Сергей П]

В чем прикол? Вообще не думая пишешь, или правда не понимаешь?
price никогда не меняется. сравнение грешного с праведным (строки с числом).
Говорят если бесконечное число обезьян будут тарабанить по клавиатуре, то однажды у них получится написать Войну и Мир, но это не тот случай.
Вы просто не читаете что пишете и не думаете над написанным.

[temjiu]

Сергей Паньков, мне price в цикл засунуть?

[temjiu]

Сергей Паньков, Можете написать этот код примерно хотяб, мне очень интересно как это выглядеть будет

[Сергей П]

temjiu, нет, этого будет недостаточно. Этот ресурс не для фриланса. У вас явно недостаточно знаний, чтобы решать такие задачи. Закажите код на фрилансе или наймите репетитора. Ответ на ваш вопрос не даст вам никакого понимания как делать такие вещи. Сперва надо учиться программировать.

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

Answer 2

[Aragorn]

await msg.edit_text(f'{name}: <b>{price}</b> ' + '\n' + f'Кол-во обновлений было: {i+1}')

Вы редактируете сообщение, но текст при этом не изменяется, так как i у Вас всегда остаётся 1

Comments

[temjiu]

Как мне текст поменять

[Aragorn]

temjiu,

while True:
    i = i + 1
    await msg.edit_text(f'{name}: <b>{price}</b> ' + '\n' + f'Кол-во обновлений было: {i}')
    asyncio.sleep(15)

[temjiu]

Task exception was never retrieved
future: exception=UnboundLocalEr
ror("local variable 'i' referenced before assignment")>
Traceback (most recent call last):
File "C:\Users\achub\AppData\Local\Programs\Python\Python39\lib\site-packages\aiogram\dispatcher\dispatcher.py", line 415, in _process_polling_updates
for responses in itertools.chain.from_iterable(await self.process_updates(updates, fast)):
File "C:\Users\achub\AppData\Local\Programs\Python\Python39\lib\site-packages\aiogram\dispatcher\dispatcher.py", line 235, in process_updates
return await asyncio.gather(*tasks)
File "C:\Users\achub\AppData\Local\Programs\Python\Python39\lib\site-packages\aiogram\dispatcher\handler.py", line 117, in notify
response = await handler_obj.handler(*args, **partial_data)
File "C:\Users\achub\AppData\Local\Programs\Python\Python39\lib\site-packages\aiogram\dispatcher\dispatcher.py", line 256, in process_update
return await self.message_handlers.notify(update.message)
File "C:\Users\achub\AppData\Local\Programs\Python\Python39\lib\site-packages\aiogram\dispatcher\handler.py", line 117, in notify
response = await handler_obj.handler(*args, **partial_data)
File "C:\Users\achub\OneDrive\Рабочий стол\Python\telegram-bot\main.py", line 33, in start_handler
i = i + 1
UnboundLocalError: local variable 'i' referenced before assignmen

[temjiu]

Если

while True:
        i = 0
        i = i + 1
        await msg.edit_text(f'{name}: <b>{price}</b> ' + '\n' + f'Кол-во обновлений было: {i}')
        asyncio.sleep(15)

то:

C:\Users\achub\OneDrive\Рабочий стол\Python\telegram-bot\main.py:36: RuntimeWarning: coroutine 'sleep' was never awaited
asyncio.sleep(15)
RuntimeWarning: Enable tracemalloc to get the object allocation traceback
Task exception was never retrieved
future: exception=MessageNotModi
fied('Message is not modified: specified new message content and reply markup are exactly the same as a current content and reply markup of the message')>
Traceback (most recent call last):
File "C:\Users\achub\AppData\Local\Programs\Python\Python39\lib\site-packages\aiogram\dispatcher\dispatcher.py", line 415, in _process_polling_updates
for responses in itertools.chain.from_iterable(await self.process_updates(updates, fast)):
File "C:\Users\achub\AppData\Local\Programs\Python\Python39\lib\site-packages\aiogram\dispatcher\dispatcher.py", line 235, in process_updates
return await asyncio.gather(*tasks)
File "C:\Users\achub\AppData\Local\Programs\Python\Python39\lib\site-packages\aiogram\dispatcher\handler.py", line 117, in notify
response = await handler_obj.handler(*args, **partial_data)
File "C:\Users\achub\AppData\Local\Programs\Python\Python39\lib\site-packages\aiogram\dispatcher\dispatcher.py", line 256, in process_update
return await self.message_handlers.notify(update.message)
File "C:\Users\achub\AppData\Local\Programs\Python\Python39\lib\site-packages\aiogram\dispatcher\handler.py", line 117, in notify
response = await handler_obj.handler(*args, **partial_data)
File "C:\Users\achub\OneDrive\Рабочий стол\Python\telegram-bot\main.py", line 35, in start_handler
await msg.edit_text(f'{name}: {price} ' + '\n' + f'Кол-во обновлений было: {i}')
File "C:\Users\achub\AppData\Local\Programs\Python\Python39\lib\site-packages\aiogram\types\message.py", line 2892, in edit_text
return await self.bot.edit_message_text(
File "C:\Users\achub\AppData\Local\Programs\Python\Python39\lib\site-packages\aiogram\bot\bot.py", line 2990, in edit_message_text
result = await self.request(api.Methods.EDIT_MESSAGE_TEXT, payload)
File "C:\Users\achub\AppData\Local\Programs\Python\Python39\lib\site-packages\aiogram\bot\base.py", line 236, in request
return await api.make_request(await self.get_session(), self.server, self.__token, method, data, files,
File "C:\Users\achub\AppData\Local\Programs\Python\Python39\lib\site-packages\aiogram\bot\api.py", line 140, in make_request
return check_result(method, response.content_type, response.status, await response.text())
File "C:\Users\achub\AppData\Local\Programs\Python\Python39\lib\site-packages\aiogram\bot\api.py", line 115, in check_result
exceptions.BadRequest.detect(description)
File "C:\Users\achub\AppData\Local\Programs\Python\Python39\lib\site-packages\aiogram\utils\exceptions.py", line 140, in detect
raise err(cls.text or description)
aiogram.utils.exceptions.MessageNotModified: Message is not modified: specified new message content and reply markup are exactly the same as a current content and reply markup of the message

[Модератор]

Lord_of_Rings, вы постоянно предлагаете оставить тэг Python и удаляете тэг Telegram.
Не вижу обоснования для этого. Тэги стоят правильно, а вы их зачем-то меняете.
Не надо так, это совершенно не нужная излишняя нагрузка на модераторов.

[Aragorn]

Модератор, понял

[Модератор]

Lord_of_Rings, в действительности, есть случаи, когда убрать тэг бота - обоснованно. К примеру, когда ошибка вообще не имеет отношения к ботам, ну тупо кавычку забыл или всё такое.

Но ошибки, связанные со структурой объектов бота, либо связанные с неверным использованием API - лучше оставлять оба тэга. А это надо всякий раз глубоко вникать… не всегда очевидно. И, поскольку тут бывают разные случаи и порой спорные - пусть остаются оба тэга, если вы не абсолютно уверены.