c# - NullReferenceException 在尝试检索配置属性时出现未处理错误

Question

    public string GetLogName(string config)
    {
        XDocument xDoc = XDocument.Load(config);
        XElement[] elements = xDoc.Descendants("listeners").Descendants("add").ToArray();

        foreach (var element in elements)
        {
            if (element.Attribute("fileName").Value != null)
            {
                string filename = element.Attribute("fileName").Value;
                int location = filename.IndexOf("%");
                Console.WriteLine("string to return: " + filename.Substring(0, location));
                return filename.Substring(0, location);
            }
        }
    }

我正在尝试从元素数组中的每个元素中检索“fileName”属性，但在某些情况下，“fileName”属性不存在并且失败并出现以下错误：NullReferenceException 未处理。你调用的对象是空的。

在我的例子中，有两个“添加”节点没有“文件名”属性，但第三个添加节点有它。

如何跳过没有“fileName”属性的条目，或者您能否推荐一种更好的方法来检索此属性？

Answer 1

一种方法是在处理之前过滤掉列表：

XElement[] elements = xDoc.Descendants("listeners")
                          .Descendants("add")
                          .Where (d => d.Attribute("filename") != null )
                          .ToArray();

---恕我直言，这就是我使用linq和正则表达式重写方法的方式---

var elements =
XDocument.Load(config);
         .Descendants("listeners")
         .Descendants("add")
         .Where (node => node.Attribute("filename") != null )
         .ToList();


return elements.Any() ? elements.Select (node => node.Attribute("filename").Value )
                                .Select (attrValue => Regex.Match(attrValue, "([^%]+)").Groups[1].Value)
                                .First ()
                      : string.Empty;

Answer 2

您应该可以通过更改此行来做到这一点：

if (element.Attribute("fileName").Value != null)

至：

if (element.Attribute("fileName") != null)

Answer 3

将您的 if 语句更改为：

if (element.Attribute("fileName") != null)