Почему в параметр функции идёт только последний индекс?

Question

[Dark Hacker]

Используется библиотека Pystray.

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

По неведомым причинам при нажатии на любую кнопку приходит только последний индекс.

В моём случае индексы 3-ех микрофонов - 4 5 6, и независимо от того, какую кнопку я нажимаю – выбирается индекс 6.

Код

menu_items = []
        for device in only_recording_devices:
            index = device[0][-1]
            is_default = device[1].split('Default              : ')[1]
            name = device[4].split("Name                 : ")[1]
            menu_items.append(MenuItem(f"{index} - {name}", lambda _: set_default_microphone(index),
                                       radio=True))

Общий вид менюшки

Answer 1

[Vindicar]

Поздравляю, ты попался на лямбду.
Лямбды сохраняют ссылки на переменные, а не их значения.
lambda _: set_default_microphone(index) сохранит ссылку на index, а потому все лямбды будут видеть одно и то же значение index, которое было установлено последним.
Это можно обойти, сохранив ссылку при объявлении лямбды. Самый простой способ - засунуть сохраняемое значение в значение по умолчанию для неиспользуемого параметра. Значения по умолчанию для параметров вычисляются один раз при объявлении функции, в т.ч. лямбды.
Например, так:

lambda _, *, ind=index: set_default_microphone(ind)

Ну или даже так:

lambda _, *, index=index: set_default_microphone(index)

Comments

[Dark Hacker]

Едрить коптить! Мне уже показывал один человек этот способ, но я там писал lambda x=index: func(x) и мне туда полз объект в памяти, а не цифра. Твой способ сработал. А что за звездочка?

[Vindicar]

Dark Hacker, вообще это означает "все параметры после звёздочки должны передаваться только по имени". Иными словами, если у нас есть функция def func(a, *, b=42): ..., то будут корректными следующие вызовы:

func(1)
func(a=1)
func(1, b=2)
func(a=1, b=2)

А вот вызов func(1, 2) корректным не будет, так как тут b передаётся позиционно, а не по имени.
Я звёздочку ставлю в лямбдах, чтобы уменьшить вероятность того, что лямбда будет вызвана с большим числом параметров. Ведь в этом случае вместо вспомогательного параметра index придёт чёрт знает что, и понять, что случилось, будет непросто. А так будет сообщение об ошибке, что index попытались задать позиционно, по которому можно догадаться, что случилось. Вероятность того, что лямбду по ошибке вызовут с "правильным" именным параметром всё же куда меньше.