파이썬 클래스에서 인스턴스 변수를 없음으로 선언하는 것이 좋은 습관입니까?

다음 클래스를 고려하십시오.

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__) 에서 공간을 차지하는 많은 클래스 정적 준 글로벌 변수 를 만들려고합니다. 어떤 것."

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 또는 변하지 않는 좁은 범위의 다른 값 이라면 무해하다고 선언합니다 .

그것은 atavism과 정적 타입 언어에 대한 과도한 애착이있다. 그리고 코드만큼 좋지 않습니다. 그러나 문서에 남아있는 사소한 목적이 있습니다.

예상되는 이름이 무엇인지 문서화하므로 누군가와 코드를 결합하고 우리 중 하나가 'username'과 다른 user_name을 가지고 있다면 인간에게 우리가 길을 나 and 고 동일한 변수를 사용하지 않는다는 단서가 있습니다.

정책으로 전체 초기화를 강요하면보다 파이썬적인 방식으로 동일한 일을 달성 할 수 있지만에 실제 코드가있는 __init__경우 사용중인 변수를 문서화 할 수있는 더 명확한 위치를 제공합니다.

분명히 여기서 큰 문제는 사람들이 아닌 다른 값으로 초기화하도록 유혹한다는 것입니다 None.

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

전역 추적을 남기고 x에 대한 인스턴스를 만들지 않습니다.

그러나 이것은 실제 연습보다 파이썬 전체의 단점입니다. 우리는 이미 그것에 대해 편집증이 있어야합니다.