IQueryable<T>.OrderBy<T, object> 抛出：无法将类型“System.Int16”转换为类型“System.Object”

Question

我们正在使用一个 GenericCriteria 类，该类用于查询 IQueryable 而无需直接访问它。基本上，查询构建在“业务”层并传递到“数据访问”层。为了防止重复，它需要尽可能通用。

这是 GenericCriteria 类的简化版本：

public class GenericCriteria<T>
{
    public Expression<Func<T, bool>> Where { get; set; }
    public Expression<Func<T, object>> OrderBy { get; set; }
}

解析条件是通过 IQueryable 上的扩展方法完成的：

public static IQueryable<T> ResolveCriteria<T>(this IQueryable<T> query, GenericCriteria<T> criteria)
{
    query = query.Where(criteria.Where);
    query = query.OrderBy(criteria.OrderBy);
    return query;
}

这是它在数据层中的解析方式：

public ICollection<Person> GetPersons(GenericCriteria<Person> criteria)
{
    using (var context = new EFContext())
    {
        return context.Persons
            .AsQueryable()
            .ResolveCriteria(criteria)
            .ToList();
    }
}

其用法示例：

var criteria = new GenericCriteria<Person>();
criteria.Where = c => c.Age > 20;
criteria.OrderBy = c => c.Age;

var persons = Data.Instance.GetPersons(criteria);

问题是Entity Framework无法处理OrderBy表达式的object类型，抛出异常：

Unable to cast the type 'System.Int16' to type 'System.Object'

所以我现在想弄清楚的是如何让 GenericCriteria 类接受某种通用 OrderBy 构造，以便 EF 能够完成它的工作。

我应该寻找什么？

更新：

如果不清楚，我宁愿避免：

public class GenericCriteria<T, TKey>
{
    public Expression<Func<T, bool>> Where { get; set; }
    public Expression<Func<T, TKey>> OrderBy { get; set; }
}

因为该类的用户可能并不总是想要订购，所以要求一个类型似乎有点脏。

Answer 1

既然你不希望GenericCriteria 有两个类型参数，而OrderBy 坚持两个类型参数，也许最好的方法是在ResolveCriteria 中做一些肮脏的工作，如下所示：

public static IQueryable<T> ResolveCriteria<T>(this IQueryable<T> query, GenericCriteria<T> criteria)
{
  query = query.Where(criteria.Where);

  var t = criteria.OrderBy;
  var b = t.Body;
  if (b.NodeType == ExpressionType.Convert &&
      ((UnaryExpression)b).Type == typeof(Object)) {
    // Handle simple types, such as short, int, long, etc.
    var bb = ((UnaryExpression)b).Operand;
    var tt = Expression.Lambda(bb, t.Parameters);
    if (bb.Type == typeof(short))
      query = query.OrderBy((Expression<Func<T, short>>)tt);
    else if (bb.Type == typeof(int))
      query = query.OrderBy((Expression<Func<T, int>>)tt);
    else if (...)
      ...
  } else
    // Handle non-simple types, such as string.
    query = query.OrderBy(t);

  return query;
}

每个简单类型都需要一个“else if”；幸运的是没有很多。简单类型列表请参考C# Simple Types。

Answer 2

我将重写Criteria如下，

public class GenericCriteria<T>
{
     public List<Func<IQueryable<T>,IQueryable<T>>> 
          List { get; private set;}

     public GenericCriteria<T>(){
         List = new List<Func<IQueryable<T>,IQueryable<T>>>();
     }

}

解决

public static IQueryable<T> ResolveCriteria<T>(this IQueryable<T> 
       query, GenericCriteria<T> criteria)
{
    foreach(var exp in criteria.List){
       query = exp(query);
    }
    return query;
}

用法

var criteria = new GenericCriteria<Person>();
criteria.List.Add( q => q.Where( c => c.Age > 20) );
criteria.List.Add( q => q.OrderBy( c => c.Age ));

var persons = Data.Instance.GetPersons(criteria);