コンストラクタで仮想関数を呼び出すのを避けるには？

Question

ほとんどの場合（すべてではないにしても）、私のEntity Framework POCOに仮想関数があります。エンティティが遅延ロードされるようにこれらの関数を仮想的にする必要があります。

コンストラクタでAccommodationsを初期化すると、コンストラクタで仮想関数が呼び出されますが、これは悪い習慣です。

しかし、コンストラクタでない場合、どうすればAccommodationsを初期化できますか？遅延ロードで

public class Venue 
{ 
    public Venue() 
    { 
     Accommodations = new HashSet<Accommodation>(); 
    } 

    public virtual ICollection<Accommodation> Accommodations { get; set; } 
} 

Answer 1

public class Venue 
{ 
    private accommodations_ = new HashSet<Accommodation>(); 

    public Venue() { } 

    public virtual ICollection<Accommodation> Accommodations 
    { 
     get { return accommodations_; } 
     set { accommodations_ = value; } 
    } 
} 

Answer 2

、あなたもそれが最初にアクセスするので、nullそれを残していますAccommodationsそれまでは初期化されません。

あなたはそれが自動に自分自身を初期化するために、次のようにラップすることができます：

private ICollection<Accommodation> _accommodations; 

public virtual ICollection<Accommodation> Accommodations { 
    get { 
     if (_accommodations == null) 
     { 
      // Initialize or load data here. 
      _accommodations = new HashSet<Accommodation>(); 
     } 
     return _accomodations; 
    } 
    set { 
     _accommodations = value; 
    } 
} 

---

このソリューションに関する以下のコメントを必ずお読みください！

Answer 3

もう1つのオプションは、セッターをプライベートとしてマークすることです。これはコンストラクターで仮想メンバーを呼び出す問題をなくします。

これを実行したら、発信者（EF以外）がデザインで必要に応じてそのプロパティを設定する方法を提供する必要があります。オーバーロードされたコンストラクタを使用して宿泊施設のリストを渡すか、オプションでコレクション（ドメイン駆動型デザイン）をカプセル化し、アイテムを追加/削除するメソッドを使用できます（EFでは完全にカプセル化されていないためNHibernateのとは異なり、コレクション、）：それはプライベート変数を使用せずに簡単にすることができ、C＃6.0のデフォルトの初期設定では

public class Venue 
{ 
    public Venue() 
    { 
     Accommodations = new HashSet<Accommodation>(); 
    } 

    public Venue(ICollection<Accommodation> accommodations) 
    { 
     Accommodations = new List<Accommodation>(accommodations); 
    } 

    public virtual ICollection<Accommodation> Accommodations { get; private set; } 
} 

Answer 4

：

public class Venue 
{ 
    public virtual ICollection<Accommodation> Accommodations {get; set; } = new HashSet<Accommodation>(); 
}