Pythonのif文省略について

２つの関数ってどう違うんですか

python
1def f(x=None):
2     if x:
3         return x
4     else:
5         pass
6
7def g(x=None):
8     if x is not None:
9         return x
10     else:
11         pass
12
13f(5) # 5
14g(5) # 5
15f() # None
16g() # None

dis.disで調べたら

dis.dis(f)
"""
  2           0 LOAD_FAST                0 (x)
              2 POP_JUMP_IF_FALSE        8

  3           4 LOAD_FAST                0 (x)
              6 RETURN_VALUE

  5     >>    8 LOAD_CONST               0 (None)
             10 RETURN_VALUE
"""

dis.dis(g)
"""
              2 LOAD_CONST               0 (None)
              4 COMPARE_OP               9 (is not)
              6 POP_JUMP_IF_FALSE       12

  3           8 LOAD_FAST                0 (x)
             10 RETURN_VALUE

  5     >>   12 LOAD_CONST               0 (None)
             14 RETURN_VALUE
"""

って感じで違うじゃんみたいな

もし得られる結果が一般的に同じならLOAD等の処理が多い関数g()の方が明らかに冗長ですよね？

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

回答2件

ベストアンサー

NoneかどうかとFalseかどうかはまったく別の問題です。

参考：
4. 組み込み型 — Python 3.6.5 ドキュメント　4.1. 真理値判定

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

はじめに — pep8-ja 1.0 ドキュメント　プログラミングに関する推奨事項

余談ですが、gは恒等関数にしてよさそうな気がします。

投稿2018/09/01 04:14

hayataka2049

総合スコア30935

退会済みユーザー

2018/09/01 04:20

あくまでテキトーに書いた例文なのでそれはそうだと思いますｗただまあ、OSSとかみてて絶対に`None`と判定されないようなオブジェクトしかこない文のなかで、 `if x is not None`とか使ってたのが不思議だったので

hayataka2049

2018/09/01 04:23

その方がコードの意図が明確になり、可読性が上がります。また、一般にis比較（オブジェクトのid（アドレス）に基づく比較）はかなり高速です。

退会済みユーザー

2018/09/01 04:25

可読性は確かにこちらのほうがあるかもしれません。ありがとうございます。

hayataka2049

2018/09/01 04:25

timeitで実測してみました。型や中身によって多少の速度差はあると思いますが…… >>> import timeit >>> obj = "hoge" >>> timeit.timeit(lambda : bool(obj)) 0.6472279839981638 >>> timeit.timeit(lambda : obj is None) 0.3192401329979475 >>> timeit.timeit(lambda : obj is not None) 0.35001876699971035 真理値変換よりis比較の方が普通に速い

hayataka2049

2018/09/01 04:31

でもこれはbool関数の実装が遅いだけかもしれませんということで質問文のfとgで実測したところ、ほとんど差は認められませんでした。可読性で選んで良いと思います

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

None以外にもFalseに判定される要素があるのです。

Python
1def f(x=None):
2     if x:
3         return x
4     else:
5         pass
6
7def g(x=None):
8     if x is not None:
9         return x
10     else:
11         pass
12
13f(0)  # None
14g(0)  # 0