foreach循环中的item是否可能为null？代码检测警告解析

先看你贴出的这段代码：

foreach(var item in list) { 
    var valueForPropertyA = getPropertyAValue(item?.Id ?? 0); 
    if(valueForPropertyA == null) { 
        continue; 
    } 
    item.PropertyA = new PropertyADto(valueForPropertyA); 
}

代码检测工具给出的警告是：'item' is null on at least one execution path，这个警告并不是误报——你的list里确实有可能存在null的item。

为什么会出现这种情况？ #

• foreach循环本身不会过滤集合里的null元素。如果你的list是可空引用类型集合（比如List<YourObject?>），或者是从外部传入、旧代码生成的集合，里面完全可能混入null值。
• 你代码里用item?.Id其实已经隐含考虑了item为null的场景，但后面直接执行item.PropertyA = ...时，如果item真的是null，就会触发NullReferenceException，这正是代码检测工具预警的风险点。

怎么处理这个警告？ #

最简单的方式是在循环开头就加一层null过滤：

foreach(var item in list) { 
    if(item == null) continue; // 先筛掉null项
    var valueForPropertyA = getPropertyAValue(item.Id); // 这里无需再用空合并，item已确定非空
    if(valueForPropertyA == null) { 
        continue; 
    } 
    item.PropertyA = new PropertyADto(valueForPropertyA); 
}

如果你的业务逻辑里list绝对不应该包含null元素，那也可以从源头入手：确保集合的创建、填充过程不会加入null，同时将集合类型声明为非可空的List<YourObject>，这样代码检测工具就会自动消除这个警告。

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