pythonの変数について

Question

現在pythonでwindowsアプリを作成しようと学習中です。よろしくお願いします。

suirimain.pyの変数　hh、hb・・・・を_suisin.pyにて計算を行い、計算結果を表示させたっかですが

SyntaxError: invalid syntax
PS E:\winpython\python try> & c:/users/ougi-notepc/appdata/local/programs/python/python39/python.exe "e:/winpython/python try/suirimain.py"
Traceback (most recent call last):
File "e:\winpython\python try\suirimain.py", line 3, in <module>
import _suisin
File "e:\winpython\python try_suisin.py", line 26, in <module>
ha = ryuseki(hh, hm, hb)
NameError: name 'hh' is not define

のエラーが出ています。
ファイルのデレクトリーは同じ階層にあります。

import、globalなどをわからないなりに色々なサイトで試しましたが駄目でした。

コードは以下の通りです。よろしくお願いします。

suirimain.py

import _suisin

hb = 0.3
hh = 0.3
hm = 0.3
hi = 0.02
hn = 0.014
hq = 0.5

_suisin.py

def ryuseki(hh, hm, hb): # 1流積

return (hh*hm*2+hb+hb)/2*hh

def jyunpen(hm, hh, hb): # 2 潤辺
return pow(1+hm ** 2, 0.5) * hh*2+hb

def keisin(ha, hp): # 3 径深
return ha/hp

def ryusoku(hn, hi, hk): # 4 流速
return 1/hn*pow(hi, 0.5) * pow(hk, 2/3)

def ryuka(ha, hv): # 5 流下流量
return ha*hv

ha = ryuseki(hh, hm, hb)
hp = jyunpen(hm, hh, hb)
hk = keisin(ha, hp)
hv = ryusoku(hn, hi, hk)
hqq = ryuka(ha, hv)

print(ha, hp, hk, hv, hqq)

Comments

[meg_]

インデントを表示させるために、コードは「コードの挿入｣で記入してください。

Answer 1

エラーの原因

pythonはファイルの最初から最後に向かって順番に処理を行います。
import文については、それがimport済みでなければ読み込んで実行しますが、import済みであれば名前の登録をするだけです。

hcs15さんが

python suirimain.py

と実行すると、まず

import _suisin

が実行されます。
その結果

・
・

ha = ryuseki(hh, hm, hb)
hp = jyunpen(hm, hh, hb)
hk = keisin(ha, hp)
hv = ryusoku(hn, hi, hk)
hqq = ryuka(ha, hv)

print(ha, hp, hk, hv, hqq)

が実行されます。
ha = ryuseki(hh, hm, hb)
が実行されたときには、まだhhが定義されていませんので、
NameError: name 'hh' is not define
が出ました。

どのようにすれば良いのか。

もしも、このプログラムが今回のデータで一回だけ実行するだけならTakaiYさんとkairi003さんが書かれているように変更しても良いですし、一個ののファイルに書いても良いでしょう。
文法的な問題ではありませんが、importされるファイルには、関数定義、クラス定義、定数として扱う変数の定義だけにするのが普通のやり方です。

そういう方法に従って、複数のデータでこのプログラムを書き直すと以下のようになります

'''_suisin.py'''
def ryuseki(hh, hm, hb):   # 1流積
    return (hh*hm*2+hb+hb)/2*hh

def jyunpen(hm, hh, hb):  # 2 潤辺
    return pow(1+hm ** 2, 0.5) * hh*2+hb

def keisin(ha, hp):  # 3 径深
    return ha/hp

def ryusoku(hn, hi, hk):  # 4 流速
    return 1/hn*pow(hi, 0.5) * pow(hk, 2/3)

def ryuka(ha, hv):  # 5 流下流量
    return ha*hv

def print_suisin(ha, hp, hk, hv, hqq):
    ha = ryuseki(hh, hm, hb)
    hp = jyunpen(hm, hh, hb)
    hk = keisin(ha, hp)
    hv = ryusoku(hn, hi, hk)
    hqq = ryuka(ha, hv)
    print(ha, hp, hk, hv, hqq)

として、

'''suisin.py'''
import _suisin

hb = 0.3
hh = 0.3
hm = 0.3
hi = 0.02
hn = 0.014
hq = 0.5

print_suisin(ha, hp, hk, hv, hqq)

このようにすれば、データを変えて実行することが簡単になります。

Answer 2

pythonでは基本的にimportしなければ変数を共有しません．
suirimain.pyで定義されていても，_suisin.pyではhbなどの変数が定義されていないのでエラーになります．
_suisin.pyの先頭でsuirimain.pyから変数をインポートすれば期待するように動作すると思います．

#_suisin.py
from suirimain import *

def ryuseki(hh, hm, hb):   # 1流積
    return (hh*hm*2+hb+hb)/2*hh

def jyunpen(hm, hh, hb):  # 2 潤辺
    return pow(1+hm ** 2, 0.5) * hh*2+hb

(略)

ちなみにpythonでは相互にインポートしても循環はしないようになっていますが，動作が分かりづらいという意味で設計上あまりよろしくないので，_suisin.pyには関数だけ定義してsuirimain.pyでそれを使うといった実装のほうが無難です．

Comments

[hcs15]

大変助かりました。ありがとうございます。
importする方法に大きな誤りがあり、勉強をさせていたきました。

Answer 3

インポートの考えかたが間違えていると思います。

この関係であれば「suirimain.py」がメインのスクリプトになるので、

ha = ryuseki(hh, hm, hb)
hp = jyunpen(hm, hh, hb)
hk = keisin(ha, hp)
hv = ryusoku(hn, hi, hk)
hqq = ryuka(ha, hv)

print(ha, hp, hk, hv, hqq)

この計算部分は、suirimain.py に入っていなければなりません。

現状だと、インポートしたときにこの部分が実行され、さらにこれらの処理に使っている変数は、suirimain.py側にあるので、「 _suisin.py」では見えずに未定義となります。

Comments

[hcs15]

ありがとうございます。
勉強させていただきました。