Pythonでは、変数がNone、True、またはFalseであるかどうかをどのようにテストすればよいですか

次の3つのいずれかを返すことができる関数があります。

• 成功（True）
• 失敗（False）
• ストリームの読み取り/解析エラー（None）

私の質問は、Trueまたはに対してテストすることになっていない場合False、結果がどうなるかをどのように見ればよいかです。以下は私が現在それをやっている方法です：

result = simulate(open("myfile"))
if result == None:
    print "error parsing stream"
elif result == True: # shouldn't do this
    print "result pass"
else:
    print "result fail"

== Trueパーツを削除するのと同じくらい簡単ですか、それともトライブールデータタイプを追加する必要がありますか。simulate外部プログラムでエラーを処理するためにログに記録して続行するだけなので、関数に例外をスローさせたくありません。

例外を恐れるな！プログラムをログに記録して続行させるのは、次のように簡単です。

try:
    result = simulate(open("myfile"))
except SimulationException as sim_exc:
    print "error parsing stream", sim_exc
else:
    if result:
        print "result pass"
    else:
        print "result fail"

# execution continues from here, regardless of exception or not

また、エラー/エラーなしでは十分な情報が得られない場合に備えて、正確に何が問題だったかについて、simulateメソッドからの通知の種類を大幅に増やすことができます。

if result is None:
    print "error parsing stream"
elif result:
    print "result pass"
else:
    print "result fail"

シンプルかつ明確にしてください。もちろん、辞書を事前に定義することもできます。

messages = {None: 'error', True: 'pass', False: 'fail'}
print messages[result]

simulateより多くの戻りコードを含めるように関数を変更することを計画している場合、このコードの保守は少し問題になるかもしれません。

またsimulate、解析エラーで例外が発生する場合があります。その場合、ここでそれをキャッチするか、レベルを上げて、出力ビットを1行のif-elseステートメントに減らします。

決して、決して、決して言わない

if something == True:

決して。ifステートメントの冗長条件ルールとして冗長に指定されているものを冗長に繰り返しているので、それはクレイジーです。

さらに悪いことに、まだ、決して、決して、決して言わない

if something == False:

あなたが持っていnotます。お気軽にご利用ください。

最後に、行うことa == Noneは非効率的です。してくださいa is None。 None特別なシングルトンオブジェクトです。1つしか存在できません。そのオブジェクトがあるかどうかを確認してください。

私は状況があっても、それを強調したいif expr :1を確認するために望んでいるので、十分ではないexprですTrueからわずかに異なるといない0/ None何でも、/ isから好まする== S.Lottを回避するためmentionned同じ理由は、== None。

それは確かに少し効率的であり、ケーキのチェリーで、より人間が読みやすいです。

In [1]: %timeit (1 == 1) == True
38.1 ns ± 0.116 ns per loop (mean ± std. dev. of 7 runs, 10000000 loops each)

In [2]: %timeit (1 == 1) is True
33.7 ns ± 0.141 ns per loop (mean ± std. dev. of 7 runs, 10000000 loops each)

例外をスローする方があなたの状況に適していると思います。別の方法は、タプルを返すシミュレーションメソッドです。最初の項目はステータスで、2番目の項目は結果です。

result = simulate(open("myfile"))
if not result[0]:
  print "error parsing stream"
else:
  ret= result[1]