时间戳

首页 > 解决方案 > 在 c# 中编写带有返回值的空引用的更好方法

c# - 在 c# 中编写带有返回值的空引用的更好方法

问题描述

有没有更好的方法来编写这个空检查？我正在检查一个表中的DataSet空值。

if (dataSet == null || dataSet.Tables == null || dataSet.Tables[0].Rows == null)
{
    Console.WriteLine($"Error at {nameof(dataSet)}");
    return vatPeriodList;
}

我在 ADO.NET 中工作。

标签： c#null-check

解决方案

你的检查没有意义，也忘记了一个重要的。

  • DataSet.Tables也不能null因为它是只读属性，不能赋值null，所以第二次检查是没有意义的。
  • dataSet.Tables[0].Rows不可能，null因为它是一个只读属性，你不能分配null，所以最后的检查是多余的。

但是您忘记了DataSet可能是空的，因此不包含任何DataTables. 在这种情况下，您if会在dataSet.Tables[0].

我会使用：

int? firstTablesRowCount = dataSet?.Tables.Cast<DataTable>().FirstOrDefault()?.Rows.Count;
if (firstTablesRowCount.GetValueOrDefault() == 0)
{
    Console.WriteLine($"Error at {nameof(dataSet)}");
}

这可确保DataSet不为空并包含表，并且第一个表包含行。

推荐阅读