Как проверить наличие OrderBy в дереве выражения ObjectQuery <T>

Боюсь, это немного сложнее. Видите ли, Entity Framework при определенных обстоятельствах будет молча игнорировать OrderBy., поэтому недостаточно просто искать OrderBy в дереве выражений. OrderBy должен находиться в «правильном» месте, а определение «правильного» места является деталью реализации Entity Framework.

Как вы уже догадались, я нахожусь в том же месте, что и вы; Я использую шаблон репозитория сущностей и делаю Take / Skip на уровне представления. Решение, которое я использовал, которое, возможно, не идеально, но достаточно хорошо для того, что я делаю, состоит в том, чтобы не выполнять никаких заказов до последнего возможного момента, чтобы гарантировать, что OrderBy всегда будет последним элементом в дереве выражений. Таким образом, любое действие, которое будет выполнять Take / Skip (прямо или косвенно), сначала вставляет OrderBy. Код структурирован так, что это может произойти только один раз.

Пейджинг во многом зависит от порядка. Почему бы не объединить операции плотно? Вот один из способов сделать это:

Объекты поддержки

public interface IOrderByExpression<T>
{
  ApplyOrdering(ref IQueryable<T> query);
}

public class OrderByExpression<T, U> : IOrderByExpression<T>
{
  public IQueryable<T> ApplyOrderBy(ref IQueryable<T> query)
  {
    query = query.OrderBy(exp);
  }
  //TODO OrderByDescending, ThenBy, ThenByDescending methods.

  private Expression<Func<T, U>> exp = null;

  //TODO bool descending?
  public OrderByExpression (Expression<Func<T, U>> myExpression)
  {
    exp = myExpression;
  }
}

Обсуждаемый метод:

public IQueryable<Category> List(int startIndex, int count, IOrderByExpression<Category> ordering)
{
    NorthwindEntities ent = new NorthwindEntities();
    IQueryable<Category> query = ent.Categories;
    if (ordering == null)
    {
      ordering = new OrderByExpression<Category, int>(c => c.CategoryID)
    }
    ordering.ApplyOrdering(ref query);

    return query.Skip(startIndex).Take(count);
}

Некоторое время спустя вызов метода:

var query = List(20, 20, new OrderByExpression<Category, string>(c => c.CategoryName));

    ProvideDefaultSorting(ref query);
    if (!IsSorted(query))
    {
            query = query.OrderBy(c => c.CategoryID);
    }

Измените на:

    //apply a default ordering
    query = query.OrderBy(c => c.CategoryID);
    //add to the ordering
    ProvideDefaultSorting(ref query);

Это не идеальное решение.

Это не решает проблему "фильтра в функции упорядочивания", которую вы указали. Это решает проблему «Я забыл оформить заказ» или «Я решил не заказывать».

Я тестировал это решение в LinqToSql:

    public void OrderManyTimes()
    {
        DataClasses1DataContext myDC = new DataClasses1DataContext();
        var query = myDC.Customers.OrderBy(c => c.Field3);
        query = query.OrderBy(c => c.Field2);
        query = query.OrderBy(c => c.Field1);

        Console.WriteLine(myDC.GetCommand(query).CommandText);

    }

Генерирует (обратите внимание на обратный порядок следования):

SELECT Field1, Field2, Field3
FROM [dbo].[Customers] AS [t0]
ORDER BY [t0].[Field1], [t0].[Field2], [t0].[Field3]

Вы можете решить эту проблему в возвращаемом типе ProvideDefaultSorting. Этот код не строит:

    public IOrderedQueryable<int> GetOrderedQueryable()
    {
        IQueryable<int> myInts = new List<int>() { 3, 4, 1, 2 }.AsQueryable<int>();
        return myInts.Where(i => i == 2);
    }

Этот код строится, но коварен, и программист получает то, что заслуживает.

    public IOrderedQueryable<int> GetOrderedQueryable()
    {
        IQueryable<int> myInts = new List<int>() { 3, 4, 1, 2 }.AsQueryable<int>();
        return myInts.Where(i => i == 2) as IOrderedQueryable<int>;
    }

Та же история с ref (это не сборка):

    public void GetOrderedQueryable(ref IOrderedQueryable<int> query)
    {
        query = query.Where(i => i == 2);
    }

Благодаря Дэвиду Б. у меня есть следующее решение. (Мне пришлось добавить обнаружение ситуации, когда частичный метод не выполнялся или просто возвращал его параметр).

public partial class Repository
{
    partial void ProvideDefaultSorting(ref IOrderedQueryable<Category> currentQuery);

    public IQueryable<Category> List(int startIndex, int count)
    {
        NorthwindEntities ent = new NorthwindEntities();
        IOrderedQueryable<Category> query = ent.CategorySet;
        var oldQuery = query;
        ProvideDefaultSorting(ref query);
        if (oldQuery.Equals(query)) // the partial method did nothing with the query, or just didn't exist
        {
            query = query.OrderBy(c => c.CategoryID);
        }
        return query.Skip(startIndex).Take(count);
    }
    // the rest..        
}

public partial class Repository
{
    partial void ProvideDefaultSorting(ref IOrderedQueryable<Category> currentQuery)
    {
        currentQuery = currentQuery.Where(c => c.CategoryName.Contains(" ")).OrderBy(c => c.CategoryName); // compile time forced sotring
    }
}

Во время компиляции он гарантирует, что, если частичный метод реализован, он должен, по крайней мере, сохранить его как IOrderdQueryable.

И когда частичный метод не реализован или просто возвращает свой параметр, запрос не будет изменен, и будет использоваться резервная сортировка.

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

См. Обсуждение и универсальный код в http://johnkaster.wordpress.com/2011/05/19/a-bug-fix-for-system-linq-dynamic-and-a-solution-for-the-entity-framework-4-skip-problem/. (И случайное исправление ошибки для Dynamic LINQ.)