Có phải là một cách thực hành tốt để khai báo các biến thể hiện là Không có trong một lớp trong Python?

Hãy xem xét các lớp sau:

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

Đồng nghiệp của tôi có xu hướng định nghĩa nó như thế này:

class Person:
    name = None
    age = None

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

Lý do chính cho điều này là trình soạn thảo lựa chọn của họ hiển thị các thuộc tính để tự động hoàn thành.

Cá nhân, tôi không thích cái thứ hai, bởi vì nó không có nghĩa là một lớp có các thuộc tính đó được đặt thành None.

Cái nào sẽ được thực hành tốt hơn và vì lý do gì?

Tôi gọi thực tiễn xấu thứ hai theo quy tắc "điều này không làm những gì bạn nghĩ nó làm".

Vị trí đồng nghiệp của bạn có thể được viết lại thành: "Tôi sẽ tạo ra một loạt các biến toàn cầu tĩnh lớp không bao giờ được truy cập, nhưng nó chiếm không gian trong các bảng không gian tên của lớp khác nhau ( __dict__), chỉ để làm cho IDE của tôi làm một cái gì đó

1. Làm cho mã của bạn dễ hiểu

Mã được đọc thường xuyên hơn nhiều so với văn bản. Làm cho công việc bảo trì mã của bạn dễ dàng hơn (cũng có thể là chính bạn vào năm tới).

Tôi không biết về bất kỳ quy tắc cứng nào, nhưng tôi thích có bất kỳ trạng thái ví dụ nào trong tương lai được tuyên bố rõ ràng hoàn toàn. Đâm với một AttributeErrorlà đủ xấu. Không thấy rõ vòng đời của một thuộc tính thể hiện là tồi tệ hơn. Số lượng thể dục tinh thần cần thiết để khôi phục các chuỗi cuộc gọi có thể dẫn đến thuộc tính được gán có thể dễ dàng trở nên không tầm thường, dẫn đến lỗi.

Vì vậy, tôi thường không chỉ định nghĩa mọi thứ trong hàm tạo, mà còn cố gắng giữ số lượng thuộc tính có thể thay đổi ở mức tối thiểu.

2. Không trộn lẫn các thành viên cấp lớp và cấp thể hiện

Bất cứ điều gì bạn xác định ngay bên trong classkhai báo đều thuộc về lớp và được chia sẻ bởi tất cả các phiên bản của lớp. Ví dụ, khi bạn định nghĩa một hàm bên trong một lớp, nó sẽ trở thành một phương thức giống nhau cho tất cả các thể hiện. Áp dụng tương tự cho các thành viên dữ liệu. Điều này hoàn toàn không giống với các thuộc tính thể hiện mà bạn thường xác định __init__.

Các thành viên dữ liệu cấp lớp là hữu ích nhất như các hằng số:

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
    # ...

Cá nhân tôi xác định các thành viên trong phương thức __ init __ (). Tôi chưa bao giờ nghĩ về việc xác định chúng trong phần lớp. Nhưng điều tôi luôn làm: Tôi khởi tạo tất cả các thành viên trong phương thức __ init__, ngay cả những người không cần thiết trong phương thức __ init__.

Thí dụ:

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

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

Tôi nghĩ điều quan trọng là xác định tất cả các thành viên ở một nơi. Nó làm cho mã dễ đọc hơn. Cho dù đó là bên trong __ init __ () hay bên ngoài, điều đó không quan trọng. Nhưng điều quan trọng là một nhóm phải cam kết ít nhiều cùng một kiểu mã hóa.

Ồ, và bạn có thể nhận thấy tôi từng thêm tiền tố "_" vào các biến thành viên.

Đây là một thực hành xấu. Bạn không cần những giá trị đó, chúng làm lộn xộn mã và chúng có thể gây ra lỗi.

Xem xét:

>>> 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'

Tôi sẽ đi với "một chút giống như các tài liệu, sau đó" và tuyên bố điều này vô hại, miễn là nó luôn luônNone , hoặc một phạm vi hẹp của các giá trị khác, tất cả đều không thay đổi.

Nó tràn ngập sự thờ ơ và gắn bó quá mức với các ngôn ngữ được gõ tĩnh. Và nó không tốt như mã. Nhưng nó có một mục đích nhỏ còn lại, trong tài liệu.

Nó ghi lại tên dự kiến ​​là gì, vì vậy nếu tôi kết hợp mã với ai đó và một trong số chúng tôi có 'tên người dùng' và tên người dùng khác, có một manh mối cho con người rằng chúng tôi đã chia tay và không sử dụng cùng một biến.

Buộc khởi tạo đầy đủ như một chính sách đạt được điều tương tự theo cách Pythonic hơn, nhưng nếu có mã thực tế trong __init__, điều này cung cấp một nơi rõ ràng hơn để ghi lại các biến được sử dụng.

Rõ ràng vấn đề LỚN ở đây là nó cám dỗ mọi người khởi tạo với các giá trị khác None, có thể là xấu:

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

để lại dấu vết toàn cầu và không tạo một thể hiện cho x.

Nhưng đó chỉ là một sự ngớ ngẩn trong Python nói chung hơn là trong thực tế này, và chúng ta đã nên hoang tưởng về nó.