C# 7.3 ValueTuple：解构后默认值检查优化及缺陷探讨

首先得给你点个赞——你不仅发现了写法冗余的问题，还敏锐察觉到了用default表示错误状态的致命bug，这确实是ValueTuple作为返回值时很容易踩的坑：当Tuple的成员类型本身的默认值是合法业务值时（比如(int, int)的(0,0)），你根本没法区分“方法执行失败返回默认”和“方法成功返回了刚好是默认值的合法结果”。

所以我们的优化要分成两步：先解决歧义问题，再简化写法，一举两得。

方案1：用带状态标识的Tuple返回（最推荐） #

直接修改方法的返回类型，把“执行是否成功”作为第一个成员，从根源上消除歧义：

// 修改后的方法
(bool IsSuccess, string Name, int Value) GetData(bool ok) 
{ 
    if (!ok) return (IsSuccess: false, Name: default, Value: default); 
    return (IsSuccess: true, Name: "John", Value: 5); 
}

调用时用模式匹配直接解构并判断，完全不需要中间变量：

// 写法1：简洁的否定模式
if (GetData(true) is not (IsSuccess: true, Name: var name, Value: var val))
    return false;

// 写法2：更直观的匹配成功模式
if (GetData(true) is { IsSuccess: true, Name: var name, Value: var val })
{
    // 后续直接使用name和val
}
else
{
    return false;
}

这种写法清晰、无歧义，不管Tuple有多少个成员，判断逻辑都只需要检查IsSuccess，完全不会繁琐。

方案2：用可空ValueTuple（适合无法修改原方法的场景） #

如果原方法的返回类型不能改，那可以把它改成返回可空ValueTuple，用null表示执行失败，代替容易混淆的default：

// 修改后的方法
(string Name, int Value)? GetData(bool ok) 
{ 
    if (!ok) return null; 
    return ("John", 5); 
}

调用时同样用模式匹配解构，一行搞定检查：

if (GetData(true) is not (var name, var val))
    return false;

// 直接使用name和val

这里null明确代表“方法执行失败”，和任何合法的Tuple实例都不会混淆，完美解决原有的bug。

关于你原来写法的补充说明 #

如果实在没法修改原方法（比如是第三方库的方法），那只能退而求其次用模式匹配检查default，但一定要记住这个写法的局限性：

if (GetData(true) is not (var name, var val) || (name, val) == default)
    return false;

但还是那句话——只要业务上存在“合法值等于default”的可能，这个写法就会有bug，所以尽量优先用前面两种方案。

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