tkinterのButtonを押したときの関数について

Question

前提・実現したいこと

tkinterでButtonを押したときにファイルダイアログを開くようにしています．
そのファイルダイアログを開いた後，保存を押したら，selenniumでWebスクレイピングを行い，ファイルダイアログで設定したファイル(csv)に保存するプログラムを作っています．
そこで，ファイルダイアログの保存を押す→seleniumを実装した関数を実行に移る際，関数の中に関数を入れているのですが，これを外部関数化する方法はありますか？

理由は並列処理をしたいので，外部関数にする必要があるからです．
初心者で見にくいコードですがよろしくお願いします

該当のソースコード

from selenium.webdriver.common.keys import Keys 
import tkinter as tk
from tkinter import filedialog


def file_save():
    root = tk.Tk()
    root.withdraw()
    root.filename =  filedialog.asksaveasfilename(initialfile=".csv",initialdir = "/",title = "保存先",filetypes =  [("csv file","*.csv")])

    def  check():
        #スクレイピング内容

　　if root.filename != '':
        check()  



root = tk.Tk()
root.geometry('200x200')
root.title("スクレイピング")

okButton = tk.Button(root, text='OK', command = file_save)
okButton.place(height=20,x=50,y=100)

root.mainloop()

試したこと

from selenium.webdriver.common.keys import Keys 
import tkinter as tk
from tkinter import filedialog


def file_save():
    root = tk.Tk()
    root.withdraw()
    root.filename =  filedialog.asksaveasfilename(initialfile=".csv",initialdir = "/",title = "保存先",filetypes =  [("csv file","*.csv")])
    return root.filename

def  check():
    #スクレイピング内容

root = tk.Tk()
root.geometry('200x200')
root.title("スクレイピング")

okButton = tk.Button(root, text='OK', command = file_save)
okButton.place(height=20,x=50,y=100)

root.mainloop()

if file_save() != '':
    check()

Answer 1

外部関数化が必要な理由は multiprocessing で使う為でしょうか

def file_save():
    filepath = "test.html" # (仮)

    def check():
        print(filepath) # <-- 外側のスコープの変数参照

    check()

def check(filepath):
    print(filepath)


def file_save():
    filepath = "test.html"

    check(filepath) # <-- 引数で与えるように変更する

「外部関数にする必要」から、「スレッド」ではなく「プロセス」だと推測しますが。
注意点は、UI部品等は引数に渡せないので、別プロセスからは直接操作できません。

プロセス間でやり取りできるデータはpickle化できるデータに限られます。
スレッドであればその制限はありません。（代わりに別の制限はあります）

何れの場合にしても、並列処理の結果をUIに反映したい場合は、
tkinter 側のメインスレッドで処理しなければない為、
キューを通してデータをやり取りすることが多いです。

---

# XXX: tkinter等の、プロセス側で不要なモジュールはここでは読み込まない

def worker_process(queue):
    # プロセス側で必要なモジュールのロードはここで行う

    for item in iter(queue.get, None):  # <-- None が送られてくるまでループ
        # csv の保存処理
        print("save-csv", item)


if __name__ == "__main__":
    # ここは、別プロセスでは実行されない
    from multiprocessing import Process, Queue

    queue = Queue()
    worker = Process(target=worker_process, args=(queue,))
    worker.start()

    import atexit
    atexit.register(lambda: queue.put(None))
    # 終了時プロセスの処理を待ち、安全に終わらせる
    # None を送っている理由は、キュー読み出しの iter() で Noneを終端に指定した為。


    def file_save():
        # キューへ送信
        queue.put("test.csv")


    import tkinter as tk
    root = tk.Tk()
    root.title("スクレイピング")

    okButton = tk.Button(root, text='OK', font=("", 30), command=file_save)
    okButton.pack()

    root.mainloop()

プロセスを起動したままデータをやり取りする例ですが、
プロセスの結果をUI側で得たい場合は、逆方向にデータを送るキューがもう一つ必要になり、
メインプロセス側でも別途キューの読み出し処理をします。

また、複数のファイルを同時に処理となってくると、ProcessPool にしたり、
concurrent モジュールを使う等、他にも改善手段はあります。

Comments

[takugon92]

遅くなり申し訳ありません
queueを使い，マルチプロセス化できました
ありがとうございます