Exception de référence nulle après vérification de null (la vérification de null ne fonctionne pas)

Lorsque vous travaillez avec des collections et IEnumerable , évitez d'utiliser null ; si vous n'avez rien à renvoyer, renvoyez une collection vide (et non null ).

Dans votre cas particulier, SelectMany ne renverra jamais null , mais collection vide, c'est pourquoi la vérification de categories! = null est inutile , et vous devez vérifier jetons à la place

// if tokens is null change it for an empty collection
tokens = tokens ?? new MyToken[0];

...

if (tokens 
      .Where(x => x != null && x.Categories != null)
      .SelectMany(x => x.Categories)
      .Contains("interp"))
    return null;

Cependant, la vérification constante de null rend le code illisible , c'est pourquoi essayez de vérifier null once:

if (null != tokens)
  // Where(x => x != null) - to be on the safe side if x == null or x.Categories == null
  if (tokens
       .Where(x => x != null && x.Categories != null)
       .SelectMany(x => x.Categories)
       .Contains("interp"))
    return null;

var categories = tokens.SelectMany (x => x.Categories) .ToList ();

ajoutez .ToList () et vous devriez en savoir plus sur l'emplacement de l'erreur avec ces informations que nous avons dans le post, nous ne pouvons que deviner

Ce code lancera un NRE dans categories.Contains uniquement si la propriété Categories est nulle.

Le code suivant lancera:

class Token
{
    string[] _categories=new string[0];
    public string[] Categories{
        get => _categories;
        set => _categories = value??new string[0];
    }

}

Mais il en irait de même

var categories = tokens.Where(x=>x.Categories!=null).SelectMany(x => x.Categories);

La chose qui lance réellement est le itérateur dans SelectMany , pas ToArray () ou Contains . La trace de pile pour cette exception est:

at System.Linq.Enumerable.<SelectManyIterator>d__17`2.MoveNext()
at System.Linq.Enumerable.Contains[TSource](IEnumerable`1 source, TSource value, IEqualityComparer`1 comparer)
at UserQuery.Main()

SelectMany essaiera d'itérer sur chaque entrée Categories , trouver que le La propriété est en fait nulle et rejetée.

La solution rapide consiste à ajouter un Where avant SelectMany pour éliminer les catégories nulles:

tokens.SelectMany(x => x.Categories).ToArray();

La vraie solution est de s'assurer que Catégories n'est jamais vide - il devrait être initialisé à un tableau, une liste vide, quoi que ce soit lors de la construction. Lorsqu'il est réaffecté, il ne doit jamais être défini sur null.

Cet exemple définit le champ _categories sur nouvelle chaîne [0] même si un appelant passe null aux catégories

class Token
{
    public string[] Categories{get;set;}
}

var tokens=new []{new Token()};
var categories = tokens.SelectMany(x => x.Categories);
if (categories != null)
{
    if (categories.Contains("interp")) 
    {
        Console.WriteLine("Found");
    }
}

Avec cela, Where (x => x.Categories! = null) n'est plus nécessaire

Peut utiliser la clause where et en faire une liste, puis vérifier simplement s'il y a un élément dans la liste

 var categories = list.Where(x => x.Categories.Contains("interp")).ToList();
 if (categories.Count() == 0)
  {
     return null;

   }