我们可以使用 Skip()、Take() 和 OrderBy() 控制 LINQ 表达式顺序吗

Question

我正在使用 LINQ to Entities 来显示分页结果。但我在使用 Skip()、Take() 和 OrderBy() 调用的组合时遇到问题。

一切工作正常，除了 OrderBy() 分配得太晚。它在结果集被 Skip() 和 Take() 削减后执行。

因此，每页结果都有按顺序排列的项目。但排序是在一页数据上完成的，而不是对整个数据集进行排序，然后使用 Skip() 和 Take() 限制这些记录。

如何设置这些语句的优先级？

我的示例（简化）

var query = ctx.EntitySet.Where(/* filter */).OrderByDescending(e => e.ChangedDate);
int total = query.Count();
var result = query.Skip(n).Take(x).ToList();

一种可能（但不好）的解决方案

一种可能的解决方案是应用聚集索引按列排序，但该列经常更改，这会导致会降低插入和更新时的数据库性能。我真的不想这样做。

编辑

我在查询中运行了 ToTraceString() ，我们可以实际看到何时将 order by 应用于结果集。不幸的是最后。 :(

SELECT 
-- columns
FROM  (SELECT 
    -- columns
    FROM   (SELECT -- columns
        FROM ( SELECT 
            -- columns
            FROM table1 AS Extent1
            WHERE  EXISTS (SELECT 
                -- single constant column
                FROM table2 AS Extent2
                WHERE (Extent1.ID = Extent2.ID) AND (Extent2.userId = :p__linq__4)
            )
        )  AS Project2
        limit 0,10  ) AS Limit1
    LEFT OUTER JOIN  (SELECT 
        -- columns
        FROM table2 AS Extent3 ) AS Project3 ON Limit1.ID = Project3.ID
UNION ALL
    SELECT 
    -- columns
    FROM   (SELECT -- columns
        FROM ( SELECT 
            -- columns
            FROM table1 AS Extent4
            WHERE  EXISTS (SELECT 
                -- single constant column
                FROM table2 AS Extent5
                WHERE (Extent4.ID = Extent5.ID) AND (Extent5.userId = :p__linq__4)
            )
        )  AS Project6
        limit 0,10  ) AS Limit2
    INNER JOIN table3 AS Extent6 ON Limit2.ID = Extent6.ID) AS UnionAll1
ORDER BY UnionAll1.ChangedDate DESC, UnionAll1.ID ASC, UnionAll1.C1 ASC

Answer 1

我的解决方案

我已经成功解决了这个问题。不要误会我的意思。到目前为止我还没有解决优先级问题，但我已经减轻了它。

我做了什么？

这是我在从 Devart 得到答案之前一直使用的代码。如果他们无法解决这个问题，我最终将不得不使用此代码。

// get ordered list of IDs
List<int> ids = ctx.MyEntitySet
    .Include(/* Related entity set that is needed in where clause */)
    .Where(/* filter */)
    .OrderByDescending(e => e.ChangedDate)
    .Select(e => e.Id)
    .ToList();

// get total count
int total = ids.Count;

if (total > 0)
{
    // get a single page of results
    List<MyEntity> result = ctx.MyEntitySet
        .Include(/* related entity set (as described above) */)
        .Include(/* additional entity set that's neede in end results */)
        .Where(string.Format("it.Id in {{{0}}}", string.Join(",", ids.ConvertAll(id => id.ToString()).Skip(pageSize * currentPageIndex).Take(pageSize).ToArray())))
        .OrderByDescending(e => e.ChangedOn)
        .ToList();
}

首先，我得到了我的实体的有序 ID。即使对于较大的数据集，仅获取 ID 也具有良好的性能。 MySql 查询非常简单并且性能非常好。在第二部分中，我对这些 ID 进行分区并使用它们来获取实际的实体实例。

考虑到这一点，这应该比我一开始的方式执行得更好（如我的问题中所述），因为由于简化的查询，获得总计数要快得多。第二部分实际上非常相似，只是我的实体是通过 ID 返回的，而不是使用 Skip 和 Take 进行分区...

希望有人会发现这个解决方案有帮助。

Answer 2

我没有直接使用 Linq to Entities，但它应该有一种方法可以在需要时将特定的存储过程挂接到某些位置。 （Linq to SQL 就是这样做的。）如果是这样，您可以将此查询转换为存储过程，准确地执行所需的操作，并且高效地执行。

Answer 3

假设您的评论认为将值保留在列表中是不可接受的：

没有办法按照您的意图完全最小化迭代（正如我也尝试过的那样，生活在希望中）。将迭代次数减少 1 次就好了。是否可以只获取一次计数并缓存/会话它？然后你可以：

int total = ctx.EntitySet.Count;  // Hopefully you can not repeat doing this.
var result = ctx.EntitySet.Where(/* filter */).OrderBy(/* expression */).Skip(n).Take(x).ToList();

希望你能以某种方式缓存计数，或者避免每次都需要它。即使你做不到，这也是你能做的最好的事情了。

Answer 4

您能否创建一个示例来说明问题并将其发送给我们（支持 * devart * com，主题“EF：Skip、Take、OrderBy”）？
希望我们能够帮助您。
您还可以使用我们的论坛或联系表。

Answer 5

您完全确定订单已关闭吗？ SQL 是什么样的？

您可以按如下方式重新排序代码并发布输出吗？

// Redefine your queries. 
var query = ctx.EntitySet.Where(/* filter */).OrderBy(e => e.ChangedDate); 
var skipped = query.Skip(n).Take(x);

// let's look at the SQL, shall we?
var querySQL = query.ToTraceString();
var skippedSQL = skipped.ToTraceString();

// actual execution of the queries...
int total = query.Count(); 
var result = skipped.ToList(); 

编辑：

我绝对确定。您可以检查我的“编辑”以查看查询的跟踪结果，并跳过跟踪结果，这在这种情况下是必需的。计数其实并不重要。

是的，我看到了。哇，这真是一个难题。甚至可能是一个彻底的错误。我注意到你没有使用 SQL Server...你使用什么数据库？看起来可能是MySQl。

Answer 6

一种方法：

var query = ctx.EntitySet.Where(/* filter */).OrderBy(/* expression */).ToList();
int total = query.Count;
var result = query.Skip(n).Take(x).ToList();

在跳过之前将其转换为列表。请注意，这不是太有效...