vérifier si un objet est nul en c#

Question

Je voudrais empêcher la poursuite du traitement d'un objet s'il est nul.

Dans le code suivant, je vérifie si l'objet est nul par soit :

if(!data.Equals(null))

et

if(data != null)

Cependant, je reçois un NullReferenceException sur dataList.Add(data) . Si l'objet était nul, il n'aurait même pas dû entrer dans l'énoncé if !

Je demande donc si c'est une bonne façon de vérifier si un objet est nul :

public List<Object> dataList;
public  bool AddData(ref Object data)
    bool success = false;
    try
    {
        // I've also used if(data != null) which hasn't worked either
        if (!data.Equals(null))   
        {
           //NullReferenceException occurs here ...
           dataList.Add(data);                      
           success = doOtherStuff(data);
        }
    }
    catch (Exception e)
    {
        throw new Exception(e.ToString());
    }
    return success;
}

Si c'est la bonne façon de vérifier si l'objet est nul, qu'est-ce que je fais mal (comment puis-je empêcher la poursuite du traitement de l'objet pour éviter l'exception NullReferenceException) ?

Answer 1

Ce n'est pas data c'est-à-dire null mais dataList .

Vous devez en créer un avec

public List<Object> dataList = new List<Object>();

Encore mieux : puisque c'est un champ, faites-en un private . Et si rien ne t'empêche de le faire, fais-le aussi. readonly . Juste une bonne pratique.

Aside

La façon correcte de vérifier la nullité est if(data != null) . Ce type de vérification est omniprésent pour les types de référence ; même Nullable<T> remplace l'opérateur d'égalité pour être un moyen plus pratique d'exprimer nullable.HasValue lors de la vérification de la nullité.

Si vous le faites if(!data.Equals(null)) alors vous obtiendrez un NullReferenceException si data == null . Ce qui est assez comique puisque l'objectif premier était d'éviter cette exception.

C'est aussi ce que vous faites :

catch (Exception e)
{
    throw new Exception(e.ToString());
}

Ce n'est vraiment pas bon. Je peux imaginer que vous l'avez mis là juste pour pouvoir entrer dans le débogueur tout en restant dans la méthode, auquel cas ignorez ce paragraphe. Sinon, n'attrapez pas les exceptions pour rien. Et si vous le faites, relancez-les en utilisant simplement throw; .

Answer 2

Votre dataList est nulle car elle n'a pas été instanciée, à en juger par le code que vous avez affiché.

Essayez :

public List<Object> dataList = new List<Object>();
public  bool AddData(ref Object data)
bool success = false;
try
{
    if (!data.Equals(null))   // I've also used if(data != null) which hasn't worked either
    {
       dataList.Add(data);                      //NullReferenceException occurs here
       success = doOtherStuff(data);
    }
}
catch (Exception e)
{
    throw new Exception(e.ToString());
}
return success;

}

Answer 3

[Modifié pour refléter l'astuce de @kelton52]

Le moyen le plus simple est de faire object.ReferenceEquals(null, data)

Depuis (null==data) n'est PAS garantie de fonctionner :

class Nully
{
    public static bool operator ==(Nully n, object o)
    {
        Console.WriteLine("Comparing '" + n + "' with '" + o + "'");
        return true;
    }
    public static bool operator !=(Nully n, object o) { return !(n==o); }
}
void Main()
{
    var data = new Nully();
    Console.WriteLine(null == data);
    Console.WriteLine(object.ReferenceEquals(null, data));
}

Produit :

Comparaison entre '' et 'Nully'.

Véritable

Faux

Answer 4

Non, vous devriez utiliser != . Si data est en fait nulle, alors votre programme se plantera avec une erreur de type NullReferenceException à la suite d'une tentative d'appel de la Equals méthode sur null . Sachez également que, si vous souhaitez spécifiquement vérifier l'égalité des références, vous devez utiliser la fonction Object.ReferenceEquals car on ne sait jamais comment Equals a été mis en œuvre.

Votre programme se bloque parce que dataList est nulle car vous ne l'initialisez jamais.

Answer 5

Le problème dans ce cas n'est pas que data est nulle. C'est que dataList est lui-même nul.

A l'endroit où vous déclarez dataList vous devez créer un nouveau List et l'assigne à la variable.

List<object> dataList = new List<object>();