代码优化 - 删除这个总是评估为“真”的表达式

Question

我正在尝试使用 SonarQube 优化代码

List<string> itemList = serviceObject.GetItems();

我尝试使用以下代码验证列表

if(itemList != null && itemList.Any()
{
     //Some operations
}

执行上述代码时，我收到 Sonarqube 错误 删除此表达式，该表达式的计算结果始终为“true” 所以我将代码重构为

if(itemList == null || !itemList.Any())
    return;
//Some Operations

执行上述代码时，我收到 Sonarqube 错误 删除此表达式，该表达式的计算结果始终为“false”

谁能告诉我这里出了什么问题？

Answer 1

您可以将其缩短为

if (itemList?.Count >0)
{
...
}

或

if (itemList?.Any() ==true)
{
...
}

?. 是Null conditional operators 之一（另一个是?[]），简称Elvis operator，它允许您访问潜在的空变量而无需抛出。 Elvis 运算符之后的整个表达式的结果可以为 null，如果变量为 null，则返回 null。

这意味着itemList?.Count 返回一个Nullable<int>，而itemList?.Any() 返回一个Nullable<bool>。 Nullable<T> 定义了自身与其基类型 T 之间的关系运算符，但不能在没有显式转换的情况下用作 T。这就是需要(itemList?.Any() ==true) 的原因。

如果你使用Nullable Reference Types，itemList 不能为空，所以一个简单的比较就足够了：

if (itemList.Count >0)
{
...
}

如果通过在源文件中设置#nullable enable 或在csproj 文件中设置<Nullable>enable</Nullable> 来启用可空引用类型，编译器会确保所有返回引用类型的变量、字段和方法不是 em> null，迫使您要么解决问题，要么明确指定一个变量可以为空。

启用 NRT 后，此行：

List<string> itemList = serviceObject.GetItems();

如果GetItems 从未返回空值，则只会在没有警告的情况下编译。如果编译器对此表示怀疑，它会发出警告，建议您修复问题或明确声明 itemList 可以为空：

List<string>? itemList = serviceObject.GetItems();

Answer 2

我相信这是由于空比较。

itemList != null

serviceObject.GetItems(); 很有可能通过 [NotNull] 注释保证不返回 null。因此，空检查是多余的。

How can I show that a method will never return null (Design by contract) in C#

Answer 3

if( itemList !=null ) {//做某事...}

Answer 4

List<string> itemList = new List<string>();
itemList = serviceObject.GetItems();
if(itemList!=null && itemList.Count() > 0 )
{
     //Some operations
}

Count 足以验证列表

List 永远不会为 null，但不会有任何元素。但是您必须将其实例化，以免引发异常。