在 Python 中使用统一化变量

Question

背景：我有一个类对带有寄存器的芯片进行建模，该芯片有一堆寄存器，其中一个是内置温度传感器的高温限制。

我有以下几点：

class foo():
  def __init__(self):
    # does not set self._hiTemp!
    ...
  def setHiTemp(self, t):
    self._hiTemp = t
  def getHiTemp(self):
    return self._hiTemp
  def checkHiTemp(self):
    return self._temp > self._hiTemp

我不在 __init__ 中声明 self._hiTemp 的原因是因为用户可能不关心芯片的温度感应功能。用户可以以不同的方式使用芯片，给这个变量赋予一个无意义的值是没有意义的。但是，如果用户在没有先设置的情况下尝试使用 self._hiTemp ，则使用未声明变量的错误比一些模糊错误（例如比较数字和 None ）更容易调试/回溯（或者在某些情况下）甚至根本没有错误）。

一切都很顺利，直到我启动 pylint，当然我几乎到处都得到了 W0201：在 init 之外定义的属性。我只是想知道这种编码风格是否令人不悦，如果是，那么“Pythonic 方式”是什么。

谢谢

Answer 1

我的方法是将其设置为 None 或其他一些“本质上”不会出现的哨兵值。然后，对于需要设置它的操作，请使用 assert 快速失败，以防调用者尝试不恰当地使用您的对象。

def __init__(self):
    self._hiTemp = None

def checkHiTemp(self):
    assert self._hiTemp is not None, 'Why you no set _hiTemp before checking it?'
    return self._temp > self._hiTemp

Answer 2

Python 不是 Java，所以不要像那样编写 getter 和 setter。你可以这样解决你的问题

class Foo(object):
    def __init__(self, hiTemp=None):
        self._hiTemp = hiTemp

    @property
    def hiTemp(self):
        if self._hiTemp is None:
            raise AttributeError("You have not initialized hiTemp")
        return self._hiTemp

    @hiTemp.setter
    def hiTemp(self, value):
        self._hiTemp = value

    def checkHiTemp(self):
        return self._temp > self._hiTemp

foo=Foo()
foo.hiTemp = 50
print foo.hiTemp # Prints 50

foo=Foo(hiTemp=20)
print foo.hiTemp # Prints 20

foo=Foo()
print foo.hiTemp # Raises exception

Answer 3

如果您希望通过方法设置对象的特定属性，那么总是有理由在初始化程序中使用它。您可以只使用 self._hiTemp=None 您已经通过前导 _ 将其声明为私有，因此用户会理解不要依赖它。

Answer 4

通常使用值 None 来表示没有实际值。您可以将其初始化为 __init__ 中的值。在这种情况下，您还可以使用无意义的值（例如负数），并在 checkHiTemp 方法中检查该值，如果在没有正确初始化的值的情况下使用，则引发自定义异常。这是一个更有意义的错误。

Answer 5

我认为从用户的角度来看，拥有正确的错误消息比未声明的变量错误更好。我会这样做：

def __init__(self):
    self._hiTemp = None

def setHiTemp(self, t):
    self._hiTemp = t

def getHitemp(self):
    if self._hiTemp is None:
        raise Exception('You need to setHiTemp() before using it.')
    else:
        return self._hiTemp

def checkHiTemp(self):
    if self._hiTemp is None:
        raise Exception('You need to setHiTemp() before using it.')
    else:
        return self._temp > self._hiTemp