可能的 NullReferenceException

Question

例如这个sn-p的代码：

private void LoadComments(Member member, XElement commentsXml)
{
    member.Comments = (from comment in commentsXml.Descendants("comment")
                     select new Comment()
                     {
                         ID = comment.Element("id").Value,
                         Text = comment.Element("text").Value,
                         Date = comment.Element("date").Value,
                         Owner = comment.Element("user").Element("name").Value
                     }).ToList();
}

ReSharper 警告我可能在 comment.Element 行上出现 NullReferenceException。确实，异常被触发了。

关于如何避免这种情况的任何建议？如果它返回null呢，只返回一个空字符串，这可能吗？

Answer 1

我更喜欢有一个扩展类：

public static class XElementExtension
{
   public static string GetValue(this XElement input)
   {
      if (input == null)
        return null;
      return input.Value as T;
   }

   public static XElement GetSubElement(this XElement element, string id)
   {
      if (element == null)
        return null;
      return element.Element(id);
   }
}

并将其用作：

ID = comment.Element("id").GetValue()

或

Owner = comment.Element("user").GetSubElement("name").GetValue()

还有其他方法，比如：

http://www.codeproject.com/KB/cs/maybemonads.aspx

Answer 2

一些扩展可能会有所帮助：

示例：

public static class XElementExtensions
{
    public static XElement GetElement(this XElement element, XName elementName)
    {
        if (element != null)
        {
            XElement child = element.Element(elementName);

            if (child != null)
            {
                return child;
            }
        }

        return null;
    }

    public static String GetElementValue(this XElement element, XName elementName)
    {
        if (element != null)
        {
            XElement child = element.Element(elementName);

            if (child != null)
            {
                return child.Value;
            }
        }

        return null;
    }
}

用法：

private void LoadComments(Member member, XElement commentsXml)
{
    member.Comments = (from comment in commentsXml.Descendants("comment")
                        select new Comment()
                        {
                            ID = comment.GetElementValue("id"),
                            Text = comment.GetElementValue("text"),
                            Date = comment.GetElementValue("date"),
                            Owner = comment.GetElement("user").GetElementValue("name")
                        }).ToList();
}

Answer 3

我只能认为您需要检查每个元素引用：

private void LoadComments(Member member, XElement commentsXml)
{
    member.Comments = (from comment in commentsXml.Descendants("comment")
                     select new Comment()
                     {
                         ID = (comment.Element("id")==null)?"":comment.Element("id").Value,
                         Text = (comment.Element("text")==null)?"":comment.Element("text").Value,
                         Date = (comment.Element("date")==null)?"":comment.Element("date").Value,
                         Owner = (comment.Element("user")==null)?"":comment.Element("user").Element("name").Value
                     }).ToList();
}

最后一个有点弱，因为确实应该检查两个节点，但嵌套看起来有点不愉快，但这是唯一确定的方法。

编辑----

作为扩展方法：

public static class MyExtensions
{

    public static string ValueSafe(this XElement target)
    {
        if (target==null)
            { return ""; }
        else
            { return target.Value; }
    }

}

然后您可以将.Value 替换为.ValueSafe，也就是说，没有机会测试。

Answer 4

这是可能的 - 您需要将每个 comment.Element().Value 语句包装在一个函数中。

我喜欢用：

public string UnNull(object v)
{
   if (v == null) return "";
   return v.ToString();
}

至于最后一行，您需要格外小心，以确保 comment.Element("user") 不是 null 并处理这种情况。