C#空条件运算符在方法参数中的使用及代码重构咨询

问题背景 #

你有以下C#代码：

float myMethod(MyObject[][] myList) { 
    float a = 0; 
    if (myListProcessingMethod(myList?.Where(x => x.mySatisfiedCondition()).ToList())) { 
        a = 5; 
    } 
    return a; 
}

bool myListProcessingMethod(List<MyObject[]> myList) { 
    bool isSuccess = false; 
    if (myList.Any()) { 
        isSuccess = true; 
    } 
    return isSuccess; 
}

你计划将原条件判断：

if (myListProcessingMethod(myList?.Where(x => x.mySatisfiedCondition()).ToList()))

重构为：

if (myList?.Length != 0) { ... }

想知道这次重构是否符合原业务逻辑，以及是否存在潜在问题。

结论：这次重构完全不符合原业务逻辑，还会引入新的行为差异 #

我们来拆解原逻辑和重构后逻辑的核心差异：

1. 核心业务逻辑的本质区别

原逻辑的执行流程是：

• 先对二维数组myList的每一行（MyObject[]）做过滤，只保留满足x.mySatisfiedCondition()的行
• 将过滤后的结果转为列表，传给myListProcessingMethod判断过滤后的列表是否非空
• 只有过滤后存在符合条件的行，才会进入分支给a赋值为5

而重构后的逻辑是：

• 仅判断原二维数组myList是否不为空且长度大于0，完全跳过了「行是否满足条件」的筛选步骤
• 只要原数组有元素（哪怕所有行都不满足mySatisfiedCondition()），就会进入分支赋值a=5

这直接违背了原代码的业务意图——原代码是要筛选符合条件的行，重构后完全忽略了这个核心筛选逻辑，逻辑一致性完全不匹配。

2. 空值处理的行为差异

原代码本身存在潜在Bug：当myList为null时，myList?.Where(...)会返回null，接着调用ToList()会抛出ArgumentNullException（因为Enumerable.ToList不接受null源）；即使没在ToList()报错，myListProcessingMethod里的myList.Any()也会抛出NullReferenceException。

重构后的myList?.Length != 0：

• 当myList为null时，myList?.Length返回null，null != 0的结果是true，会直接进入分支
• 这和原代码在myList为null时的行为（抛出异常）完全不同，会导致空数组场景下的逻辑错误

3. 正确的简化重构方向

如果你想简化原代码但保留业务逻辑，应该把过滤和非空判断合并，比如：

if (myList?.Any(x => x.mySatisfiedCondition()) == true) {
    a = 5;
}

这样既保留了「过滤满足条件的行并判断是否存在」的核心逻辑，又避免了原代码中ToList()的不必要内存分配，同时正确处理了myList为null的情况（返回false，不进入分支）。

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