pythonの変数について

現在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)

meg_

2021/02/23 05:40

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

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

回答3件

エラーの原因

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

hcs15さんが

python suirimain.py
と実行すると、まず

python
1import _suisin

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

python
1・
2・
3
4ha = ryuseki(hh, hm, hb)
5hp = jyunpen(hm, hh, hb)
6hk = keisin(ha, hp)
7hv = ryusoku(hn, hi, hk)
8hqq = ryuka(ha, hv)
9
10print(ha, hp, hk, hv, hqq)

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

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

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

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

python
1'''_suisin.py'''
2def ryuseki(hh, hm, hb):   # 1流積
3    return (hh*hm*2+hb+hb)/2*hh
4
5def jyunpen(hm, hh, hb):  # 2 潤辺
6    return pow(1+hm ** 2, 0.5) * hh*2+hb
7
8def keisin(ha, hp):  # 3 径深
9    return ha/hp
10
11def ryusoku(hn, hi, hk):  # 4 流速
12    return 1/hn*pow(hi, 0.5) * pow(hk, 2/3)
13
14def ryuka(ha, hv):  # 5 流下流量
15    return ha*hv
16
17def print_suisin(ha, hp, hk, hv, hqq):
18    ha = ryuseki(hh, hm, hb)
19    hp = jyunpen(hm, hh, hb)
20    hk = keisin(ha, hp)
21    hv = ryusoku(hn, hi, hk)
22    hqq = ryuka(ha, hv)
23    print(ha, hp, hk, hv, hqq)

として、

python
1'''suisin.py'''
2import _suisin
3
4hb = 0.3
5hh = 0.3
6hm = 0.3
7hi = 0.02
8hn = 0.014
9hq = 0.5
10
11print_suisin(ha, hp, hk, hv, hqq)

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

投稿2021/02/23 07:24

ppaul

総合スコア24672

ベストアンサー

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

py
1#_suisin.py
2from suirimain import *
3
4def ryuseki(hh, hm, hb):   # 1流積
5    return (hh*hm*2+hb+hb)/2*hh
6
7def jyunpen(hm, hh, hb):  # 2 潤辺
8    return pow(1+hm ** 2, 0.5) * hh*2+hb
9
10(略)
11

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

投稿2021/02/23 05:58

編集2021/02/23 13:54

kairi003

総合スコア1332

hcs15

2021/02/23 07:09

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

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

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

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

python
1ha = ryuseki(hh, hm, hb)
2hp = jyunpen(hm, hh, hb)
3hk = keisin(ha, hp)
4hv = ryusoku(hn, hi, hk)
5hqq = ryuka(ha, hv)
6
7print(ha, hp, hk, hv, hqq)