Python 関数 条件分岐による戻り値について

Question

実現したいこと

PySimpleGUIを使用し、ファイルを二つ選択し、「読み込み開始」ボタンを押すとファイルの読み込みを開始するGUIを作成しています。
それぞれのファイルが未選択、またはファイル形式外（テキストファイルのみ選択可能な仕様にしています）の場合、エラーメッセージを表示するようにしています。
どちらも正常なファイルが選択された場合、次の処理（ファイルの読み込み等）に進みたいと考えているのですが、現状のコードだとどちらもTrueの場合にエラーとなってしまいます。
ファイル選択時ではなく、ファイルの読み込み等の処理が完了したらcode = '0'で終了したいと考えています。
異常がある場合のみエラーコードと、エラーメッセージを返却するようにしたいです。

import PySimpleGUI as sg
import file_function as ff
from file_select import FileSelect
sg.theme('Dark Brown')
layout = [
    [sg.Text('1つ目のファイルのを選択してください'),sg.InputText(key='FILE1', size=(45,1)), sg.FileBrowse('参照', target='FILE1')],
    [sg.Text('1つ目のファイルのを選択してください'),sg.InputText(key='FILE2', size=(45,1)), sg.FileBrowse('参照', target='FILE2')],
    [sg.Button('読み込み開始',key=FileSelect.START)],
    [sg.CloseButton('閉じる')]
]
window = sg.Window('ファイル選択サンプル', layout)
while True:
    event, values = window.read()
    if event == sg.WIN_CLOSED: 
        break       
    code, message = ff.file_func(event,values['FILE1'], values['FILE2'])    
    #if code = '0':
    #      sg.popup_ok(message,title='ファイルの読み込み完了'     
    if code == '1':
        sg.popup_error(message, title='選択エラー')   
window.close()

import PySimpleGUI as sg
import os
from file_select import FileSelect
def file_func(event, first_file_path,second_file_path): 
    first_file_state = False
    if first_file_path == '':
        first_file_message = "1つめが選択されていません。"
    elif os.path.splitext(first_file_path)[1] != '.txt':
        first_file_message = "1つめがテキストファイルではありません。"
    second_file_state = False
    if second_file_path == '':
        second_file_message = "2つめが選択されていません。"
    elif os.path.splitext(second_file_path)[1] != '.txt':
        second_file_message = "2つめがテキストファイルではありません"
    if (first_file_state == False) or (second_file_state == False):
        code = "1"
    return code, f"{first_file_message}\n{second_file_message}"

#ファイルの読み込みの処理記載予定
#    code = "0"

試したこと

file_functionの15行目if条件の変更で、
どちらもTrueだったら何もしないという意味でpassを入れたり色々試しているのですがうまくいかず・・・アドバイスをいただきたいです。

if first_file_state and second_file_state:
    pass
else:
    code = "1"

Comments

[y_waiwai]

なにがどういうふうにうまく行かないのかを説明しよう

[quickquip]

> 異常がある場合のみエラーコードと、エラーメッセージを返却するようにしたいです。
この部分が、「やりたいこと」に対する「正しくない解決方法」なせいで回答できない質問になっているように思います。

https://ja.meta.stackoverflow.com/questions/2701/#answer-2702

Answer 1

前の質問で回答しましたが、処理の内容を理解不十分なので、コードのロジックが間違えています。

    first_file_state = False    ★1
    if first_file_path == '':  ★ 2
        first_file_message = "1つめが選択されていません。"
    elif os.path.splitext(first_file_path)[1] != '.txt':  ★ 3
        first_file_message = "1つめがテキストファイルではありません。"
    else:   ★ 4
        first_file_state = True    ★5

