LINQ OrderBy против ThenBy

Может ли кто-нибудь объяснить, в чем разница между:

tmp = invoices.InvoiceCollection
              .OrderBy(sort1 => sort1.InvoiceOwner.LastName)
              .OrderBy(sort2 => sort2.InvoiceOwner.FirstName)
              .OrderBy(sort3 => sort3.InvoiceID);

и

tmp = invoices.InvoiceCollection
              .OrderBy(sort1 => sort1.InvoiceOwner.LastName)
              .ThenBy(sort2 => sort2.InvoiceOwner.FirstName)
              .ThenBy(sort3 => sort3.InvoiceID);

Какой правильный подход, если я хочу упорядочить по 3 элементам данных?

Вам определенно следует использовать, ThenByа не несколько OrderByвызовов.

Я бы предложил следующее:

tmp = invoices.InvoiceCollection
              .OrderBy(o => o.InvoiceOwner.LastName)
              .ThenBy(o => o.InvoiceOwner.FirstName)
              .ThenBy(o => o.InvoiceID);

Обратите внимание, как вы можете использовать одно и то же имя каждый раз. Это также эквивалентно:

tmp = from o in invoices.InvoiceCollection
      orderby o.InvoiceOwner.LastName,
              o.InvoiceOwner.FirstName,
              o.InvoiceID
      select o;

Если вы вызовете OrderByнесколько раз, он полностью переупорядочит последовательность три раза ... так что последний вызов будет фактически доминирующим. Вы можете (в LINQ to Objects) написать

foo.OrderBy(x).OrderBy(y).OrderBy(z)

что было бы эквивалентно

foo.OrderBy(z).ThenBy(y).ThenBy(x)

поскольку порядок сортировки стабилен, но вам категорически не следует:

• Трудно читать
• Он не работает (потому что меняет порядок всей последовательности)
• Это может не работать в других поставщиках (например, LINQ to SQL)
• По сути, это не так, как OrderByбыло задумано для использования.

Смысл в OrderByтом, чтобы обеспечить «наиболее важную» упорядочивающую проекцию; затем используйте ThenBy(повторно), чтобы указать вторичные, третичные и т. д. упорядочивающие прогнозы.

Фактически, подумайте об этом так: OrderBy(...).ThenBy(...).ThenBy(...)позволяет вам построить одно составное сравнение для любых двух объектов, а затем один раз отсортировать последовательность, используя это составное сравнение. Это почти наверняка то, что вам нужно.

Я обнаружил, что это различие раздражает при попытке создавать запросы в общем виде, поэтому я сделал небольшой помощник для создания OrderBy / ThenBy в правильном порядке для любого количества сортировок.

public class EFSortHelper
{
  public static EFSortHelper<TModel> Create<TModel>(IQueryable<T> query)
  {
    return new EFSortHelper<TModel>(query);
  }
}  

public class EFSortHelper<TModel> : EFSortHelper
{
  protected IQueryable<TModel> unsorted;
  protected IOrderedQueryable<TModel> sorted;

  public EFSortHelper(IQueryable<TModel> unsorted)
  {
    this.unsorted = unsorted;
  }

  public void SortBy<TCol>(Expression<Func<TModel, TCol>> sort, bool isDesc = false)
  {
    if (sorted == null)
    {
      sorted = isDesc ? unsorted.OrderByDescending(sort) : unsorted.OrderBy(sort);
      unsorted = null;
    }
    else
    {
      sorted = isDesc ? sorted.ThenByDescending(sort) : sorted.ThenBy(sort)
    }
  }

  public IOrderedQueryable<TModel> Sorted
  {
    get
    {
      return sorted;
    }
  }
}

Есть много способов использовать это в зависимости от вашего варианта использования, но если вы, например, передали список столбцов сортировки и направлений в виде строк и логических значений, вы могли бы перебрать их и использовать их в переключателе, например:

var query = db.People.AsNoTracking();
var sortHelper = EFSortHelper.Create(query);
foreach(var sort in sorts)
{
  switch(sort.ColumnName)
  {
    case "Id":
      sortHelper.SortBy(p => p.Id, sort.IsDesc);
      break;
    case "Name":
      sortHelper.SortBy(p => p.Name, sort.IsDesc);
      break;
      // etc
  }
}

var sortedQuery = sortHelper.Sorted;

Результат sortedQueryсортируется в желаемом порядке, а не повторяется снова и снова, как предупреждает другой ответ.

если вы хотите отсортировать более одного поля, выберите ThenBy:

как это

list.OrderBy(personLast => person.LastName)
            .ThenBy(personFirst => person.FirstName)

Да, вам никогда не следует использовать несколько OrderBy, если вы играете с несколькими клавишами. ThenBy - более безопасная ставка, поскольку она будет действовать после OrderBy.