エンティティフレームワークでの関連エンティティのフィルタリング

Question

関連する子エンティティの読み込みとフィルタリングの最良の方法を探しています。私はうまくいくものを持っていますが、私が望むものを達成するためにはそれが最良か、あるいは正しい方法であるかどうかは分かりません。以下の作業コードの例。長所と短所は素晴らしいだろう！ありがとう！

public Site Find(int siteID) 
{ 
    // Can't use include here, not possible to filter related (child) entities 
    // return _context.Sites.Where(x => x.ID == siteID) 
    //      .Include("SiteLoggers") 
    //      .Where(x => x.Deleted == false) 
    //      .FirstOrDefault(); 

    var site = _context.Sites.Where(x => x.ID == siteID).FirstOrDefault(); 

    if(site != null) 
    { 
     site.SiteLoggers = site.SiteLoggers.Where(x => x.SiteID == siteID && 
                 x.Deleted == false) 
              .ToList(); 
    } 

    return site; 
} 

EDIT：

追加POCOS

public class Site 
{ 
    public int ID { get; set; } 
    public int LocationID { get; set; } 
    public bool Active { get; set; } 
    public bool Deleted { get; set; } 
    public string Name { get; set; } 
    public virtual Location Location { get; set; } 
    public virtual ICollection<SiteLogger> SiteLoggers { get; set; } 
    public virtual ICollection<LinkDcSite> DcSiteLinks { get; set; } 
} 

public class SiteLogger 
{ 
    public int ID { get; set; } 
    public int UID { get; set; } 
    public int SiteID { get; set; } 
    public int LocationID { get; set; } 
    public string Name { get; set; } 
    public bool Active { get; set; } 
    public bool Deleted { get; set; } 
    public virtual Site Site { get; set; } 
    public virtual Location Location { get; set; } 
    public virtual ICollection<SiteLoggerSensor> SiteLoggerSensors { get; set; } 
    public virtual ICollection<LinkLoggerSiteLogger> LinkLoggerSiteLogger { get; set; } 
} 

Answer 1

単純なクエリであることあなたが行うことができます。

var site = _context.SiteLoggers.Where(sl => sl.SiteId = siteId && !sl.Deleted).ToList(); 

SiteLoggersとサイトとの間の関係がある場合、あなたは「ドンサイトが存在することを確認する必要はありません。あなたはIDによる検索場合も

.... 
site.SiteLoggers = site.SiteLoggers.Where(x => !x.Deleted).ToList(); 
.... 

それはSingleOrDefaultを使用することをお勧めしますので、あなたは、同じIDを持つどの2つの要素が存在しないことを確認している意味：

Answer 2

あなたの方法は、私はあなたがx.SiteID == siteIDのためだけの余分なチェックを持っていると思う結構ですFirstOrDefaultの代わりに、1つのIDを持つ複数の項目がある場合に例外をスローします。

var site = _context.Sites.Where(x => x.ID == siteID).SingleOrDefault();