これは、複数のテーブル間でデータを比較する非効率的な方法ですか？

Question

次のコードを使用して、文字列がデータベースの列のどこかにあるかどうかを最初に確認しています。そうであれば、データベースのさまざまな部分を調べることによって、いくつかの追加基準が満たされているかどうかを確認する必要があります（下記のコードを参照）。私は、これはこれを行うための効率的な方法があるかどうかわからないですか、はるかに簡単な方法があるとします（私のコントローラから）

[HttpPost] 
    public ActionResult Index(FormCollection sampleKey) 
    { 
     string code = sampleKey["sampleCode"]; 
     ViewBag.code = code; 
     // Need to check if this code is active 
     var order = db.Orders.SingleOrDefault(
      o => o.OrderCode == code 
       && o.Active == true); 
     if (order == null) 
     { 
      //Invalid 
     } 
     else 
     { 
      var orderIdent = db.OrderDetails.SingleOrDefault(
       p => p.OrderDetailId == order.OrderId); 
      var barIdent = db.Drink.SingleOrDefault(
       q => q.EstablishmentsID == orderIdent.DrinksId); 
      var barName = db.Establishment.SingleOrDefault(
       r => r.EstablishmentsId == barIdent.EstablishmentsID); 
      ViewBag.barId = barName.name; 
      ViewBag.sample = order.Email; 
      var custProfile = CustomProfile.GetUserProfile(); 
      if (custProfile.OwnedBar != barName.name) 
      { 
       //Not a match 
      } 
      else 
      { 
       //Match 
      } 
     } 
     return View(); 
    } 

は心配するには、この何かありますか？私が現在行っている行動をより効率的に実行する方法はありますか？テーブルからデータをインクルードするために参照される最初のテーブルを変更する必要がありますか？異なるテーブルの情報を比較する非効率的な方法を避けるために、最終的に比較しますか？

Answer 1

生成されるSQLクエリを確認する必要があります。たとえば、次のようにすることができます。コンソールにクエリを出力します。これは、db.Log = Console.Out;を設定して行います。あなたのケースでWebページに出力するのと同様の方法があるはずです。 LINQの怠惰な性質は、物事を予測することを困難にします。

テーブル間に外部キーの関係を作成すると、OrderDetailsがOrders.OrderIdのFKになります。これにより、Entity Frameworkはデータベースのナビゲーションプロパティを生成できます。あなたのコードは次のようになります：

[HttpPost] 
public ActionResult Index(FormCollection sampleKey) 
{ 
    string code = sampleKey["sampleCode"]; 

    var detail = db.Orders.Where(o => o.OrderCode == code && o.Active == true) 
     .Select(o => new { 
      OrderCode = o.OrderCode, 
      BarId = o.Drink.Establishment.Select(n => n.name), 
      Sample = o.Email 
      }) 
     .SingleOrDefault(); 

    if (detail != null) 
    { 
     ViewBag.code = detail.OrderCode; 
     ViewBag.barId = detail.BarId; 
     ViewBag.sample = detail.Sample; 

     var custProfile = CustomProfile.GetUserProfile(); 
     if (custProfile.OwnedBar == detail.BarId) 
     { 
      //Match 
     } 
     else 
     { 
      //Not a match 
     } 
    } 
    else 
    { 
     //Invalid 
    } 

    return View(); 
}