3

我们使用实体框架和“POCO”生成器来创建我们的类型，这些类型可以传递到各个层，它们属于（为了保护无辜的）命名空间 Company.Application.Project.Module。这些 POCO 对象都继承自一个为我们处理一些基本内容的基类。

我想编写一个函数，可以获取这些对象的集合并按属性名称对它们进行排序。

我编写了以下函数——它完成了我想做的事情，但我不喜欢它有几个原因：

1）这不适用于任何对象类型，它必须是 SortHelper 类知道的对象类型（因此最后一个 using 语句）。

2）“POCO”对象的 Type 和 BaseType 似乎不一致 - 取决于您在应用程序中调用此函数的位置（单元测试项目与从我们的 MVP 应用程序的演示者对象调用），这会导致线路出现问题我加粗了，因为如果它抓取了错误的类型，属性就不会出现在下面的行中。

从演示者对象中，.GetType 显示为：ClassName_96D74E07A154AE7BDD32624F3B5D38E7F50333608A89B561218F854513E3B746 ...在 System.Data.Entity.DynamicProxies 命名空间内。

这就是为什么代码在该行上显示 .GetType().BaseType 的原因，这给了我： ClassName ...within Company.Application.Project.Module

但在单元测试中， .GetType() 在 Company.Application.Project.Module 中显示为 ClassName

BaseType 在 Company.Application.Project.Module 中显示为 BaseClass

...这更有意义，但我不理解不一致 - 而且不一致让我害怕。

3）一般讨厌使用反射来做到这一点。

如果有人有更好的方法来做到这一点，或者甚至是使反射与命名空间/类型一起运行的修复程序——我当然会很感激！

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Reflection;
using Company.Application.Project.Module;

namespace Company.Application.Project
{
    public static class SortHelper
    {
        public static IOrderedQueryable<T> Sort<T>(this IQueryable<T> source, string propertyName, bool descending)
        {

            // bail out if there's nothing in the list
            if (source == null)
            {
                return null;
            }
            if (source.Count() == 0)
            {
                return source as IOrderedQueryable<T>;
            }

            // get the type -- or should it be the BaseType?  Nobody knows!
            Type sourceType = source.First().GetType().BaseType;

            // this works fine _assuming_ we got the correct type on the line above
            PropertyInfo property = sourceType.GetProperty(propertyName);
            if (descending)
            {
                return source.OrderByDescending(e => property.GetValue(e, null));
            }
            else
            {
                return source.OrderBy(e => property.GetValue(e, null));
            }

        }
    }
}
4

3 回答 3

3

在我看来，根本没有充分的理由这样做。你OrderBy在你的函数中使用了扩展方法，为什么不直接调用它而不是这个函数呢？除非您的示例中未显示某些内容，否则该函数除了弄清楚如何调用OrderBy（或OrderByDescending）然后调用它之外没有做任何事情。

如果只是您想使用布尔标志在升序和降序之间切换（而不必在您尝试对某些内容进行排序的每个地方都使用 if 语句），那么我强烈建议您使用以下方法：

public static IOrderedQueryable<TSource> Sort<TSource, TKey>(this IQueryable<TSource> source, Expression<Func<TSource, TKey>> keySelector, bool descending = false)
{
  if (descending) return source.OrderByDescending(keySelector);
  else return source.OrderBy(keySelector);
}

它整洁干净，您仍然可以使用 lambda 表达式来定义键，而无需依赖于您的模型，您可以（可选地）使用布尔标志将排序顺序更改为降序，并且没有反射。那是双赢！;)

[开始编辑以回应评论]

好的，现在我已经让 VS2010 再次运行，我能够按照承诺找到一种方法来封装排序 ASP.NET GridView 控件时使用的 SortExpression。所以就在这里。

由于我们没有为这个问题定义模型，我将创建一个简单的模型。

public class Customer
{
  public int CustomerID { get; set; }
  public string Name { get; set; }
  public virtual List<Order> Orders { get; set; }
}

public class Order
{
  public int OrderID { get; set; }
  public DateTime OrderDate { get; set; }
  public decimal TotalPurchaseAmount { get; set; }
  public string Comments { get; set; }
  public bool Shipped { get; set; }
  public virtual Customer Customer { get; set; }
}

