如何优化我的实体框架查询以获得更好的性能和更高的可读性

Question

我的网站中有一个沉重的页面，名为HotelDetail，与酒店相关的几件事将在此页面加载。酒店，酒店的服务，酒店的目的地，用户的评论，酒店的房间与他们的服务，价格和容量。 所有这些都将在一段时间内加载下面的代码，现在我想优化这段代码，有什么解决方案来优化它以获得更好的性能和更好的可读性？有没有办法在一次不加载所有内容？

public PlaceViewModel GetPlaceDetail(string languageKey,string catKey , string placeKey, SearchViewModel searchOptions) 
     { 
       DateTime startDate = searchOptions.CheckIn.ToMiladiDateTime(); 
       DateTime endDate = searchOptions.CheckOut.ToMiladiDateTime(); 
       int nights = (int)endDate.Subtract(startDate).TotalDays; 
       placeViewModel.NightCount = nights; 
       var query = (from r in _unitOfWork.RoomServiceRepository.Get() 
          join 
           a in _unitOfWork.InventoryRepository.Get() 
           on r equals a.RoomService 
          where r.Room.Place.Id == place.Id && !r.SoftDelete && !r.Room.SoftDelete && 
            a.Date >= startDate && a.Date < endDate && (a.CertainAvailability + a.FloatAvailability) > 0 
          select new { a, MaxCapacity = r.Room.Capacity + r.ExtraCapacity, r.RoomId }); 

       placeViewModel.HasAvailability = query.ToList().Count != 0; 

       var grp = query.GroupBy(y => y.a.RoomServiceId) 
        .Select(z => new { 
         key = z.Key, 
         count = z.Count(), 
         minAvail = z.Min(ax => ax.a.CertainAvailability + ax.a.FloatAvailability), 
         minDate = z.Min(y => y.a.Date), 
         minPrice = z.Min(ax=>ax.a.Price), 
         price = z.Average(ax=>ax.a.Price) }); 

       var tmp = query.Where(x => grp.Any(q => 
        q.key == x.a.RoomServiceId && 
        q.count == nights && 
        q.minDate == x.a.Date)).ToList(); 
       foreach (var item in tmp) 
       { 
        var roomViewModel = placeViewModel.Rooms.FirstOrDefault(x => x.Id == item.RoomId); 
        var avail = new AvailabilityViewModel(item.a, item.MaxCapacity); 
        avail.Availability = grp.FirstOrDefault(x => x.key == item.a.RoomServiceId).minAvail; 
        var otherInfos = grp.SingleOrDefault(x => x.key == item.a.RoomServiceId); 
        avail.JabamaPrice = otherInfos.price; 
        avail.JabamaMinPriceInPeriod = otherInfos.minPrice; 
        roomViewModel.Availabilites.Add(avail); 
        if (maxDiscount == null || (maxDiscount != null &&(maxDiscount.BoardPrice - maxDiscount.JabamaPrice) < (avail.BoardPrice - avail.JabamaPrice))) 
         maxDiscount = avail; 
        if (minPrice == null || (minPrice != null && avail.JabamaPrice > 0 && minPrice.JabamaPrice > avail.JabamaPrice)) 
         minPrice = avail; 
       } 
       var discountQuery = tmp.Where(x => x.a.RoomService.ExtraCapacity == 0 && x.a.Date == minPrice.Date); 
       try 
       { 
        if (discountQuery.Any()) 
         maxDiscountOfMinPercentageDay = tmp != null && minPrice != null ? discountQuery.Max(x => (x.a.BoardPrice - x.a.Price)/x.a.BoardPrice * 100) : 0; 
        else 
         maxDiscountOfMinPercentageDay = 0; 

       } 
       catch (Exception) 
       { 
        maxDiscountOfMinPercentageDay = 0; 
       } 
       foreach (var roomVM in placeViewModel.Rooms) 
       { 
        if (roomVM.Availabilites.Count() == 0) 
         roomVM.Availabilites.Add(new AvailabilityViewModel(-1)); 
       } 
} 

Answer 1

我很不熟悉Linq，但我知道一些SQL的缺陷。我注意到的一件事是，你（以及其他许多人）使用.Count != 0来确定是否可能有匹配的记录。在SQL中，这是一种非常低效的方法，因为系统将有效地遍历整个表来查找和计数符合先决条件的记录。在这种情况下，我建议使用WHERE EXISTS()结构;我相信林克有类似的作品.Any()。

这也是你在现实生活中所做的事情：当有人问你是否有冰箱里剩下的奶酪时，你就不会开始计算你可以在那里找到的所有类型的奶酪，而是在第一类你看到的奶酪并回答'是的';通过整个冰箱没有附加价值，只会花费额外的时间。

PS：这大概也是如此的roomVM.Availabilites.Count() == 0这可能与反转逻辑可以更好地处理...

免责声明：这可能是因为EntityFwk足够聪明，在后台优化这个给你。 .. 我不知道;我非常怀疑它，但就像我说的，我不是专家。我建议首先确定当前代码的每个部分需要多长时间，然后优化最慢的代码。有基线也给你一个更好的主意，如果你的变化有任何影响，无论是积极的还是消极的。