tkinterをクラスで表現したい

Question

前提

tkinterによる席替えのソフトをクラスで表現したいです。

実現したいこと

main.pyの「csvファイルを選ぶボタン」を押したら，sansyo.pyのクラスがインスタンス化されて参照画面を出したいです。
疑問点なのはどこに以下のコードを書けばいいかということです。また，ほかにも改善点があれば教えてください。

def sub():
    subroot = Tk()
    subapp = Subapplication(master=subroot)
    subapp.mainloop()

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

main.pyとsansyo.pyが同時にインスタンス化されていて，またmain.pyのウィジェットが一部消えています。

_tkinter.TclError: can't invoke "button" command: application has been destroyed

該当のソースコード

#main.py
from cProfile import label
import tkinter as tk
from tkinter import LEFT, Tk
from regex import W
import sansyo

class Application(tk.Frame):
    def __init__(self, master = None):
        super().__init__(master)
        self.master.geometry("600x350")
        self.master.title("席替えメーカー")
        self.master.resizable(False,False)
        self.canvas1 = tk.Canvas(self.master,width=600, height=175, bg="#fafafa")
        self.canvas1.pack()
        self.canvas2 = tk.Canvas(self.master,width=600, height=176, bg="#c51949")
        self.canvas2.pack()
        self.canvas2.create_window(85,30, window=tk.Label(master=self.canvas2,
        text="CSVファイル",anchor=tk.W,bg="#c51949", fg="#fafafa",font=('MS Pゴシック', '17')),anchor=tk.W)
        self.canvas2.create_window(25,80, window=tk.Button(master=self.canvas2,
        text="CSVファイルを選ぶ",command=sub(),bg="#fafafa", fg="#c51949",font=('MS Pゴシック', '22'),justify=tk.CENTER),anchor=tk.W)
        self.canvas2.create_window(425,30, window=tk.Label(master=self.canvas2,
        text="次へ",bg="#c51949", fg="#fafafa",font=('MS Pゴシック', '17')),anchor=tk.W)
        self.canvas2.create_window(330,80, window=tk.Button(master=self.canvas2,
        text="      次へ進む      ",bg="#fafafa", fg="#c51949",font=('MS Pゴシック', '22'),justify=tk.CENTER),anchor=tk.W)
   
def main():
    root = tk.Tk()
    app = Application(master=root)
    app.mainloop()

def sub():
    subroot = Tk()
    subapp = sansyo.Subapplication(master=subroot)
    subapp.mainloop()

if __name__ == "__main__":
    main()

#sansyo.py
import os
from tkinter import *
from tkinter import ttk
from tkinter import messagebox
from tkinter import filedialog
import csv

class Subapplication(ttk.Frame):
    def __init__(self, master = None):
        super().__init__(master)
        self.master.title("CSVファイルを選ぶ")
        self.master.resizable(False,False)
        # Frame2の作成
        self.frame2 = ttk.Frame(self.master, padding=5)
        self.frame2.grid(row=2, column=1, sticky=E)

        # 「ファイル参照」ラベルの作成
        self.IFileLabel = ttk.Label(self.frame2, text="ファイル参照＞＞", padding=(5, 2))
        self.IFileLabel.pack(side=LEFT)

        # 「ファイル参照」エントリーの作成
        self.entry2 = StringVar()
        self.IFileEntry = ttk.Entry(self.frame2, textvariable=self.entry2, width=30)
        self.IFileEntry.pack(side=LEFT)

        # 「ファイル参照」ボタンの作成
        self.IFileButton = ttk.Button(self.frame2, text="参照", command=self.filedialog_clicked)
        self.IFileButton.pack(side=LEFT)

        # Frame3の作成
        self.frame3 = ttk.Frame(self.master, padding=0)
        self.frame3.grid(row=3,column=1,sticky=E)

        # 実行ボタンの設置
        self.button1 = ttk.Button(self.frame3, text="適用", command=self.conductMain)
        self.button1.pack(side=LEFT,padx=5)

        # ファイル指定の関数
    def filedialog_clicked(self):
        self.fTyp = [("", "*")]
        self.iFile = os.path.abspath(os.path.dirname(__file__))
        self.iFilePath = filedialog.askopenfilename(filetype = self.fTyp, initialdir = self.iFile)
        self.entry2.set(self.iFilePath)

    # 実行ボタン押下時の実行関数
    def conductMain(self):
        self.text = ""
        self.filePath = self.entry2.get()
        if self.filePath:
            self.text += "ファイルパス：" + self.filePath
            #csvを辞書型に変換
            with open(self.filePath, mode='r',encoding='utf-8') as inp:
                reader = csv.reader(inp)
                dict_from_csv = {rows[0]:rows[1] for rows in reader}
            print(dict_from_csv)
        
        if self.text:
            #ウィンドウ閉店の確認メッセージを表示
            self.res = messagebox.askokcancel("確認","ウィンドウを閉じていいですか？")
            if self.res:
                #rootを削除
                self.master.destroy()
        else:
            messagebox.showerror("error", "パスの指定がありません。")

