在Python的类中将实例变量声明为None是一种好习惯吗？

考虑以下类别：

class Person:
    def __init__(self, name, age):
        self.name = name
        self.age = age

我的同事倾向于这样定义：

class Person:
    name = None
    age = None

    def __init__(self, name, age):
        self.name = name
        self.age = age

这样做的主要原因是他们选择的编辑器显示了自动补全的属性。

就我个人而言，我不喜欢后者，因为一个类将那些属性设置为毫无意义None。

哪种方法更好，原因何在？

我将后一种不好的做法称为“这没有按照您的想法做”。

您的同事的位置可以改写为：“我将创建一堆类静态准全局变量，这些变量从未被访问过，但是它们确实占用了各个类的命名空间表（__dict__）中的空间，这只是为了使我的IDE能够做到东西。”

1.使您的代码易于理解

读取代码比编写代码要多得多。使您的代码维护人员的工作更加轻松（明年也可能要自己做）。

我不知道任何硬性规则，但我更喜欢将任何未来的实例状态明确声明为完全不使用。与崩溃已经AttributeError很糟糕了。不清楚实例属性的生命周期会更糟。恢复可能导致导致属性分配的呼叫序列所需的精神体操量很容易变得不平凡，从而导致错误。

因此，我通常不仅在构造函数中定义所有内容，而且还努力将可变属性的数量保持在最少。

2.不要混用类级别和实例级别的成员

您在class声明内定义的所有内容均属于该类，并且由该类的所有实例共享。例如，当您在类内定义函数时，该函数将成为所有实例都相同的方法。同样适用于数据成员。这与您通常在中定义的实例属性完全不同__init__。

类级数据成员作为常量最有用：

class Missile(object):
  MAX_SPEED = 100  # all missiles accelerate up to this speed
  ACCELERATION = 5  # rate of acceleration per game frame

  def move(self):
    self.speed += self.ACCELERATION
    if self.speed > self.MAX_SPEED:
      self.speed = self.MAX_SPEED
    # ...

我个人使用__ init __（）方法定义成员。我从没想过要在课堂上定义它们。但是我总是这样做：我初始化__ init__方法中的所有成员，甚至那些__ init__方法中不需要的成员。

例：

class Person:
    def __init__(self, name, age):
        self._name = name
        self._age = age
        self._selected = None

   def setSelected(self, value):
        self._selected = value

我认为在一个地方定义所有成员很重要。它使代码更具可读性。无论是在__ init __（）内部还是外部，都不那么重要。但是，团队必须采用或多或少相同的编码风格，这一点很重要。

哦，您可能会注意到我曾经在成员变量中添加前缀“ _”。

这是一个坏习惯。您不需要这些值，它们会使代码混乱，并且可能导致错误。

考虑：

>>> class WithNone:
...   x = None
...   y = None
...   def __init__(self, x, y):
...     self.x = x
...     self.y = y
... 
>>> class InitOnly:
...   def __init__(self, x, y):
...     self.x = x
...     self.y = y
... 
>>> wn = WithNone(1,2)
>>> wn.x
1
>>> WithNone.x #Note that it returns none, no error
>>> io = InitOnly(1,2)
>>> InitOnly.x
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
AttributeError: class InitOnly has no attribute 'x'

我将使用“有点像docstrings，然后”并声明此无害，只要它始终为None，或者其他值的狭窄范围都不变。

它散发出狂热的气息，并且过度依赖静态类型的语言。而且它不适合作为代码。但是在文档中保留了次要目的。

它记录了期望的名称，因此，如果我与某人组合代码，而我们中的一个拥有“ username”，另一个拥有user_name，那么对于人类来说，有一条线索是我们已经分道扬and，并且没有使用相同的变量。

强制将完全初始化作为策略可以以更加Python的方式实现相同的目的，但是，如果中包含实际的代码，则__init__可以提供一个更清晰的位置来记录使用中的变量。

显然，这里的BIG问题是，它会诱使人们使用以外的值进行初始化None，这可能很糟糕：

class X:
    v = {}
x = X()
x.v[1] = 2

留下全局跟踪，并且不为x创建实例。

但这在整个Python中比在本实践中更像是一个怪癖，我们应该已经对此产生了偏执。