GUIでitemconfigを使って画像を切り替えたいのですが、上手く動きません

Question

前提・実現したいこと

２段階のプルダウンを作成して、選択された項目によって表示する画像を変えるGUIプログラムを作成しています。Tkinterを使用しています。画像を変えるのにitemconfigを使いました。
言語はPython3.7.3、Anaconda3上で実装しています。

発生している問題・エラーメッセージ

このプログラムでは'alphabet'の'a'を選択するとimg_1_2のファイルの１番目の画像を表示するように作成しましたが、'alphabet'の'a'を選択するとなぜかcanvasの表示が背景色の真っ黒（画像が表示されてない）になってしまいます。
エラーメッセージは一切出ていませんが上手く動作しません。
恐らく画像切り替えの部分に問題があるのだと思うのですが、どう直したら良いか分かりません。

該当のソースコード

#画像のファイルパス
img_1_2 = glob.glob("C:/Users/admin/Desktop/clothes_pic/pic/1/2/*")
##画像表示部分
#画像を表示するための準備
img = Image.open('C:/Users/admin/Desktop/clothes_pic/pic/top.bmp')
img = ImageTk.PhotoImage(img)
#画像を表示するためのキャンバスを作成(
canvas = tkinter.Canvas(bg = "black",width=400,height=800)
canvas.place(x=250,y=280)#左上の座標を指定
#キャンバスに画像を表示する。第一引数と第二引数はx,yの座標
image_on_canvas = canvas.create_image(0,0,image=img,anchor = tkinter.NW)


#プルダウンの作成    
#プルダウンの中身
lang = {'ひらがな': ('あ', 'い', 'う', 'え', 'お'),
        'カタカナ': ('ア', 'イ', 'ウ', 'エ', 'オ'),
        'alphabet': ('a', 'i', 'u', 'e', 'o')}
#一個目のプルダウン
cb1 = ttk.Combobox(window, state='readonly',width=30,height=50)
cb1['values'] = list(lang.keys())
cb1.place(x=250,y=190)
#二個目のプルダウン
cb2 = ttk.Combobox(window, state='readonly',width=30,height=50)
cb2['value'] = ''
cb2.place(x=450,y=190)

def selected(event):
    var = cb1.get()
    cb2['value'] = lang[var]
    x=var
    return var

def select_cb(event):
    global window
    
    window.after(1000,select_pic(cb2.get()))


cb1.bind('<<ComboboxSelected>>', selected)
cb2.bind('<<ComboboxSelected>>', select_cb)


#画像の切り替え部分
number=0

def select_pic(x):
    global length,filename,canvas,window
    
    filename=[]
    if x=="a":
        for name in img_1_2:
            filename.append(name)
        length=len(filename)

        #切り替えたい画像を定義
        print(filename[number])
        img2 = Image.open(filename[number])
        img2 = ImageTk.PhotoImage(img2)
        #画像のを差し替え
        canvas.itemconfig(image_on_canvas,image=img2)
        
window.mainloop()

    
   

試したこと

最初はselect_picのcanvasが反応していないのかと思い、globalでcanvasの定義をしてみたりしたのですが変わりませんでした。

補足情報（FW/ツールのバージョンなど）

Pythonは最新のバージョンです。Pythonのプログラムはあまり経験はなく、classはよくわからないのでできれば使いたくないです、、。

Answer 1

問題点: tkinterのウィジェット等はローカル変数でも問題ないのですが、
PhotoImage や StringVar 等の非ウィジェットのオブジェクトは、
ローカル変数にした場合、デストラクタにより関数の実行後に破棄されてしまいます。

# グローバル変数なので、変数の生存期間はプログラムの実行中
img = ImageTk.PhotoImage(img)

def select_pic(x):

    ...

    # 関数内のローカル変数は、select_pic() 実効終了後破棄される 
    img2 = ImageTk.PhotoImage(img2)

解決策: (どちらか)

• グローバル変数にする global img2 (select_pic 関数内で、事前に)
• 他の場所に変数を保持する。例: canvas.image = img2 (select_pic 関数内で、最後に)

---

その他の問題点

以下のコードは、意図通りの１秒後に呼び出しにはなってません
after の使い方は、指定時間後に呼び出す関数を渡しますが、
このコードでは呼び出した結果をタイマーに渡してます。

# NO
    window.after(1000,select_pic(cb2.get()))

# lambda を使う方法 (※ for文内で使う場合は注意が必要)
    window.after(1000, lambda: select_pic(cb2.get()))

# functools.partial を使う方法、推奨
    from functools import partial
    window.after(1000, partial(select_pic, cb2.get()))

Comments

[Tomo-25]

教えてくださった部分を直したらちゃんと動きました！！ありがとうございます、とても助かりました！