Name can be not defined

Question

if STATUS == 0:
    check_str = "Success"
if STATUS == 1:
    check_str = "Error"
print(check_str)

例えばこのようなコードの時、check_strは"Name can be not defined"になると思います。
この書き方はお行儀が悪いように感じるのですがどうすればnot definedにならないコードになるのでしょうか？

Answer 1

提示コードですが、まずはifを連ねるのではなくif～elif～にしたいところです。そのうえで…

①先に定義しておくか

check_str = '???'
if STATUS == 0:
    # 略

②if～elif～elseにするか

if STATUS == 0:
    check_str = "Success"
elif STATUS == 1:
    check_str = "Error"
else
    check_str = "???"

でしょうか。どちらでもよいと思いますが、私なら①を採用することが多いです。

Comments

[noob_o]

てきとうに書いていたのでelifになっていなくてすいません。
私も①check_str = ""と書く事などを検討していたのですがこのように宣言ばかりを羅列させると見栄えが悪く見えるような気がして困っていました。
②のように全選択肢の中に文字列を入れるとName can be not definedには該当しないのですね。今更だけど意識してなかった為に気づかなかったです。ありがとうございます

Answer 2

check_status = ...
の代入を print の前に置けば、各自に check_status に値が設定されます。

STATUS = -1
check_str = "Success" if STATUS == 0 else "Error" if STATUS == 1 else "???"
print(check_str)

後置 if で書きましたが、これを function にしたり、 Dict （デフォルト値あり)を引くようにしたりと、他の書き方もできます。