使用LINQ修改指定MessageDetail属性值失败，求排查原因

场景回顾 #

你定义了以下C#类结构：

public class QualifyResponse { public Message[] MessageList{get; set;} }
public class Message { public MessageDetail[] MessageDetailList{get; set;} public string MessageStatus{get; set;} public string ProviderCode{get; set;} }
public partial class MessageDetail { public string MessageCategory{get; set;} public string MessageCode{get; set;} public string MessageSeverity{get; set;} public string MessageText{get; set;} }

你的需求很明确：遍历所有MessageDetailList，把其中MessageCode等于"status"的对象的MessageText改成"Complete"。但你尝试的LINQ代码并没有生效，我们来看看问题出在哪。

你写的代码是这样的：

response.MessageList.FirstOrDefault()
    .MessageDetailList
    .Where(message => message.MessageCode
        .ToLower()
        .Equals("status"))
    .ToList()
    .ForEach(status => { status.MessageText = "Complete"; });

问题根源 #

咱们拆解一下这段代码的逻辑：

1. 只处理了第一个Message对象：你用了FirstOrDefault()，这只会从MessageList数组里取出第一个Message（如果数组为空，还会返回null，这时候调用MessageDetailList直接会抛空引用异常）。但你的需求是遍历所有Message下的MessageDetailList，所以只处理第一个肯定覆盖不到所有目标对象。
2. 空引用隐患：如果MessageList是空的，或者某个Message的MessageDetailList是空的，这段代码很容易崩溃，而且你没做任何空值判断。

修复方案 #

我们需要遍历MessageList里的每一个Message，再处理每个Message下的MessageDetailList，同时做好空值防护，这里给你两种常用的写法：

方案1：嵌套foreach循环（可读性拉满） #

这种写法最直观，新手也能一眼看明白逻辑，而且空值判断做得很周全：

// 先确保response和MessageList不为空
if (response?.MessageList != null)
{
    foreach (var message in response.MessageList)
    {
        // 再确保当前Message的MessageDetailList不为空
        if (message?.MessageDetailList != null)
        {
            foreach (var detail in message.MessageDetailList)
            {
                // 最后判断MessageCode不为空且匹配目标值
                if (detail?.MessageCode != null && detail.MessageCode.ToLower() == "status")
                {
                    detail.MessageText = "Complete";
                }
            }
        }
    }
}

方案2：LINQ风格写法（保持简洁） #

如果你偏爱LINQ的链式调用风格，可以用嵌套的ForEach来实现，同时加上空值判断：

response?.MessageList?.ToList().ForEach(message => 
{
    message?.MessageDetailList?.Where(detail => 
        detail?.MessageCode != null && detail.MessageCode.ToLower() == "status")
        .ToList()
        .ForEach(detail => detail.MessageText = "Complete");
});

补充说明 #

这里要注意：ToList().ForEach()本质上是对集合的遍历操作，因为数组本身没有ForEach方法，所以我们先转成List再调用。另外，修改MessageDetail对象的属性是生效的，因为Where返回的是原对象的引用，不是副本，所以修改属性会直接作用于原对象——你原来的代码逻辑里这部分是对的，只是没遍历所有Message而已。

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