提示したコードは1つめのファイルの判定の部分です。
「first_file_state」変数は、1つめのファイルの判定結果を表わしています。
この処理では、始め(★1)で判定結果をNG(False)としています。これは今回の判定が「だめでなければOK」という方式だからです。
その後if文で、NGの条件をチェックしていくわけです。 ★2 ★3 NGにひっかかれば、first_file_stateはなにもせずそのままです。(メッセージは設定しています)
で、質問にあるコードはここまでで終りです。 これでは、問題なかったことを判定できません。
判定するには、★4 のように else節を付けて、そこでfirst_file_stateをTrueにしてやる必要があります。

この方式がわかりにくいということであれば、ロジックを逆にしてはどうでしょうか。

    first_file_state = True    ★1
    if first_file_path == '':  ★ 2
        first_file_state = False
        first_file_message = "1つめが選択されていません。"
    elif os.path.splitext(first_file_path)[1] != '.txt':  ★ 3
        first_file_state = False
        first_file_message = "1つめがテキストファイルではありません。"

始めに first_file_state の値 をTrueにしておいて、チェックしてNGであれば、Falseにする方式です。

判定が正しくできていれば、以下のように判定できます。
ただし、ここもelse節でcode="0"を設定する必要があります。

 if (first_file_state == False) or (second_file_state == False):
        code = "1"
else:
       code = "0"
    return code, f"{first_file_message}\n{second_file_message}"

Comments

[emanu]

何度もご回答本当にありがとうございました。
ステータスではなく、メッセージをリストに追加する方式でメッセージの長さが0以上の場合はリターンすることで異常がある場合のみ戻すことができました。

[TakaiY]

解決して納得ということであればいいのですが、メッセージが空であればOKという判定の方式はいい方法とは言えません。きちんと状態管理すべきだと思います。

Answer 2

file_function.py

import os

def file_func(event, first_file_path,second_file_path): 
    first_file_state = second_file_state = False
    first_file_message = second_file_message = ''

    if first_file_path == '':
        first_file_message = "1つめが選択されていません。"
    elif os.path.splitext(first_file_path)[1] != '.txt':
        first_file_message = "1つめがテキストファイルではありません。"
    else:
        first_file_state = True

    if second_file_path == '':
        second_file_message = "2つめが選択されていません。"
    elif os.path.splitext(second_file_path)[1] != '.txt':
        second_file_message = "2つめがテキストファイルではありません"
    else:
        second_file_state = True

    if first_file_state and second_file_state:
        return True, ''
    else:
        return False, '\n'.join((first_file_message, second_file_message))

filesample.py

import PySimpleGUI as sg
import file_function as ff
from file_select import FileSelect

sg.theme('Dark Brown')
layout = [
    [sg.Text('1つ目のファイルのを選択してください'),sg.InputText(key='FILE1', size=(45,1)), sg.FileBrowse('参照', target='FILE1')],
    [sg.Text('1つ目のファイルのを選択してください'),sg.InputText(key='FILE2', size=(45,1)), sg.FileBrowse('参照', target='FILE2')],
    [sg.Button('読み込み開始',key=FileSelect.START)],
    [sg.CloseButton('閉じる')]
]

window = sg.Window('ファイル選択サンプル', layout)
while True:
    event, values = window.read()
    if event == sg.WIN_CLOSED: 
        break       
    code, message = ff.file_func(event,values['FILE1'], values['FILE2'])    
    if code:
        sg.popup_ok(message, title='ファイルの読み込み完了')
    else:
        sg.popup_error(message, title='選択エラー')   

window.close()

Answer 3

とりあえず、そのifが不成立のときにはcodeと言う変数が未定義になってしまいますね

不成立のときにはcodeはどのようにしたいんでしょうか

Comments

[emanu]

コメントくださりありがとうございます。
ifが不成立の場合（どちらも正常なファイルが選択されていた場合）は、コードを返さず、ファイル選択ミスの場合にだけポップアップに遷移したいと考えています。if不成立の場合はfile_functionでファイルの読み込みを開始する処理に続きたいのですが、ご説明になっておりますでしょうか。

[y_waiwai]

いや、私が言ってるのは、そのコードがエラーになってしまうってことです。
エラーでプログラムを落としたいってことなら、いまのまま未定義でもいいですが。