C#中IEnumerable双键排序问题：ThenBy不生效如何解决？

问题根源 #

你的ThenBy子句未生效，主要有两个原因：

1. 主排序键逻辑重叠：原代码中当EndDate为空时，主排序键直接用StartDate，此时ThenBy(auction => auction.StartDate)相当于对相同主键的项按同一个值排序，自然看不到排序效果。
2. 字符串日期比较风险：直接比较字符串类型的日期，若日期格式不统一（如"MM/DD/YYYY"和"DD/MM/YYYY"混用），会导致排序结果不符合实际时间顺序。

修正方案 #

先将字符串日期转换为DateTime类型（避免字符串比较的问题），再明确排序逻辑：优先按EndDate排序，空EndDate可指定默认值（比如放到最后），EndDate相同时再按StartDate排序。

方案一：先投影转换日期，再排序 #

var activeAuctions = auctions
    .Where(auction => !auction.Status.Equals(Consts.AuctionStatus.Past, StringComparison.InvariantCultureIgnoreCase))
    // 投影转换日期，处理空/无效日期
    .Select(auction => new {
        OriginalAuction = auction,
        ParsedEndDate = DateTime.TryParse(auction.EndDate, out var endDt) ? endDt : DateTime.MaxValue, // 空EndDate排最后
        ParsedStartDate = DateTime.TryParse(auction.StartDate, out var startDt) ? startDt : DateTime.MinValue
    })
    .OrderBy(item => item.ParsedEndDate)
    .ThenBy(item => item.ParsedStartDate)
    .Select(item => item.OriginalAuction);

方案二：直接在排序委托中转换日期 #

var activeAuctions = auctions
    .Where(auction => !auction.Status.Equals(Consts.AuctionStatus.Past, StringComparison.InvariantCultureIgnoreCase))
    .OrderBy(auction => {
        // 尝试解析EndDate，失败则用StartDate，再失败则设为MaxValue排最后
        if (DateTime.TryParse(auction.EndDate, out var endDt))
            return endDt;
        return DateTime.TryParse(auction.StartDate, out var startDt) ? startDt : DateTime.MaxValue;
    })
    .ThenBy(auction => {
        // 解析StartDate，失败设为MinValue排最前
        DateTime.TryParse(auction.StartDate, out var startDt);
        return startDt;
    });

日期排序的核心要点 #

• 类型转换优先：永远不要直接用字符串比较日期，必须转换为DateTime/DateTimeOffset类型，确保比较逻辑符合时间先后顺序。
• 空值/无效日期处理：明确空日期或解析失败的日期的排序位置——用DateTime.MaxValue放到末尾，DateTime.MinValue放到开头。
• 多字段排序逻辑清晰：主排序键（EndDate）和次排序键（StartDate）要避免逻辑重叠，保证当主键相同时，次键能起到区分排序的作用。

内容的提问来源于stack exchange，提问作者Swarel Jadhav