空对象引用

Question

使用 Nunit 使用以下代码块测试 C# 代码：

foreach (XmlNode node in nodeList)
{
    thisReport.Id = node.Attributes.GetNamedItem("id").Value;
    thisReport.Name = node.Attributes.GetNamedItem("name").Value;
    thisReport.Desc = node.Attributes.GetNamedItem("desc").Value;
    if (node.SelectNodes("subreport").Count > 0)
    {
        thisReport.HasSubReport = true;
        subReportNodeList = node.SelectNodes("subreport");
        foreach(XmlNode subNode in subReportNodeList)
        {
            mySubReport.ParentID = node.Attributes.GetNamedItem("id").Value;
            mySubReport.Priority = subNode.Attributes.GetNamedItem("priority").Value;
            mySubReport.SubReportId = subNode.Attributes.GetNamedItem("id").Value;
            mySubReport.SubReportName = subNode.Attributes.GetNamedItem("name").Value;
            string sTime = subNode.Attributes.GetNamedItem("time").Value;
            mySubReport.Time = Convert.ToInt16(sTime);
            thisReport.SubReportsList.Add(mySubReport);
        }
    }
    else
    {
        thisReport.HasSubReport = false;
    }
    reports.Add(thisReport);
}

代码失败，行上出现空对象引用：

            thisReport.SubreportsList.Add(mySubReport)

但查看局部变量， thisReport 存在，并且在块的顶部分配了值，并且 mySubReport 存在，并且在其添加到此Report 的行上方分配了值。 mySubReport 中的所有值均有效，并且 thisReport 中的 SubReportsList 是 SubReport 类型的通用列表。

那么，空值在哪里呢？看起来很简单，一定是一些很明显但我看不到的东西。

Answer 1

在调用 Add 之前您尚未实例化 SubReportsList。在添加 mySubReport 之前执行以下操作：

thisReport.SubReportsList = new List<SubReport>();
thisReport.SubReportsList.Add(mySubReport);

您还可以更改 SubReportsList 属性以使您的生活更轻松：

public class Report
{
    public IList<SubReport> SubReportsList
    {
        get
        {
            if (_subReportsList == null)
            {
                _subReportsList = new List<SubReport>();
            }
            return _subReportsList;
        }
    }
    private IList<SubReport> _subReportsList;
}

如果在 List 为 null 时调用它，则执行此操作会实例化您的 List。

Answer 2

你可能首先应该这样做：

thisReport.SubReportsList = new List<SubReport>();

Answer 3

那么它一定是 SubReportsList 为 null。

Answer 4

由于 @GenericTypeTea 和 @Dan Dumitru 已经提供了很好的答案，我将补充一点，如果在调用属性时值为 null，则可以通过添加隐式构造来“自动”执行此操作。如果您不使用自动属性，则可以执行此操作：

public class Report {
 // ... other code ...
 private List<SubReports> _subReports = null;

 public List<SubReport> SubReports {
    get {
      if (_subReports == null) { _subReports = new List<SubReports>(); }
      return _subReports;
    }
 }
}

有一些注意事项需要注意，例如使其成为线程安全的（这是即兴的），但基本实现将适合您。我会小心使用这种设计，因为它可能会导致仅通过检查属性就创建不一定需要的对象。如果这是不可取的，那么请坚持上面推荐的实现。

Answer 5

thisReport.SubReportsList 是您的空引用；你已经声明了它但没有初始化它。您可以在 thisReport 类型的构造函数中初始化它（可能使用新实例），或者在开始向其中添加内容之前初始化它。

Answer 6

确保使用 new 关键字初始化列表。否则列表本身将为空。