HTML Agility Pack 空引用

Question

我在使用 HTML Agility Pack 时遇到了一些问题。

当我在不包含特定节点的 HTML 上使用此方法时，出现空引用异常。一开始它起作用了，但后来就不起作用了。这只是一个片段，还有大约 10 个 foreach 循环来选择不同的节点。

我做错了什么？

public string Export(string html)
{
    var doc = new HtmlDocument();
    doc.LoadHtml(html);
    // exception gets thrown on below line
    foreach (var repeater in doc.DocumentNode.SelectNodes("//table[@class='mceRepeater']"))
    {
        if (repeater != null)
        {
            repeater.Name = "editor:repeater";
            repeater.Attributes.RemoveAll();
        }
    }

    var sw = new StringWriter();
    doc.Save(sw);
    sw.Flush();

    return sw.ToString();
}

Answer 1

AFAIK，如果没有找到节点，DocumentNode.SelectNodes 可能会返回 null。

这是默认行为，请参阅 codeplex 上的讨论线程： Why DocumentNode.SelectNodes returns null

所以解决方法可能是重写 foreach 块：

var repeaters = doc.DocumentNode.SelectNodes("//table[@class='mceRepeater']");
if (repeaters != null)
{
    foreach (var repeater in repeaters)
    {
        if (repeater != null)
        {
            repeater.Name = "editor:repeater";
            repeater.Attributes.RemoveAll();
        }
    }
}

Answer 2

此内容已更新，您现在可以通过设置 doc.OptionEmptyCollection = true 来防止 SelectNodes 返回 null，如 此 github 问题。

如果没有与查询匹配的节点，这将使它返回一个空集合而不是 null（不过，我不确定为什么这不是默认行为）

Answer 3

根据亚历克斯的回答，但我是这样解决的：

public static class HtmlAgilityPackExtensions
{
    public static HtmlAgilityPack.HtmlNodeCollection SafeSelectNodes(this HtmlAgilityPack.HtmlNode node, string selector)
    {
        return (node.SelectNodes(selector) ?? new HtmlAgilityPack.HtmlNodeCollection(node));
    }
}

Answer 4

在每个 . 之前添加简单的 ? 示例如下：

var titleTag = htdoc?.DocumentNode?.Descendants("title")?.FirstOrDefault()?.InnerText;

Answer 5

我创建了通用扩展，它可以与任何 IEnumerable 一起使用

public static List<TSource> ToListOrEmpty<TSource>(this IEnumerable<TSource> source)
{
    return source == null ? new List<TSource>() : source.ToList();
}

，用法是：

var opnodes = bodyNode.Descendants("o:p").ToListOrEmpty();
opnodes.ForEach(x => x.Remove());