遅延ロードを行うためにプライベートプロパティのヌルチェックを避けるにはどうすればよいですか？

Question

私はロード時にSQLデータベースからデータを取り込むような20のフィールドを持つクラスを持っています。現在、私はSQLのprocを呼び出し、すべての必要なフィールドに値を設定するコンストラクタの直後にデータのロードメソッドを呼び出しています。時には、私はこれらの20のフィールドに全くアクセスしないかもしれませんが、私はSQLコールの追加コストを必要としなくても追加しています。だから、私はすべてのプロパティを関連するプライベートプロパティを持つように変更し、プログラムがpublicプロパティを呼び出すとプライベートプロパティをチェックし、nullの場合はSQLからデータをロードする必要があるのでロードメソッドを呼び出します。それは素晴らしい動作しますが、私はコードを参照すると、ヌルチェックの繰り返しパターンがあり、SQLクエリをロードします。これを行うより良い方法はありますか？

Answer 1

いいえ、これは正しいです。 Here is the wikipedia article。ヌル・チェックのオーバーヘッドは、不要なデータベース・コールに比べて非常に少なくなります。今、プログラムのユーザーが実際に時間の99％の値を使用している場合、私はこのパターンが必要ではないと言います。

単なる注意点：値がnullの可能性がある場合は、不要なデータベース呼び出しを行います。 （それがちょうどビットのチェックであることからも、迅速にチェックされます）、このような何かをした方がよいかもしれません：

//Constructor default to not loaded 
bool isLoaded = false; 

private string _name; 
public string Name 
{ 
    get { 
     if (!isLoaded) 
     LoadData(); //this popultes not just but all the properties 
     return _name; 
    } 
} 

private LoadData() 
{ 
    //Load Data 
    isLoaded = true; 
} 

Answer 2

さてあなたはそれを変更することができます：

if (!initialized) 
    LoadData(); 

そして、あなたの中にLoadDataはtrueに初期化されていますが、実際にはその意味を変更しません。

void EnsureData() 
{ 
     if (!dataLoaded) 
     LoadData(); //this populates all the properties 
} 

public string Name { 
    get { 
     EnsureData(); 
     return _name; 
    } 
} 

Answer 3

ことの一つは、その各プロパティは一つだけ追加の呼び出しを含む別の方法にあれば抽出することです。プロパティを使用しない場合、クラスをインスタンス化する理由は何ですか？私はあなたがコンストラクタコードの後に​​SQLを呼び出すのは実際にはもっときれいだと信じていますが、もしあなたがそれを使用しようとしているのであればクラスのオブジェクトを作成するだけです。他のより柔軟な解決策は、LoadDataをパブリックにし、必要に応じてオブジェクトインスタンスから必要に応じて呼び出すことです。

Answer 4

私はあなたのアプリケーションの構造を検討すべきだと思う：あなたが行うことができます

Answer 5

私はデザインパターンの学習プロセスに入っています。あなたがシングルトンのデザインパターンで試してみることができるデータを一度ロードすれば、私は一つの提案があります。

public class Singleton123 
{ 
    private static readonly string _property1 = ClassLoadData.LoadData(); 

    public static string MyProperty1 
    { 
     get 
     { 
      return _property1; 
     } 
    } 

} 

public class ClassLoadData 
{ 
    public static string LoadData() 
    { 
     // any logic to load data 
     return "test"; 
    } 
} 

コールプロパティこのプロパティは一度だけロードされます

Singleton123 obj = new Singleton123(); 
     string stra = Singleton123.MyProperty1; 
     string strb = Singleton123.MyProperty1; 

次のように。

Answer 6

Btw C＃はデフォルトのレイジーローダー実装を採用しました。 isSomethingLoadedフラグを提供する代わりに、なぜそれを使用しないのですか？ :)

public class Bar 
{ 
    private Lazy<string> _name = new Lazy<string>(() => LoadString()); 

    public string Name 
    { 
     get { return _name.Value; } 
    } 
} 

静的でないLoadStringメソッドの場合、遅延ローダーはコンストラクタで初期化する必要があります。