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())
                          )
                 );
}

优化LINQ查询

2 个答案: