python Idiomatic - tài sản hoặc phương pháp?

Question

Tôi có một lớp mô hình django duy trì trạng thái là một thuộc tính đơn giản. Tôi đã thêm một số thuộc tính trợ giúp cho lớp để truy cập các trạng thái tổng hợp - ví dụ: is_live trả về false nếu trạng thái là bất kỳ một trong số ['closed', 'expired', 'deleted'] v.v.

Kết quả là mô hình của tôi có tập hợp các thuộc tính is_, thực hiện tra cứu rất đơn giản trên các đặc tính bên trong của đối tượng.

Bây giờ tôi muốn thêm thuộc tính mới, is_complete - ngữ nghĩa giống như tất cả các thuộc tính khác - kiểm tra boolean về trạng thái của đối tượng - tuy nhiên, kiểm tra này liên quan đến việc tải phụ thuộc (một-nhiều)) các đối tượng con, kiểm tra trạng thái của chúng và báo cáo lại dựa trên các kết quả - tức là thuộc tính này thực sự thực hiện một số truy vấn cơ sở dữ liệu (nhiều hơn một) và xử lý các kết quả.

Vì vậy, nó vẫn còn hợp lệ để mô hình như một tài sản (sử dụng trang trí @property trang trí), hay tôi thay vì bỏ qua trang trí và để nó như một phương pháp?

Việc sử dụng thuộc tính là phù hợp ngữ nghĩa với tất cả các thuộc tính khác is_. Sử dụng một phương pháp là nó chỉ ra cho các nhà phát triển khác rằng đây là một cái gì đó có triển khai phức tạp hơn, và do đó nên được sử dụng một cách tiết kiệm (tức là không phải bên trong vòng lặp for..).

from django.db import models 

class MyModel(models.Model): 

    state = CharField(default='new') 

    @property 
    def is_open(self): 
     # this is a simple lookup, so makes sense as a property 
     return self.state in ['new', 'open', 'sent'] 

    def is_complete(self): 
     # this is a complex database activity, but semantically correct 
     related_objects = self.do_complicated_database_lookup() 
     return len(related_objects)==0 

EDIT: Tôi đến từ một nền tảng .NET ban đầu, nơi sự phân cách đáng ngưỡng mộ xác định bởi Jeff Atwood như

"nếu có bất kỳ cơ hội ở tất cả các mã có thể đẻ trứng một đồng hồ cát, nó chắc chắn nên là một phương pháp. "

EDIT 2: nhẹ bản cập nhật cho câu hỏi - nó sẽ là một vấn đề để có nó như một phương pháp, gọi là is_complete, để có được tài sản và phương pháp trộn lẫn với tên tương tự - hoặc là chỉ khó hiểu?

Vì vậy - nó sẽ giống như thế này:

>>> m = MyModel() 
>>> m.is_live 
True 
>>> m.is_complete() 
False 

Answer 1

Nó không quan trọng để làm điều đó, đặc biệt là nếu bạn sẽ sử dụng mẫu sau:

class SomeClass(models.Model): 
    @property 
    def is_complete(self): 
     if not hasattr(self, '_is_complete'): 
      related_objects = self.do_complicated_database_lookup() 
      self._is_complete = len(related_objects) == 0 
     return self._is_complete 

Chỉ cần nhớ rằng nó là "lưu trữ" kết quả, vì vậy việc thực hiện đầu tiên sẽ tính toán, nhưng sau đó sử dụng các kết quả hiện có.