イディオム・パイソン - プロパティまたはメソッド？

Question

私は単純なプロパティとして状態を維持するdjangoモデルクラスを持っています。クラスにいくつかのヘルパープロパティを追加して集約状態にアクセスしました。 is_liveは、状態が['closed', 'expired', 'deleted']などの場合にfalseを返します。

この結果、myモデルにはis_プロパティのコレクションが含まれているため、オブジェクトの内部プロパティーについて非常に簡単な参照が行われます。

ここで、新しいプロパティー、is_completeを追加します。これは、他のすべてのプロパティーと意味的には同じですが、オブジェクトの状態をブール値でチェックします。ただし、このチェックでは、依存する（1対多）子オブジェクトの状態をチェックし、結果に基づいてレポートを返します。つまり、このプロパティは実際には複数のデータベースクエリを実行し、結果を処理します。

（@propertyデコレータを使用して）プロパティとしてモデル化するのはまだ有効ですか？それとも、デコレータを先取りしてメソッドとして残すべきですか？

プロパティを使用していることは、他のすべてのプロパティと意味的に一致していることです。

これは、他の開発者に、これはより複雑な実装であることを控えていることを示しているため、控えめに使用する必要があります（つまり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 

編集：私は、もともと.NETの背景から来分割が見事に

それは間違いなく砂時計を生み出すことができ、すべてのそのコードのいずれかのチャンスがあるかどう

」、とジェフ・アトウッドによって定義されます方法でなければなりません。

EDIT 2：質問へのわずかなアップデート - 類似した名前と混合プロパティとメソッドがあるように、is_completeと呼ばれる方法、としてそれを持っている問題だろう - あるいは単に混乱を招くということでしょうか？

ので - それは次のようになります。

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 

ちょうどそれ「キャッシュ」という点に注意してください。

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

Answer 1

あなたが次のパターンを使用する場合は特に、それを行うために大丈夫ですその結果、最初の実行は計算を行いますが、その後は既存の結果を使用します。