Tkinterにてbindしたい関数が正常動作しない原因が分からない。

発生中の問題

ttk.Comboboxを使用し、boxインスタンスに無名関数をbindさせ、選択変更があった場合にコンソールに表示する。とったコードになります。
同種のboxを使用するため、for文の中でインスタンスを生成し貼り付るといった実装方法にしました。

発生している問題点として、どこのコンボボックスでイベントが発生(選択の変更)しても最後に生成されたコンボボックスの値情報を取ってきてしまいます。

現在のコード

Python
1#coding:utf-8
2
3import tkinter as tk
4from tkinter import ttk
5
6root = tk.Tk()
7
8x = 0.1
9y = 0.2
10
11for i in range(2):
12    box = ttk.Combobox(root, values=["A","B"])
13    box.place(relx=0.1, rely=y-0.1)
14    box.current(0)
15    box.bind("<<ComboboxSelected>>", lambda e: print(box.get()))
16    # 最後に作成されたインスタンス(２つ目のbox)のbox.get()をすべてのインスタンスが所持している。
17
18    x += 0.1
19    y += 0.1
20
21root.mainloop()

試したこと

同じfor文の中で、同じインスタンス名で回しているのが原因かとおもい、
ttk.Combobox(self, state="A","B"])をforの前で違うそれぞれ違うインスタンス変数名で定義しbindしてみました。
しかし、問題解決にはいたらず、みだし現在のコードと動作は同様です。

Python
1import tkinter as tk
2from tkinter import ttk
3
4root = tk.Tk()
5
6x = 0.1
7y = 0.2
8
9box = ttk.Combobox(root, values=["A","B"])
10box2 = ttk.Combobox(root, values=["A","B"])
11
12box_array = [box, box2]
13for i in box_array:
14    i.place(relx=0.1, rely=y-0.1)
15    i.current(0)
16    i.bind("<<ComboboxSelected>>", lambda e: print(i.get()))
17
18    x += 0.1
19    y += 0.1
20
21root.mainloop()
22

動作環境

windows10
Python 3.8.3
WSL2 ubuntu

お聞きしたいこと

試したことにあります、同じfor文の中で、同じインスタンス名で回しているのが原因かとおもった。が原因ではないためお手上げ状態です。
原因を教えていただけるとありがたいです。

余談にはなりますが、
同種のウィジェットを生成したいからと、for文の中でインスタンスを生成し貼り付ける。といった実装方法は綺麗な実装方法と言えるのでしょうか。
forで回すのではなく、メソッドを作成するや、それぞれ単体で作ったほうがいいなどありましたらご指摘願います。

shiracamus

2020/07/11 15:29 編集

変数 self には何が代入されているのですか？単体で動作確認できる簡略化したソースコードを示していただければ、回答が付きやすくなると思います。

chokokorone333

2020/07/11 15:39

ご指摘ありがとうございます。簡易的に動作するコードを実装いたしました。

行動規範の内容に同意します

回答1件

ベストアンサー

Python FAQ: ループの中で異なる値で定義されたラムダ式が、同じ値を返すのはなぜですか？

ループ内でlambdaを使う場合の注意点です。

ループ変数は、 bind した時点ではなくlambdaで生成した関数が呼び出される時点の値なので、
ループが終了した後では、全てが繰り返しの最後の要素になってしまいます。

一時的な回避策:

python
1box.bind("<<ComboboxSelected>>", (lambda e, box=box: print(box.get())))

外の変数(ループの繰り返し毎に変更される)のその時点の値を、ラムダの引数に代入しておく。

python
1box.bind("<<ComboboxSelected>>", lambda e: print(e.widget.get()))

lambda式内から外の変数を参照しておらず、イベントに渡された引数から参照する。

と、工夫次第でlambdaも使えますが、そもそも「ループ内での lambda」自体を避けた方が良いです。

同種のウィジェットを生成したいからと、for文の中でインスタンスを生成し貼り付ける。

といった実装方法は綺麗な実装方法と言えるのでしょうか。

問題ありません。寧ろ自然な実装だと思いますが、注意が必要な点はあります。

forで回すのではなく、メソッドを作成するや、それぞれ単体で作ったほうがいいなどありましたらご指摘願います

ここでの注意点は、「ループ内でのlambdaの利用」です。

イベントで呼び出す処理は、直接式を書かずに関数やメソッドとして実装した方が良いでしょう。
関数に他のパラメータを渡す場合は、functools.partial も併せて調べて見て下さい。

python
1def selected(e):
2    print(e.widget.get())
3
4box.bind("<<ComboboxSelected>>", selected)

他の引数を渡したい場合

python
1import tkinter as tk
2from tkinter import ttk
3from functools import partial
4
5def selected(idx, e):
6    print(idx, e.widget.get())
7
8root = tk.Tk()
9
10for idx in range(2):
11    box = ttk.Combobox(root, values=["A", "B"])
12    box.bind("<<ComboboxSelected>>", partial(selected, idx))
13    box.pack()
14
15    ## lambda で書く場合、複雑になりがちでデバッグに余計な手間がかかります
16    # box.bind("<<ComboboxSelected>>", (lambda e, idx=idx: selected(e, idx)))
17
18root.mainloop()