この例では、インスタンス変数かメソッド変数か？

Question

この例では、シリアライザオブジェクトは、2つのメソッドのインスタンス変数またはメソッド変数である必要がありますか？両方のアプローチの長所と短所は何ですか？

public class CustomerHelper 
{ 

    private _Serializer; 
    private JavaScriptSerializer Serializer { 
     get 
     { 
      if (_Serializer == null) { 
       _Serializer = new JavaScriptSerializer(); 
      } 
      return _Serializer; 
     } 


    public Customer GetJsonLog(Customer Cust) 
    { 
     return GetFromLog(Serializer.Searilize(Cust)); 
    } 

    public void WriteToLog(Customer Cust) 
    { 
     SaveToLog(Serializer.Searilize(Cust)); 
    } 
} 

編集 - スレッドセーフに関する問題を無視して、Seralizerオブジェクトが再利用されるとします。

Answer 1

あなたのユースケースによって異なります。 CustomHelperが長期間使用されている場合、ローカルスコープのメソッドフィールドではなく、メンバープロパティ（遅延オブジェクトでもなくてもフィールドであっても問題ありません）として保持して再利用することを意味します。 。シリアライザを一度割り当ててから、そのインスタンスを再利用するだけで済みます。

また、あなたのユースケースによって異なります。もしWriteToLogメソッドがCustomHelperの状態に突然変異を起こし、それがSerializerというプロパティをインスタンス化していたとします。以降のすべての呼び出しで変異したSerializerプロパティを共有したいですか？新しいシリアライザが必要な場合は、メソッド内でインスタンス化するだけです。

あなたの例では、CustomHelperはステートレスで、Serializerもそうであると仮定しています。その場合、問題は存在しません。しかし、このような設計上の決定を下す際には、留意すべきことです。

CustomHelperが長生きしている場合は、プロパティを削除してプライベートフィールドを使用してコードを少し修正します。それが長期間続くと、ヘルパーをインスタンス化して初めて使用する確率が高くなります。したがって、コンストラクタでシリアライザをインスタンス化することは、本当に傷つくことはありません。その後の呼び出しで同じCustomHelperインスタンスを再使用する場合は、それほどです。