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

Question

[Илхомжон Матазимов]

Доброго времени суток!
Я уже давно задаю этот вопрос, но так до сих пор мне никто не ответил. В моем telegram боте, цикл for перебирает из массива только последнюю строку. Использую библиотеку для telegram ботов "Aiogram".

Вот код:

keyboard = types.InlineKeyboardMarkup()
foods = ['Бургер', 'Картофель', 'Куринные ножки']
for food in foods:
    inline_btns = types.InlineKeyboardButton(food, callback_data=food)
    keyboard.add(inline_btns)

    @dp.callback_query_handler(lambda c: c.data == food) # В food попадает лишь последняя строка из массива.
    async def process_callback(call: types.CallbackQuery):
        await bot.edit_message_text(text=f"Нажата кнопка {food}", chat_id=call.message.chat.id,
                                    message_id=call.message.message_id)
await message.answer("Выберите блюдо:", reply_markup=keyboard)

И так food в цикле берет и оставляет только лишь последнюю строку из массива. То-есть, если изменить порядок строк, и на последнюю строку поставить "Картофель", то в food придет только Картофель и т.д.

Помогите, пожалуйста!!!

Comments

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

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

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

[valikhnovsky]

Алан Гибизов, о боже. ну зачем унижать человека? дайте просто ответ

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

valikhnovsky, если я не прав, это не может унизить правого, он выше этого.

Если я прав, это не может унизить неправого, ибо он и так не прав.

Ответ дать не могу, так как вопрос требует серьезно поковыряться в проекте. Я не могу просто взять предложенный код и запустить его у себя локально и отладить его. Сидеть тупить в код, догадываясь, что делают методы - ну, займитесь...

Answer 1

[kamenyuga]

Работает ровно так, как и должно. Ибо Пайтон - язык динамический. Чтобы разобраться в проблеме, надо понимать как именно работают функции и их аргументы. (1) обычные аргументы резолвятся во время вызова функции, (2) дефолтные аргументы резолвятся во время определения функции, (3) нелокальные переменные внутри функции резолвятся во время работы функции (ищутся во внешнем или глобальном пространствах имен). Не соображу, какое слово подобрать вместо "резолвятся". Вот третий вариант и работает в приведенном коде, а должен работать второй.

Следовательно, надо писать не так:
lambda c: c.data == food

а вот так:
lambda c, food=food: c.data == food

Ну, и простой пример для воспроизведения такого эффекта:

from typing import List


def f_1(lst: List) -> None:

    funcs = list()
    for val in lst:
        funcs.append(lambda: val)
    # цикл закончился, val имеет значение последнего элемента из списка

    for func in funcs:
        print(func())


def f_2(lst: List) -> None:

    funcs = list()
    for val in lst:
        funcs.append(lambda val=val: val)

    for func in funcs:
        print(func())


if __name__ == '__main__':

    data = ['apple', 'tomato', 'potato']

    print("\nf_1:")
    f_1(data)

    print("\nf_2:")
    f_2(data)

Answer 2

[alternativshik]

Да как написал, так и работает оно.

Comments

[valikhnovsky]

Автор же написал, что нужен ответ, а не унижение. Войдите в положение уже

[Дмитрий]

valikhnovsky, внимательно перечитал вопрос автора, про запрет на унижения там ни слова.
Что написал, то и получил. Так же как и с кодом

Тут довольно адекватная аудитория (если исключить мультиаккаунты по типу https://qna.habr.com/user/valikhnovsky ), и если раз за разом на один и тот же вопрос никто не может дать ответ, значит либо данных недостаточно, либо проблему нужно долго разбирать конкретно под этот случай, либо говнокод

Answer 3

[bbkmzzzz]

уберите все лишнее и увидите, что цикл перебирает все корректно. Остальное отладка. Добавляйте логи\принты и дебаг дебаг дебаг

foods = ['Бургер', 'Картофель', 'Куринные ножки']
for food in foods:
  print(food)

Comments

[Илхомжон Матазимов]

Ага, добавлять то добавляет, но

@dp.callback_query_handler(lambda c: c.data == food)

принимает только последнюю строку

[bbkmzzzz]

В моем telegram боте, цикл for перебирает из массива только последнюю строку.

цикл работает корректно. Дальше код, который обрабатывает все так как его написали. Видимо, написан не так, как предполагается. Могу предположить, что

@dp.callback_query_handler(lambda c: c.data == food) # В food попадает лишь последняя строка из массива.

перезаписывает все последним состоянием. Не в курсе, как там дела в ботописании, но объявление фукций внутри цикла - нормальная практика?