Повышение эффективности метода

Question

Я написал следующий метод, который получает список и обновляет базу данных на основе определенных критериев:

public void UpdateInventoryGoods(List<InventoryGoods> list, int id) 
     { 
      int index = 0; 

      var query = from inventoryGoods in context.InventoryGoods 
         where inventoryGoods.ParentId == id 
         select inventoryGoods; 

      List<InventoryGoods> goodsList = query.ToList(); 

      using (var scope = new TransactionScope()) 
      { 
       foreach (InventoryGoods i in list) 
       { 
        foreach (InventoryGoods e in goodsList) 
        { 
         if (index == 30) 
         { 
          index = 0; 
          context.SubmitChanges(); 
         } 

         if (e.Gid == i.Gid && !getEventId(e.Id).HasValue && !e.ActionOn.HasValue) 
         { 
          e.Action = i.Action; 

         } 
         else if ((e.Gid == i.Gid && getEventId(e.Id).HasValue) && (e.Action != i.Action || i.ActionOn == DateTime.MinValue)) 
         { 
          e.Action = i.Action; 
          e.ActionOn = null; 

          var allEvents = from invent in context.InventoryGoodsEvents 
              where invent.InventoryGood == e.Id 
              select invent; 

          List<InventoryGoodsEvents> inventoryGoodsEventsList = allEvents.ToList(); 

          var events = from g in context.GoodsEvent           
             select g; 

          List<GoodsEvent> goodsEventList = events.ToList(); 

          foreach (InventoryGoodsEvents goodsEvent in inventoryGoodsEventsList) 
          { 
           context.InventoryGoodsEvents.DeleteOnSubmit(goodsEvent); 

           foreach (GoodsEvent ge in goodsEventList) 
           { 
            if (ge.Id == goodsEvent.EventId) 
            { 
             ge.IsDeleted = true; 
             ge.DeletedOn = DateTime.Now; 
             ge.DeletedBy = System.Web.HttpContext.Current.User.Identity.Name; 
            } 
           }         
          } 




         } 
         ++index; 
        } 
       } 
       context.SubmitChanges(); 
       scope.Complete(); 
      } 

     } 

     public int? getEventId(int InventoryGood) 
     { 


      var InventoryGoodsEvents = from i in context.InventoryGoodsEvents 
             where i.InventoryGood == InventoryGood 
             select i; 

      List<InventoryGoodsEvents> lst = InventoryGoodsEvents.ToList(); 

      if (lst.Count() > 0) 
      { 
       return lst[0].EventId; 
      } 
      else 
      { 
       return null; 
      } 
     } 

Хотя этот метод хорошо работает около 500 или 1000 объектов, она становится слишком медленно или в конце концов, раз когда я кормлю его более 8000 объектов или более. Итак, где я могу улучшить свою производительность?

Answer 1

Не вызывайте базу данных в цикле.

Попробуйте переместить запросы вне петли, как это:

public void UpdateInventoryGoods(List<InventoryGoods> list, int id) 
{ 
    int index = 0; 

    var query = from inventoryGoods in context.InventoryGoods 
       where inventoryGoods.ParentId == id 
       select inventoryGoods; 

    List<InventoryGoods> goodsList = query.ToList(); 

    using (var scope = new TransactionScope()) 
    { 
     var allEvents = from invent in context.InventoryGoodsEvents 
         where goodsList.Contains(invent.InventoryGood) 
         select invent; 

     List<InventoryGoodsEvents> inventoryGoodsEventsList = allEvents.ToList(); 

     var events = from g in context.GoodsEvent 
         select g; 

     List<GoodsEvent> goodsEventList = events.ToList(); 

     foreach (InventoryGoods i in list) 
     { 
      foreach (InventoryGoods e in goodsList) 
      { 
       if (index == 30) 
       { 
        index = 0; 
        context.SubmitChanges(); 
       } 

       var eventId = getEventId(e.Id); 
       if (e.Gid == i.Gid && !eventId.HasValue && !e.ActionOn.HasValue) 
       { 
        e.Action = i.Action; 

       } 
       else if ((e.Gid == i.Gid && eventId.HasValue) && (e.Action != i.Action || i.ActionOn == DateTime.MinValue)) 
       { 
        e.Action = i.Action; 
        e.ActionOn = null; 

        foreach (InventoryGoodsEvents goodsEvent in inventoryGoodsEventsList) 
        { 
         context.InventoryGoodsEvents.DeleteOnSubmit(goodsEvent); 

         foreach (GoodsEvent ge in goodsEventList) 
         { 
          if (ge.Id == goodsEvent.EventId) 
          { 
           ge.IsDeleted = true; 
           ge.DeletedOn = DateTime.Now; 
           ge.DeletedBy = System.Web.HttpContext.Current.User.Identity.Name; 
          } 
         } 
        } 
       } 
       ++index; 
      } 
     } 
     context.SubmitChanges(); 
     scope.Complete(); 
    } 
} 

Answer 2

Я не эксперт Linq, но я думаю, что вы, вероятно, может улучшить getEventId (должен быть капитал первой буквы кстати) с чем-то вроде

public int? GetEventId(int inventoryGood) 
{ 
    var firstInventoryGoodsEvent = context.InventoryGoodsEvents 
        .Where(i => i.InventoryGood == inventoryGood) 
        .FirstOrDefault(); 

    // ...etc 
} 

Использование FirstOrDefault() означает, что вы не обрабатывать весь список если вы найдете соответствующий элемент.

Возможно, существуют другие оптимизации, но довольно сложно следить за тем, что вы делаете. В качестве примера:

foreach (InventoryGoods i in list) 
{ 
    foreach (InventoryGoods e in goodsList) 
    { 
    } 
} 

i и e не придают большого значения здесь. Для вас может быть очевидно, что они означают, но они не очень описательны для тех, кто никогда не видел ваш код раньше. Аналогично, list не является лучшим именем для списка. Список чего? Ваше имя переменной должно описывать ее цель.

Edit:

Я не уверен, что ни о чем другом. Кажется, вы используете ToList() в нескольких местах, где, насколько я вижу, это необязательно. Я не знаю, какое влияние это оказало бы на производительность, но кто-то умнее меня мог бы рассказать вам.

Вы также можете попробовать грузоподъемных некоторые из ваших ценностей за пределами контуров, например:

foreach (foo) 
{ 
    foreach (bar) 
    { 
     DeletedOn = DateTime.Now; 
     DeletedBy = System.Web.HttpContext.Current.User.Identity.Name; 
    } 
} 

может быть переписана в виде

var deletedOn = DateTime.Now; 
var deletedBy = System.Web.HttpContext.Current.User.Identity.Name; 

foreach (foo) 
{ 
    foreach (bar) 
    { 
     DeletedOn = deletedOn; 
     DeletedBy = deletedBy; 
    } 
} 

Опять же, я не знаю, сколько разница если это будет сделано, вам нужно будет протестировать его и посмотреть.

Answer 3

Это не собирается в партиях от 30, это происходит в партиях от 1

Там в запрос без каких-либо критериев, поэтому она загружает всю таблицу. Это ваше намерение?

getEventId (e.Id) возвращает согласованное значение. Не называть его дважды (за цикл).