tag_bindで関数内関数を呼び出した後、外側の関数が機能しなくなってしまう。

Question

前提

pythonのtkinterおよびpillowを用いて、クリックにより画像を回転させるコードを作成しています。
画面をクリックすることで画像が無限に左回転を続け、再度クリックをすると画面の回転がストップします。

実現したいこと

2度目のクリックで画像の回転を止めた後、3度目のクリックでは画像の回転を再開させたいと考えています。
(奇数回目のクリックで画像を回転させ、偶数回目のクリックでは回転をストップさせたい。)

しかし、現状のコードでは、3度目のクリックを行っても何も動作しません。
エラーメッセージはなく、他の方法が分からないためアドバイスをいただけないでしょうか。

該当のソースコード

#---測定コード---

import tkinter as tk
from PIL import Image, ImageTk


#画面サイズの指定
WIDTH = 1300
HEIGHT = 800


#新規ウィンドウを作成
root = tk.Tk()
root.title("SVV測定")
root.attributes("-fullscreen", True)


#ウィンドウ上にフレームを作成
frame = tk.Frame(root)
frame.pack()


#フレーム上に、キャンバスを作成
canvas = tk.Canvas(frame, width = WIDTH, height = HEIGHT, bg = "black")
canvas.pack()


#画像のプリセット角を"cs"として定義
cs = 0


#キャンバスにSVV画像を貼り付け
img = Image.open("D:\EXPERIMENT\programming\Python\SVV\svv_bar.png")
img = img.rotate(cs)
tkimg = ImageTk.PhotoImage(img, width = WIDTH, height = HEIGHT)
canvas.create_image(WIDTH/2, HEIGHT/2, image=tkimg, anchor="center", tags="img")


#キャンバスに、透明な画像を貼り付け
img_snow = Image.open("D:\EXPERIMENT\programming\Python\SVV\svv_snow.png")
tkimg_snow = ImageTk.PhotoImage(img_snow, width = WIDTH, height = HEIGHT)
canvas.create_image(WIDTH/2, HEIGHT/2, image=tkimg_snow, anchor="center", tags="img_snow")


#"c"の初期値を定義
c = 0


#イベントが発生したときの処理
def press_L(event):
 #左クリックの回数によって、左回転させる
 global img
 global tkimg
 global c
 global id
 c = c + 0.1
 IMG = img.rotate(c)
 tkimg = ImageTk.PhotoImage(IMG)
 canvas.create_image(WIDTH/2, HEIGHT/2, image = tkimg, anchor = "center", tags = "img")
 canvas.create_image(WIDTH/2, HEIGHT/2, image=tkimg_snow, anchor = "center", tags="img_snow")
 id = root.after(1, press_L, None)
 def stop_L(event):
  #左クリックで、左回転をストップする
  root.after_cancel(id)
 canvas.tag_bind("img_snow", "<ButtonPress-1>", stop_L)


def press_R(event):
 #右クリックの回数によって、右回転させる
 global img
 global tkimg
 global c
 global id
 c = c - 0.1
 IMG = img.rotate(c)
 tkimg = ImageTk.PhotoImage(IMG)
 canvas.create_image(WIDTH/2, HEIGHT/2, image = tkimg, anchor = "center", tags = "img")
 canvas.create_image(WIDTH/2, HEIGHT/2, image=tkimg_snow, anchor = "center", tags="img_snow")
 id = root.after(1, press_R, None)
 def stop_R(event):
  #右クリックで、右回転をストップする
  root.after_cancel(id)
 canvas.tag_bind("img_snow", "<ButtonPress-3>", stop_R)


def finish(event):
 #Enterキーを押すことで全画面表示を終了し、回転角度を取得する
 root.destroy()
 print(c)


#左クリックで呼び出すイベント
canvas.tag_bind("img_snow", "<ButtonPress-1>", press_L)


#右クリックで呼び出すイベント
canvas.tag_bind("img_snow", "<ButtonPress-3>", press_R)


#Enterキーで呼び出すイベント
root.bind("<Return>", finish)


#メインループ
root.mainloop()

