如何设置断点步进调试LINQ FindAll查询定位运行异常原因

你要在FindAll的lambda执行过程中步进调试，用VS原生功能就能做到，不需要额外工具：

• 最直接的方式是把单行lambda改成带代码块的委托，直接在内部打断点。改完之后调试器遍历每个元素时都会命中你打的断点，可以直接检查当前card变量的值、Suit属性的状态，逐行步进执行判断逻辑：

tempList = cards.FindAll(card => 
{
    // 光标定位到这行按F9打个断点就行
    return card.Suit == (Suit)i;
});

• 不想改代码的话，可以给FindAll所在行打条件断点：右键断点选「条件」，输入判断规则card == null，调试器会自动在遍历到null元素时停下，不用手动跳过所有正常元素。
• 临时调试阶段也可以直接在lambda里加空判断打日志，快速定位null元素的位置：

tempList = cards.FindAll(card => 
{
    if (card == null)
    {
        Console.WriteLine("遍历到null元素");
        return false;
    }
    return card.Suit == (Suit)i;
});

你之前只在循环里打印了cards[0]到cards[3]这前4个元素的Suit，而列表总共有9个元素，剩下的5个元素你根本没做校验——只要这5个元素里有一个是null，FindAll遍历到它的时候访问card.Suit就会抛出空引用异常，和你之前看到的报错完全吻合。

• 循环外写的List<Card> tempList = new List<Card>();是无效代码，每次循环都会把FindAll返回的新列表赋值给tempList，最开始创建的空列表根本不会被用到，直接删掉就行。
• 花色循环硬编码了i < 4，如果后续Suit枚举新增或者删除成员，这段逻辑不会同步更新，要么漏匹配要么出现非法枚举值的判断，更稳妥的写法是遍历Enum.GetValues<Suit>()拿到所有合法花色值再做匹配。
• 谓词里没有做空值校验，就算这次把测试数据里的null清掉，后续只要传入的数据源混了null元素还是会抛异常，正式逻辑里建议补上非空判断：

tempList = cards.FindAll(card => card != null && card.Suit == (Suit)i);

• 排查问题时只验证了前4个元素的有效性，误以为整个列表所有元素都是合法实例，这是这次定位问题走弯路的核心原因——List.Count只代表列表的元素个数，不代表每个索引位置上的引用都是有效对象。

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