Sonar规则S2589对这段C#代码的检测是否为误报？

你的代码逻辑很清晰：当输入value是bool类型且值为true时返回FontWeights.Bold，其他情况（非bool类型，或是bool但值为false）都返回FontWeights.Normal。Sonar的S2589规则是在提示你这里有冗余的条件判断——你引入的isBold变量其实是可以省略的，而且不会破坏原有逻辑。

优化后的代码（完全匹配原逻辑） #

FontWeight Method1(object value) 
{ 
    return value is true ? FontWeights.Bold : FontWeights.Normal; 
}

为什么这样没问题？ #

C#的模式匹配中，value is true会同时完成两个判断：

1. 首先检查value是否是bool类型；
2. 如果是，再判断它的值是否为true。

这和你原来的value is bool isBold && isBold逻辑完全等价——既处理了非bool类型的情况，也正确区分了bool类型下的true/false，没有任何逻辑错误。

Sonar标记你的原代码，是因为isBold变量在这里属于冗余的中间变量：你不需要先把值赋给变量再判断，直接通过模式匹配就能一步完成类型和值的校验。所以这不是误报，而是规则在引导你写出更简洁、更符合现代C#风格的代码。

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