Python3Tkinter クラス化で意図した動作ができない

Question

tkinterでクラス化しているのですが、ウィンドウを閉じるときに応答なしになってしまいます。
クラスはまだ完全ではありませんが、ウィジェットとコントローラーの2種類に分けようと思っています。

過去の質問の回答を参考に作ってみたのですが、異なる動作結果になってしまいます。

該当のソースコード
（controller.pyから実行）

controller.py

from datetime import datetime
import time
import tkinter as tk
import queue
import threading

import create

def run():
    win = tk.Tk()
    app = create.Application(win)
    time_flag = True
    message_queue = queue.Queue()
    thread1 = threading.Thread(target=lambda:time_print(time_flag, message_queue))
    thread1.start()
    win.after(0, lambda:check_queue(win, app, message_queue))
    win.protocol("WM_DELETE_WINDOW", lambda:on_closing(win, time_flag, thread1))
    win.mainloop()

def time_print(time_flag, message_queue):
    while time_flag:
        now = datetime.now()
        d = "{0:%Y/%m/%d}".format(now)
        t = "{0:%H:%M:%S}".format(now)
        message_queue.put(d + t)
        time.sleep(0.01)

# Queueを確認してラベルを更新
def check_queue(win, app, message_queue):
    try:
        msg = message_queue.get(0)
        app.label["text"] = msg
    except queue.Empty:
        pass

    #5ms後に再度この関数を呼ぶ(main thread上で動作)
    win.after(5, lambda:check_queue(win, app, message_queue))

def on_closing(win, time_flag, thread1):
    time_flag = False
    thread1.join()
    win.destroy()

if __name__ == "__main__":
    run()

create.py

import tkinter as tk

class Application:
    def __init__(self, win):
        self.win = win
        self.widget_add()

    def widget_add(self):
        self.label = tk.Label(self.win, text="")
        self.label.grid()

if __name__ == "__main__":
    pass

Answer 1

def time_print(time_flag, message_queue):
    while time_flag:
        ...

def on_closing(win, time_flag, thread1):
    time_flag = False
    ...

問題点: 変数 time_flag は引数で与えられている為、
on_closing で変更した値が、time_print 側に影響しません。

グローバル変数にすることで期待通りの動作になると思います。

修正方法

1. ファイルの先頭の方で変数を宣言
2. time_print(), on_closing() の引数の time_flag を取り除く。run()での呼び出し側も。
3. on_closing() 内の先頭で変数を global 宣言

time_flag = True

:

def run():
    # snip
    thread1 = threading.Thread(target=lambda:time_print(message_queue))
    # snip
    win.protocol("WM_DELETE_WINDOW", lambda:on_closing(win, thread1))

def time_print(message_queue):
    while time_flag:
        ...

def on_closing(win, thread1):
    global time_flag  # 関数内からグローバル変数を変更する際は、宣言が必要
    time_flag = False
    ....

Comments

[person]

回答ありがとうございます。