if 変数 is None とするか、if not 変数のどちらを書くべきか

Question

タイトル通りなのですが、Noneを条件分岐の対象にするときに、if 変数 is None とif not 変数のどちらがベストなのか迷ったので質問です。

他言語はおおよそif !変数のようにすることが多いと思われるので、Pythonでも同様にif not 変数とやるべきなのか。
それとも、意味として伝わりやすい if 変数 is Noneとするべきか。

なにかPEP8などで決まっていたら教えてほしいです。すみませんがお願いします...!

Answer 1

そもそもその二つの条件は挙動が異なります。
例えば [] is None は False ですが、 not [] は True です。

Noneと比較するのが主目的なら is None が自然でしょう。

Answer 2

if not a in b / if a not in bの回答にも挙げられていますが、PEP8のプログラミングに関する推奨事項によると

None のようなシングルトンと比較をする場合は、常に is か is not を使うべきです。絶対に等値演算子を使わないでください。

また、 本当は if x is not None と書いているつもりで、 if x と書いている場合は注意してください - たとえば、デフォルトの値がNoneになる変数や引数に、何かしら別の値が設定されているかどうかをテストする場合です。この「別の値」は、ブール型のコンテクストでは False と評価される(コンテナのような)型かもしれませんよ！

そして 以下、別の話となるので今回は無視してください。

not ... is ... ではなく、 is not 演算子を使いましょう。これらは機能的に同じですが、後者の方が読みやすく、好ましいです。

とのことです。すなわち~~if 変数 is not Noneが良いでしょう。~~

• 「変数がNoneか」判定したい場合はif 変数 is None
• 「変数がNoneではないか」判定したい場合はif 変数 is not None

が良いでしょう。

Comments

[LouiS0616]

not 変数 is None と 変数 is not None を比較しているわけではないような。

[8524ba23]

そうですね。後半は別の話でした。
ご指摘ありがとうございます。