#実現したいこと以下の command= lambda: open_folder(inifile,_folder)) で、呼び出した関数の先で_folderに値を入れているのですが、帰ってきたときには値が書き換わっています。スコープ？の問題だというのはわかったのですが、では、どのようにすればcommandで関数を実行して、そこでの処理結果を得ることができるのでしょうか？よろしければご教授願います。 ```python #メニューに親メニュー（ファイル）を作成する menu_file = tkinter.Menu(root,tearoff=0) men.add_cascade(label='設定', menu=menu_file) #親メニューに子メニュー（処理対象のフォルダを選択する）を追加する menu_file.add_command(label='処理対象のフォルダを選択する', command= lambda: open_folder(inifile,_folder)) ``` 以上、よろしくお願いいたします。 ##追記１ ```python # 設定-フォルダ選択 def open_folder(inifile,_folder): if(inifile != ""): try: _folder = inifile.get("setting","targetdirectory") except: if(_folder == ""): _folder = "C://" _folder = filedialog.askdirectory(initialdir = _folder) return _folder ```

Tkinterでコマンドから値を返したい

#実現したいこと
以下の

command= lambda: open_folder(inifile,_folder))

で、呼び出した関数の先で_folderに値を入れているのですが、帰ってきたときには値が書き換わっています。
スコープ？の問題だというのはわかったのですが、では、どのようにすればcommandで関数を実行して、そこでの処理結果を得ることができるのでしょうか？

よろしければご教授願います。

python
1#メニューに親メニュー（ファイル）を作成する 
2menu_file = tkinter.Menu(root,tearoff=0) 
3men.add_cascade(label='設定', menu=menu_file) 
4
5#親メニューに子メニュー（処理対象のフォルダを選択する）を追加する 
6menu_file.add_command(label='処理対象のフォルダを選択する', command= lambda: open_folder(inifile,_folder))

以上、よろしくお願いいたします。

##追記１

python
1# 設定-フォルダ選択
2def open_folder(inifile,_folder):
3	if(inifile != ""):
4		try:
5			_folder = inifile.get("setting","targetdirectory")
6		except:
7			if(_folder == ""):
8				_folder = "C://"
9		_folder = filedialog.askdirectory(initialdir = _folder) 
10		return _folder

dodox86

2019/10/02 02:14

open_folderは自作の関数だと思いますが、正しい回答を得るためopen_folder関数のコードを提示しましょう。「帰ってきたときには値が書き換わっています。」とありますが、書き換わっているのではなく、変わっていないのでは？

harpy

2019/10/02 02:24

返信ありがとうございます。わかりにくくて、失礼しました。デバッガでリターンの時点までは意図した値に書き換わっているのは確認しました。以上よろしくお願いいたします。

dodox86

2019/10/02 02:40

> デバッガでリターンの時点までは意図した値に書き換わっているのは確認しました。いえ、呼び出し側の_folder変数が書き換わっていないでしょう？と言う意味でした。 ※回答しましたのでご参照ください。

t_obara

2019/10/02 03:16

classにまとめるとよろしいかと。add_commandしているステップに返却してもtkinterの場合あまり使い所がないですし。

harpy

2019/10/02 08:55

コメントありがとうございます。クラスにまとめる方法については別に質問を立てたほうがよろしいでしょうか？

t_obara

2019/10/02 09:38

すでに、直接的な回答は提示されているので、そちらをBAにして、別途質問内容が異なる質問を立てるのがよろしいのではないかと私は思います。

harpy

2019/10/03 00:22

かしこまりました。ありがとうございます。別に質問を立てて、もしお答えいただけるようでしたら、その時はよろしくお願いいたします。 dodox86さん、とりあえずは動くようになったのでありがとうございます。当初の目的は果たせました。

行動規範の内容に同意します

回答1件

ベストアンサー

変数 _folderをグローバルにして、open_folder関数内でglobal宣言してください。
※グローバルにすることの良し悪しの問題はさておき。

Python3
1# グローバル変数として定義
2_folder = ""
3
4def open_folder(inifile, _folder):
5    global _folder
6
7    if(inifile != ""):
8
9    ...以下略

追記しました： 2019-10-02 13:46

グローバル変数を使わない代案を示そうと試みましたが、残念な結果となりました。
代案を示せる、と思ったのは私のPython3に関する理解が不足していたためです。どうもすみません。

質問者であるさんharpyさんのもとのコードでは return _folder と呼び出し側に返しているのでこれを使うべきかと思い直し、ラムダ式／匿名関数内でcommand=lambda: _folder=open_folder(inifile, _folder) のように記述すれば良いと思ったのですが、確認したところ、そもそもPythonの言語仕様でラムダ式の中に文は書けないので、これは叶わない案でした。

ですので、回答としては旧案に戻ります。より良い方法があれば私も知りたいですね。

Python3
1_folder = ""
2
3# 引数の_folderは除去
4def open_folder(inifile):
5    global _folder
6
7    if (inifile != ""):
8        try:
9            _folder = inifile.get("setting","targetdirectory")
10        except:
11            if(_folder == ""):
12                _folder = "C://"
13
14        _folder = filedialog.askdirectory(initialdir = _folder)
15
16...省略
17
18menu_file.add_command(label='処理対象のフォルダを選択する', command= lambda: open_folder(inifile))

投稿2019/10/02 02:38

編集2019/10/02 04:47

dodox86

総合スコア9427

dodox86

2019/10/02 02:51 編集

回答しましたが、もとのコードがせっかく_folderをreturnしているのに、グローバルにするの安易に過ぎて回答としてもNGだと思いました。追って別解をあげます。あるいは別の回答者さんからのより良い回答をお待ちください。

harpy

2019/10/02 03:01

修正依頼の内容についても理解しました。たしかに、globalを宣言することで意図通りに値が書き換わっていました。調べたところ、そもそもあまりグローバル変数は使わないほうが良いというのも理解しました。ぜひ参考にさせていただきたいと思いますので、別の解についてもお聞かせ願えればと思います。よろしくお願いいたします。

dodox86

2019/10/02 04:48

追記しました。当初の思惑が叶わなかったため、別解は消滅してしまいました。すみません。