【Python】別ファイルのdef関数内の変数を使いたい

Question

前提・実現したいこと

スパムメールフィルターを作成しています。
judge,pyでtrain.pyのhoge関数内のps,pnの値を使うためにはjudge.pyのps,pnをどのように書けば良いでしょうか？
train.pyもjudge.pyも同じファイル内にあります。
実際のプログラムを載せてもいいのですが、分かりにくいかと思い一部のみ書いています。
足りないところがあれば、教えて下さい。

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

$ python judge.py training.txt
AttributeError: 'function' object has no attribute 'ps'

該当のソースコード

train.py

def hoge(f):
    ~ # SnumとNnumはここで処理しています
    try:
        ps = float(Snum/(Snum+Nnum))
        pn = float(Nnum/(Snum+Nnum))
    except ZeroDivisionError:
        print("ZeroDivisionError!!\n")

judge.py

import sys
imort train
with open(sys argv[1], "r") as f:
    train.hoge(f)

    # ここの書き方が分かりません
    ps = train.hoge.ps
    pn = train.hoge.pn
~

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

python 3.6.12

Answer 1

train.hoge.ps

という書き方はできません。

return psとして戻り値を使うか、global変数にして参照するかをしてください。

Comments

[退会済みユーザー]

tako15さん
ppaulさんの回答にのっかります（蛇足かも？）。
def hoge(f):の except ZeroDivisionError:になったときに返り値がない状態になります。こうなった時には既に異常な状態が起きた後で、その後プログラムを継続させるべきでないかもしれませんが、今のままだとトラップしたエラーでまたエラーが起きます。

ですので、print("ZeroDivisionError!!\n")に続けて、「ps = None」や「pn = None」をした後に、
ppaulさんの回答にあるように「return ps, pn」のようにした方が安全です。

[tako15]

ppaulさん、そうなんですね。global変数でやってみたら出来ました。ありがとうございます。
fourteenlengthさん、全然蛇足じゃないです！確かに返り値が無いですね…。仰ってもらった通り、やってみようと思います。

Answer 2

ローカル変数というのはその関数の中でだけしか使えない、って変数なので、
関数の外からどーこーしたいならグローバル変数にしましょう

Comments

[tako15]

分かりました。ありがとうございます！