那么让我们假设我们有一个 CustomerDetails.aspx 页面，我们希望向该页面添加一个 GridView 来列出订单。

<asp:GridView ID="gvOrders" AutoGenerateColumns="false" runat="server"
  AllowSorting="true" OnSorting="gvOrders_Sorting">
  <Columns>
    <asp:BoundField DataField="OrderID" SortExpression="OrderID" HeaderText="Order ID" />
    <asp:BoundField DataField="OrderDate" SortExpression="OrderDate" HeaderText="Order Date" />
    <asp:BoundField DataField="TotalPurchaseAmount" SortExpression="TotalPurchaseAmount" HeaderText="Total Purchase Amount" />
    <asp:BoundField DataField="Comments" SortExpression="Comments" HeaderText="Comments" />
    <asp:BoundField DataField="Shipped" SortExpression="Shipped" HeaderText="Shipped" />
  </Columns>
</asp:GridView>

在后面的代码中，有一个静态字典对象：

protected static readonly Dictionary<string, Expression<Func<Order, object>>> sortKeys = new Dictionary<string,Expression<Func<Order,object>>>()
{
  { "OrderID", x => x.OrderID },
  { "OrderDate", x => x.OrderDate },
  { "TotalPurchaseAmount", x => x.TotalPurchaseAmount },
  { "Comments", x => x.Comments },
  { "Shipped", x => x.Shipped }
};

然后是处理排序的函数：

protected void gvOrders_Sorting(object sender, GridViewSortEventArgs e)
{
  string exp = ViewState["gvOrders_SortExp"] as string;
  if (exp == null || exp != e.SortExpression)
  {
    e.SortDirection = SortDirection.Ascending;
    ViewState["gvOrders_SortExp"] = e.SortExpression;
    ViewState["gvOrders_SortDir"] = "asc";
  }
  else
  {
    string dir = ViewState["gvOrders_SortDir"] as string;
    if (dir == null || dir == "desc")
    {
      e.SortDirection = SortDirection.Ascending;
      ViewState["gvOrders_SortDir"] = "asc";
    }
    else
    {
      e.SortDirection = SortDirection.Descending;
      ViewState["gvOrders_SortDir"] = "desc";
    }
  }
  if (e.SortDirection == SortDirection.Ascending)
  // There's a MyCustomer property on the page, which is used to get to the Orders
  { gvOrders.DataSource = MyCustomer.Orders.OrderBy(sortKeys[e.SortExpression]); }
  else
  { gvOrders.DataSource = MyCustomer.Orders.OrderByDescending(sortKeys[e.SortExpression]); }
  gvOrders.DataBind();
}

在那里，它做到了。

我承认该gvOrders_Sorting方法确实很混乱，但这就是使用自定义排序和GridView. 它可以进一步封装，但我决定保持简单。（实际上我开始着手创建一个可以处理排序和分页的辅助类，但是 .NET 4.0 中的自定义分页比自定义排序更糟糕！）

希望你喜欢。;)

[更正]

我刚刚注意到编写的代码存在问题。MyCustomer.Orders将是一个List<Order>不是一个IQueryable<Order>。因此，要么将 更改sortKeys为 a Dictionary<string, Func<Order, object>>，要么将调用更改为MyCustomer.Orders.AsQueryable().OrderBy(sortKeys[e.SortExpression])或MyDbContext.Orders.Where(o => o.Customer == MyCustomer).OrderBy(sortKeys[e.SortExpression])。我把选择权留给你。;)

于 2013-07-10T20:04:59.283 回答
2

如果它们都继承自基类，为什么不限制泛型参数呢？

public static IOrderedQueryable<T> Sort<T>(this IQueryable<T> source, string propertyName, bool descending) where T : MyBase

这样，编译器不会让你将一个类传递给这个不继承自 的函数MyBase，并且你就脱离了“没人知道？！” 领土 ：）

于 2013-07-10T19:11:27.830 回答
1

只需投影要排序的属性：

 public static IOrderedQueryable<T> Sort<T, U>(this IQueryable<T> source, Func<T, U> sortExpr, bool descending)

...


source.OrderBy(e => sortExpr(e));
于 2013-07-10T19:18:36.327 回答