优化LINQ查询

Question

我有一个报告，显示了一个确定的商家订单，并且它工作正常，直到我需要添加一个过滤器的付款状态。

这是我如何通过过滤器生成查询，过滤器：

var queryOrder = context.Orders.Select(m=>m); 

if (viewModel.InitialDate.HasValue) 
    queryOrder = queryOrder.Where(m => m.CreatedDate.Date >= viewModel.InitialDate.Value); 

(...) /* continues building the query, filter by filter */ 


if (viewModel.SelectedPaymentStatus != null) 
    queryOrder = queryOrder.Where(m => viewModel.SelectedPaymentStatus.Contains(m.Payments.Select(p => p.PaymentStatusId).Single().ToString())); 

queryOrder = queryOrder.Where(m => m.MerchantId == merchantId); 

当我运行queryOrder，哪怕它只是一个queryOrder.Count()，它需要1分钟来执行。使用SQL Server的性能分析工具，我提取生成的查询是这样的：

SELECT [t0].[Id], [t0].[CustomerId], [t0].[MerchantId], [t0].[OrderNumber], [t0].[Amount], [t0].[SoftDescriptor], [t0].[ShippingMethod], [t0].[ShippingPrice], [t0].[IpAddress], [t0].[SellerComment], [t0].[CreatedDate] 
FROM [dbo].[Order] AS [t0] 
WHERE ([t0].[MerchantId] = @p0) 
AND ((CONVERT(NVarChar,(
     SELECT [t1].[PaymentStatusId] 
     FROM [dbo].[Payment] AS [t1] 
     WHERE [t1].[OrderId] = [t0].[Id] 
    ))) IN (@p1, @p2, @p3, @p4, @p5, @p6, @p7, @p8)) 

的@ P0参数是MERCHANTID一个GUID和@ P1直通@ P8是数字串"1"通"8"，代表paymentStatusId的。

如果我跳过行：

if (viewModel.SelectedPaymentStatus != null) 
    queryOrder = queryOrder.Where(m => viewModel.SelectedPaymentStatus.Contains(m.Payments.Select(p => p.PaymentStatusId).Single().ToString())); 

查询运行在不到1秒钟。但是当我使用它时，表演击中了地板。有关如何解决此问题的任何提示？

Answer 1

所有的查询都被延迟，这既是linq的好/坏部分。尝试拆分查询并使用一些内存中的结果。 尝试删除第一个查询（实际上没什么意义，您将返回相同的集合），并将第二个查询修改为这样，并查看它是否有任何区别。

var clause = context.Orders.Payments.Select(p => p.PaymentStatusId).Single().ToString(); 
if (viewModel.SelectedPaymentStatus != null) 
    var queryOrder = context.Orders.queryOrder.Where(m => viewModel.SelectedPaymentStatus.Contains(clause)); 

Answer 2

我注释的问题摘录：

if (viewModel.SelectedPaymentStatus != null) { 

    // Give me only orders from queryOrder where... 
    queryOrder = queryOrder.Where(

     // ...my viewModel's SelectedPaymentStatus collection... 
     m => viewModel.SelectedPaymentStatus.Contains(

       // ...contains the order's payment's PaymentStatusId... 
       m.Payments.Select(p => p.PaymentStatusId).Single() 

              // ... represented as a string?! 
             .ToString() 
              // Why are database IDs strings? 
      ) 
     ); 
} 

viewModel.SelectedPaymentStatus似乎是字符串的集合;因此，您要求数据库将PaymentStatusId转换为nvarchar，并使用SelectedPaymentStatus的元素进行字符串比较。呸。

由于viewModel.SelectedPaymentStatus小，它可能是最好创建一个临时List<int>和使用，在您的查询：

if (viewModel.SelectedPaymentStatus != null) { 

    // Let's do the conversion once, in C# 
    List<int> statusIds = viewModel.SelectedPaymentStatus.Select(i => Convert.ToInt32(i)).ToList(); 

    // Now select the matching orders 
    queryOrder = queryOrder.Where(
        m => statusIds.Contains(
           m.Payments.Select(p => p.PaymentStatusId).Single()) 
         ) 
       ); 
}