Lambda表达式过滤结果异常求助：总是选中非Expedite角色

问题根源 #

你的代码仅确保分组中存在Expedite角色的条目，但在选择分组内具体项时，没有优先锁定**Expedite角色且联系人值有效**的条目，而是直接取分组里最后一个联系人值有效的项——这就导致如果分组内其他角色的有效项排在最后，就会被错误选中。

具体解决步骤 #

1. 调整分组内的选择逻辑
   在从分组中选值时，先筛选出Expedite角色且联系人有效的项；如果存在这类项就优先返回，再根据需求处理其他情况（比如返回Expedite角色但联系人无效的项，或过滤掉无效项）。

2. 修改后的代码示例

   // group by unique key POPOrderReturnLineID
   IEnumerable<IGrouping<long, ExpediteDataDTO>> GroupedExpediteData =
   GetDirtyExpediteData()
   .ToList()
   .GroupBy(expediteItem => expediteItem.POPOrderReturnLineID);
   
   
   // 优先选择Expedite角色且联系人值有效的项
   IEnumerable<ExpediteDataDTO> ValidExpedite = GroupedExpediteData
   .Where(groupItem => groupItem.Any(expediteItem => expediteItem.PO_Role == "Expedite"))
   .Select(groupItem => 
   {
       // 先找Expedite角色+联系人有效项
       var preferredItem = groupItem.FirstOrDefault(expediteItem => 
           expediteItem.PO_Role == "Expedite" && IsContactValueValid(expediteItem.ContactValue));
       
       // 有优先项则返回，否则返回分组内任意Expedite角色项（可根据需求调整）
       return preferredItem ?? groupItem.FirstOrDefault(expediteItem => expediteItem.PO_Role == "Expedite");
   })
   // 可选：过滤掉无有效项的结果
   .Where(item => item != null);
   

3. 额外注意

   • 如果分组内存在多个Expedite有效项，可根据业务需求排序后选择（比如按创建时间取最新/最早的），替换FirstOrDefault为OrderByDescending(x => x.CreateTime).FirstOrDefault()这类逻辑。
   • 若不需要保留Expedite角色但联系人无效的项，可保留最后的.Where(item => item != null)过滤逻辑。

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