質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
Visual Studio

Microsoft Visual StudioはMicrosoftによる統合開発環境(IDE)です。多種多様なプログラミング言語に対応しています。

Tkinter

Tkinterは、GUIツールキットである“Tk”をPythonから利用できるようにした標準ライブラリである。

コードレビュー

コードレビューは、ソフトウェア開発の一工程で、 ソースコードの検査を行い、開発工程で見過ごされた誤りを検出する事で、 ソフトウェア品質を高めるためのものです。

Python

Pythonは、コードの読みやすさが特徴的なプログラミング言語の1つです。 強い型付け、動的型付けに対応しており、後方互換性がないバージョン2系とバージョン3系が使用されています。 商用製品の開発にも無料で使用でき、OSだけでなく仮想環境にも対応。Unicodeによる文字列操作をサポートしているため、日本語処理も標準で可能です。

Q&A

解決済

1回答

6667閲覧

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

chokokorone333

総合スコア32

Visual Studio

Microsoft Visual StudioはMicrosoftによる統合開発環境(IDE)です。多種多様なプログラミング言語に対応しています。

Tkinter

Tkinterは、GUIツールキットである“Tk”をPythonから利用できるようにした標準ライブラリである。

コードレビュー

コードレビューは、ソフトウェア開発の一工程で、 ソースコードの検査を行い、開発工程で見過ごされた誤りを検出する事で、 ソフトウェア品質を高めるためのものです。

Python

Pythonは、コードの読みやすさが特徴的なプログラミング言語の1つです。 強い型付け、動的型付けに対応しており、後方互換性がないバージョン2系とバージョン3系が使用されています。 商用製品の開発にも無料で使用でき、OSだけでなく仮想環境にも対応。Unicodeによる文字列操作をサポートしているため、日本語処理も標準で可能です。

1グッド

0クリップ

投稿2020/07/11 15:08

編集2020/07/11 15:38

発生中の問題

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 # 最後に作成されたインスタンス(2つ目の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で回すのではなく、メソッドを作成するや、それぞれ単体で作ったほうがいいなどありましたらご指摘願います。

teamikl👍を押しています

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

shiracamus

2020/07/11 15:29 編集

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

2020/07/11 15:39

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

回答1

0

ベストアンサー

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()

投稿2020/07/11 17:10

編集2020/07/11 21:43
teamikl

総合スコア8664

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

chokokorone333

2020/07/14 00:08

ありがとうございます。 公式ドキュメントを一読させていただきました。 ラムダにとっての外側にあるboxのスコープの問題であり、ラムダのローカル変数として設定してあげることで解決できるということ。という理解に落ち着きました。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問