Python里一个很危险的函数

python| 2016-07-15

本文由 Hacker News 上曾经排名第一的文章编译而来，作者 Hynek Schlawack 是一名德国软件工程师。

他建议，除非是编写只兼容 Python 3 的代码而且清楚地了解hasattr()的用法，否则不要使用hasattr()。

在 Python 2 下，不建议编写这样的代码：

if hasattr(x, "y"):
    print(x.y)
else:
    print("no y!")

更好的做法是像如下两例所示：

try:
    print(x.y)
except AttributeError:
    print("no y!")

或者

y = getattr(x, "y", None)
if y is not None:
    print(y)
else:
    print("no y!")

如果处理的不是用户自行创建的类，更应该采用上述写法。

上面有一处用到了getattr()，这里将属性缺失视作属性的值为None（常见的情况），如果想与此区分开来，可使用一个标记值（sentinel value）。getattr的速度不比hasattr()低，因为二者的查找过程完全相同，并且后者不会保存结果（至少在CPython实现下是如此）。

为什么不建议使用`hasattr()`

在 Python 2 下使用hasattr()，和下面的代码几乎没有分别：

try:
    print(x.y)
except:
    print("no y!")

但是这样会隐藏掉特性（property）：

>>> class C(object):
...     @property
...     def y(self):
...         0/0
...
>>> hasattr(C(), "y")
False

对于第三方库中类，我们无法确定某个属性（attribute）是否为特性（或者之后某次更新将其变成特性），因此上面那样使用hasattr()是非常危险的。

你或许不信，但是 Hacker News 上确实有程序员回复说经历过这个情况，hasattr()隐藏了一个非常深的错误，让程序调式工作变得异常艰难。

另外一个原因是，对特性使用hasttr()会执行它们的 getter 函数，但这样和hasattr()这个函数的名称并不相符。

不过，在 Python 3 中，hasattr()不存在这些问题：

>>> class C:
...     @property
...     def y(self):
...         0/0
...
>>> hasattr(C(), "y")
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "<stdin>", line 4, in y
ZeroDivisionError: division by zero