Почему Цикл неправильно завершается?

Question

[Дмитрий]

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

@dp.message_handler(content_types=["photo"])
async def handler_photo(message: types.Message, state: FSMContext):
    user_id = message.from_user.id
    username = message.from_user.username
    first_name = message.from_user.first_name
    last_name = message.from_user.last_name
    name = message.from_user.full_name
 
    async with state.proxy() as data:
        if type(data.get('photo')) is list:
            data['photo'].append(message.photo[-1].file_id)
            return
        else:
            data['photo'] = [message.photo[-1].file_id]

    for i in range(1):
        if message.photo:
            await bot.send_message(message.chat.id, f"Текст для пользователя.".format(message.from_user), parse_mode='html', disable_notification=True)
            await message.delete()
            return
    async with state.proxy() as data:
        del data["photo"]

Нужно, чтобы один раз появился текст, как сейчас и чтобы все фото удалялись, чтобы ни одна, не оставалась в группе от пользователь, при публикации до 10 фото, как галерея, с подписью или без.

Answer 1

[soremix]

Потому что есть этот кусок кода:

async with state.proxy() as data:
        if type(data.get('photo')) is list:
            data['photo'].append(message.photo[-1].file_id)
            return
        else:
            data['photo'] = [message.photo[-1].file_id]

если тип - список:
    выйти из функции
если не список:
    сделать списком

Прилетает первое фото - делаете объект списком, прилетает второе и последующие фото - выходите из функции

Ну и помимо них есть зачем-то такие куски:
for i in range(1): - просто зачем?
if message.photo: - условие не может не выполниться в контексте этого хэндлера

async with state.proxy() as data:
        del data["photo"]

- выполнение никогда не дойдет сюда

Comments

[soremix]

Дмитрий, если надо удалять медиа группу это делается в 2 пункта:

1. Проверяем, является ли сообщение медиагруппой
2. Удаляем

@dp.message_handler(content_types=["photo"])
async def handler_photo(message: types.Message, state: FSMContext):
    if message.media_group_id:
        await message.delete()

[soremix]

Дмитрий, отдельное фото - отдельный текст. отправляете текст и заносите media_group_id в БД. Делаете проверку, что если media_group_id уже есть в БД - не отправляете ничего.

[Дмитрий]

soremix, что то понял, но не пойму как это все прописать.

[Elvis]

ни чего не понимаю в пайтоне

Так может пойти одним из двух путей? Либо изучить основы, либо вообще не трогать?

[Дмитрий]

Спасибо, только пользователю soremix, а все остальные, бесполезные комментаторы, только одни вопросы задают мне, хотя помогать нужно.

[Elvis]

Дмитрий, поверь меня достали умники которые не хотят изучить основ, пишут что ничего не понимают, но требуют чтобы им написали рабочий код. Вот тот-же for i in range(1) это по вашему нормально? Это т кусок кода просто выполняется 1 раз и всё. Нафига это нужно? Не задумывались? Я так понимаю это либо с какого-то дурацкого урока, либо вам код достался по наследству, который писался по дурацким урокам.