上記の処理に使用した画像を作成するコード

#---SVV測定に使用する画像の作成---

from PIL import Image, ImageDraw

#背景色が黒色のキャンパスを作成。
im = Image.new('RGB', (500, 500), (0, 0, 0))
draw = ImageDraw.Draw(im)


#中央に線を描画
draw.line((50, 250, 450, 250), fill=(255, 255, 255), width=10)


#同じサイズの画像を作成
im_clear = Image.new("RGBA", im.size, (0, 0, 0, 0))
width = im.size[0]
height = im.size[1]


#色ごとに処理
for x in range(width):
    for y in range(height):
        pixel = im.getpixel( (x, y) )
        
        # 黒なら処理しない
        if pixel[0] == 0 and pixel[1] == 0 and pixel[2] == 0:
            continue
        
        # 黒以外なら、用意した画像にピクセルを書き込み
        im_clear.putpixel( (x, y), pixel )


#SVVで表示させるバーをpng形式で保存
im_clear.save("D:\EXPERIMENT\programming\Python\SVV\svv_bar.png", quality=100)



#---透明なpng画像の作成---

#画像サイズの指定
WIDTH = 1500
HEIGHT = 1000


#新しい画像を作成
im_snow = Image.new("RGBA", (WIDTH, HEIGHT), (0, 0, 0, 0))


#新しい画像をもう一つ作成
im_snow2 = Image.new("RGBA", (WIDTH, HEIGHT), (0, 0, 0, 0))


#色ごとに処理
for x in range(width):
    for y in range(height):
        pixel = im_snow2.getpixel( (x, y) )
        
        # 黒なら処理しない
        if pixel[0] == 0 and pixel[1] == 0 and pixel[2] == 0:
            continue
        
        # 黒以外なら、用意した画像にピクセルを書き込み
        im_snow.putpixel( (x, y), pixel )


#透明な画像をpng形式で保存
im_snow.save("D:\EXPERIMENT\programming\Python\SVV\svv_snow.png", quality=100)

試したこと

現在、"press_L"というイベントの中に"stop_L"というイベントを設置しています。
"stop_L"を呼び出すtag_bindを関数外に記述したところ、画像の回転を止めることができませんでした。
むしろ、クリックするごとに回転のスピードが上昇する結果に。。

また、ボタンウィジェットを用いて回転をストップさせる方法を試しましたが、
ボタンウィジェットを透明にすることができなかったため、ボタンは使用していません。

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

AnacondaでPythonを使用しています。

・Anacondaのバージョン：conda 4.12.0
・Pythonのバージョン：Python 3.9.12
・Windowsのバージョン：version 21H2

Answer 1

状況やイベント発生で、<ButtonPress-1>に何が設定されているか考えればわかると思います。

起動後: <ButtonPress-1>にpress_L が設定されている

左クリック: press_L が実行される
-> root.after(1, press_L, None) 設定 > 回転開始
-> <ButtonPress-1>にstop_Ｌ が設定される。 press_L は消えてしまう。

さらに左クリック: stop_L が実行される。
-> stop_Lが実行される。 回転が止まる。

これだけです。

現象だけみれば、問題の原因は、<ButtonPress-1>にstop_Lがbindされ続けているためです。

対応としては、状況によッて<ButtonPress-1>のときに起きる処理を切り替えることのはずなので、

1. ボタンを押したら、バインドされている関数をちゃんと切り替える。 stop_Lを押して止めたら、<ButtonPress-1>にpress_L を再設定する。
2. 関数は1つだけ用意して、その関数で今の状況(回転中かどうか)を確認して、適切な処理を行なうようにする。

のいずれかでしょう。 右回転中に左クリックしたときとかあるので、結果として2がいいんじゃないかな。

あと、id がglobalだけど、ＬとR で違うオブジェクトにアサインしているので、よくないでしょう。

Comments

[ppap]

ありがとうございます！
作成したコードだと途中でpress_Lが消えてしまっているんですね。。

2通りの解決方法アイデアに加え、"id"に関する助言までいただき、ありがとうございます。