副作用のゲッター

Question

オブジェクトが初期化されたクラスをXMLコードの束である で作成します。クラスは、そのXMLからさまざまなパラメータを抽出し、オブジェクト状態変数内にそれらをキャッシュする機能を持っています。これらのパラメータの潜在的な量は大きく、おそらくユーザーは大部分を必要としません。そのため、私は「怠惰な」初期化を実行することに決めました。

次のテストケースでは、このようなパラメータはtitleです。ユーザーが最初にそれにアクセスしようとすると、getter関数は、XMLを解析し、適切な状態変数を初期化し、その値を返します。

class MyClass(object):  
    def __init__(self, xml=None): 
     self.xml = xml 
     self.title = None 

    def get_title(self): 
     if self.__title is None: 
      self.__title = self.__title_from_xml() 
     return self.__title 

    def set_title(self, value): 
     self.__title = value 

    title = property(get_title, set_title, None, "Citation title") 

    def __title_from_xml(self): 
     #parse the XML and return the title 
     return title   

これは良さそうに見えますし、私のために正常に動作します。しかし、ゲッター関数は実際にはオブジェクトに非常に重大な副作用があるという意味では "セッター"であるという事実によって少し邪魔されています。これは正当な懸念ですか？もしそうなら、どのように私はそれに対処する必要がありますか？

Answer 1

ゲッターは確かに副作用がありますが、これは伝統的に悪い副作用とは考えられないことです。ゲッターは常に同じ状態を返すので（状態の間に変化があっても）、ユーザーには見えない副作用はありません。これはプロパティの典型的な使用ですので、心配する必要はありません。

Answer 2

このデザインパターンはLazy initializationと呼ばれ、正式な使用方法があります。

Answer 3

かなり遅いですが、うまくいきません：怠惰な初期化自体は問題ありませんが、誰かがオブジェクトのtitleにアクセスするまで、xml解析などを延期しません。計算された属性はプレーンな属性のように振る舞い、平文の属性アクセスはになりません。 raise（もちろん属性が存在すると仮定します）。

FWIW以前の開発者がOPの例とまったく同じ方法でプロパティを使用していたため、XMLパースエラーが最も予想外の場所で発生したため、私が引き継いだプロジェクトで非常に似たケースがありました。それはインスタンス化時に解析と検証の部分を置くことによってそれです。

そうでは、場合にのみ、としたときに最初にアクセスする決して昇給を知っ遅延初期化のプロパティを使用します。実際には、（少なくとも設定が異なる状況では）何かのためにプロパティを使用しないでください。それ以外の場合は、プロパティを使用しないで、ゲッターを明示的なメソッドにし、明示的に文書化してください。

注意：何かをキャッシュするためのプロパティを使用することはここでは問題ではありませんが、これだけで問題ありません。