Ссылка на пустой пакет 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: Почему DocumentNode.SelectNodes возвращает ноль

Таким образом, обходной путь может заключаться в переписывании блока 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

Это было обновлено, и теперь вы можете запретить SelectNodes возвращать ноль, установив doc.OptionEmptyCollection = true, как подробно описано в этой проблеме github .

Это заставит его возвращать пустую коллекцию вместо нуля, если нет узлов, соответствующих запросу (хотя я не уверен, почему это не было поведением по умолчанию для начала)

Answer 3

Вы добавляете простые ? перед каждым . примером удара:

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

Answer 4

Согласно ответу Алекса, но я решил это так:

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

Answer 5

Я создал универсальное расширение, которое будет работать с любым IEnumerable<T>

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());