C# 使用LINQ时避免代码重复

好的，我有很多这样的方法：-按艺术家、专辑、年份等对列表进行排序

        public void SortByAlbum(SortOrder sortOrder)
        {
           if (sortOrder == SortOrder.Ascending)
              _list = _list.OrderBy(x => x.Album).ToList();
           else if (sortOrder == SortOrder.Descending)
              _list = _list.OrderByDescending(x => x.Album).ToList();
        }

这是：

        public void SortByArtist(SortOrder sortOrder)
        {
           if (sortOrder == SortOrder.Ascending)
              _list = _list.OrderBy(x => x.Artist).ToList();
           else if (sortOrder == SortOrder.Descending)
              _list = _list.OrderByDescending(x => x.Artist).ToList();
        }

很明显，这不是一个好代码，所以需要重构成一个Sort（）方法，但我不知道如何用最简单的方法来实现。我不在乎它是使用IComparer还是LINQ

我希望它看起来像这样：

    public void Sort(SortOrder sortOrder, SortType sortType)
    {
        //implementation here
    }

    public enum SortType
    {
       Artist,
       Album,
       Year
    }

Sort(SortOrder.Descending, x => x.Album);

public static class SortColumn
{
    public static readonly Func<Song, string> Artist = x => x.Artist;
    public static readonly Func<Song, string> Album = x => x.Album;
}

那么，在没有代码重复的情况下，最干净的方法是什么呢

---

谢谢，Lee

您应该能够模拟OrderBy扩展方法的签名：

更新1您必须在keySelector Func的第一个通用参数中明确显示。我想猜猜你的类型，叫它“歌”

更新2

继Tom Lokhorst的评论之后：如果您想预定义一些速记排序标准，可以通过定义如下类来实现：

    public void Sort(SortOrder sortOrder, SortType sortType)
    {
        //implementation here
    }

    public enum SortType
    {
       Artist,
       Album,
       Year
    }

Sort(SortOrder.Descending, x => x.Album);

public static class SortColumn
{
    public static readonly Func<Song, string> Artist = x => x.Artist;
    public static readonly Func<Song, string> Album = x => x.Album;
}

---

您可以尝试使用。

如果您有几种专门的方法，那么排序听起来就像是有了自己的生命。也许他们可以聚在一起上课

public enum SortOrder
{
    Ascending = 0,
    Descending = 1
}
public class Sorter<T>
{
    public SortOrder Direction { get; set; }
    public Func<T, object> Target { get; set; }
    public Sorter<T> NextSort { get; set; }

    public IOrderedEnumerable<T> ApplySorting(IEnumerable<T> source)
    {
        IOrderedEnumerable<T> result = Direction == SortOrder.Descending ?
            source.OrderByDescending(Target) : 
            source.OrderBy(Target);

        if (NextSort != null)
        {
            result = NextSort.ApplyNextSorting(result);
        }
        return result;
    }

    private IOrderedEnumerable<T> ApplyNextSorting
        (IOrderedEnumerable<T> source)
    {
        IOrderedEnumerable<T> result = Direction == SortOrder.Descending ?
            source.ThenByDescending(Target) :
            source.ThenBy(Target);
        return result;
    }
}

公共枚举排序器
{
升序=0，
递减=1
}
公共类分拣机
{
公共排序器方向{get；set；}
公共函数目标{get；set；}
公共分拣机NextSort{get；set；}
公共IOR可数应用程序排序（IEnumerable源）
{
IOrderedEnumerable结果=方向==排序器。下降？
source.OrderByDescending（目标）：
source.OrderBy（目标）；
if（NextSort！=null）
{
结果=NextSort.ApplyNextSorting（结果）；
}
返回结果；
}
私有IOrderedEnumerable ApplyNextSorting
（不可数源）
{
IOrderedEnumerable结果=方向==排序器。下降？
源。然后按降序（目标）：
来源。然后由（目标）；
返回结果；
}
}

---

以下是示例用法：

List<string> source = new List<string>()
    { "John", "Paul", "George", "Ringo" };

Sorter<string> mySorter = new Sorter<string>()
{
    Target = s => s.Length,
    NextSort = new Sorter<string>()
    {
        Direction = SortOrder.Descending,
        Target = s => s
    }
};

foreach (string s in mySorter.ApplySorting(source))
{
    Console.WriteLine(s);
}

列表源=新列表（）
{“约翰”、“保罗”、“乔治”、“林戈”}；
分拣机mySorter=新分拣机（）
{
目标=s=>s.长度，
NextSort=新分拣机（）
{
方向=排序器下降，
目标=s=>s
}
};
foreach（mySorter.ApplySorting中的字符串s（源））
{
控制台。写入线（s）；
}

---

输出是Paul，John，Ringo，George。

我认为您应该向IList添加一个扩展方法：

---

此外，如果出于某种原因，您不想在调用

Sort

时写下显式lambda（它可能会变大），您可以使用预定义的

Func

对象创建某种类型的列表（相当于示例中的枚举）有趣的是，我今天早上在Linq in Action一书中读到，无法从用法推断方法“System.Linq.Enumerable.OrderByDescending（System.Collections.Generic.IEnumerable，System.Func）”的类型参数。尝试显式指定类型参数。在orderby和orderbyDesc行上。@Lee-这是在几分钟前编辑之前还是之后？在测试代码并发现同样的错误后，我修改了代码。是的，在我阅读你的更新之前，我已经修复了错误，它工作得很好。谢谢你，马特。

IList<Person> list = new List<Person>();

list.Add(new Person("David","Beckham"));
list.Add(new Person("Gennaro","Gattuso"));
list.Add(new Person("Cristian","Carlesso"));

list = list.Sort(SortOrder.Descending, X => X.Name);

using System.Data.SqlClient;