def sub():
    subroot = Tk()
    subapp = Subapplication(master=subroot)
    subapp.mainloop()

if __name__ == "__main__":
    sub()

試したこと

main.pyのクラスの中に

#省略
        self.canvas2.create_window(330,80, window=tk.Button(master=self.canvas2,
        text="      次へ進む      ",bg="#fafafa", fg="#c51949",font=('MS Pゴシック', '22'),justify=tk.CENTER),anchor=tk.W)
    def sub(self):
        self.subroot = Tk()
        self.subapp = sansyo.Subapplication(master=self.subroot)
        self.subapp.mainloop()

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

python3.9.7,win11

Answer 1

クラス分けの前に、tkinter 等のイベント駆動と呼ばれるプログラム
(多くのGUIライブラリや asyncio等)では、イベントループというイベント処理などを行うループ処理を持ち
通常はプログラム中にひとつだけとします。

app.mainloop() 内で、クリック時の処理などが行われるのですが
その中で subapp.mainloop() を呼び出すのは、イベントループの入れ子となってしまいます。

• tkinter.Tk を複数作りたい場合は、別プロセスにすると可能ですが、このケースは滅多にないはずです。

　Tk() は ライブラリの初期化を担っていて、複数回の呼び出しは想定されてません。
プログラムが複雑化する上に、困難な問題（エラーもなく不可解な挙動をする）の原因となります。

• ウィンドウが複数欲しいだけであれば Toplevel を使う。「モーダルダイアログ」の作り方を調べて下さい。

つまり、sansyo.py 側で tkinter.Tk() mainllop() とするのは単体のテストとしての正しいコードですが、
部品として利用する main.py 側での sub() 関数の実装は、tkinterモジュールの誤用で、
Subapplication は tkinter.Tk ではなく、 tkinter.Toplevel に載せて使うべきです。

• main.sub 関数の内容は Tk()/mainloop() ではなく Toplevel 利用に書き換える
• sansyo.sub -> sansyo.main モジュールを直接した場合に呼び出される関数として、解りやすく mainにしておく

　※ 機能的には問題ありません。

Comments

[power-chan]

回答ありがとうございます。Tkを複数作るのはナンセンスとは初めて知りました。
tkinter.Toplevel(),commandの書き方を改めたら，理想の動きになりました！

Answer 2

問題はここにあります。(見やすいように整形してあります。)

        self.canvas2.create_window(
            25,80, 
            window=tk.Button(master=self.canvas2,
                text="CSVファイルを選ぶ",
                command=sub(),     #★★
                bg="#fafafa", fg="#c51949",
                font=('MS Pゴシック', '22'),
                justify=tk.CENTER),
            anchor=tk.W)

ここで、コマンドに sub() を入れていますね。
commandには「ボタンが押されたときに 呼び出す関数」を指定する必要があって、コードの内容からするとここはsub関数そのものを指定すべきですが、この書き方だと「sub関数を呼び出した返り値」を指定するようになってしまっています。(sub関数が関数を返す関数であればOKですけど)
結果として、この処理を実行したときにsub関数が引数無しで呼び出され(sub()だから)、関数が返るのを待ってしまっています。
このため、画面には mainとsubの両方が表示され、mainのこの行以降の部分はまだ実行されないので画面も中途半端になってしまっています。

正しくは、ここは

                command=sub,     # 正しい

とすべきです。

---

subの定義場所ですが、mainでなく、sansyoの中の方がいいと思います。
この処理はsasyoの中身にだけ必要なものです。別のファイルでインポートして使うことを考えるとわかると思います。
この場合は

                command=sansyo.sub,     # 正しい

になります。

main も sansyo も両Tk()を使っているので、現状は別のアプリケーションとして動作しています。しかし、sansyoの役割はファイル選択ダイアログに近いので、mainに従属するウィンドウであるほうが自然だと思います。
なので、tkinter.Toplevel()を使って作ることが多いパターンでしょう。

Comments

[power-chan]

回答ありがとうございます。
tkinter.Toplevel(),commandの書き方を改めたら，理想の動きになりました！