不，你应该用!=。如果数据实际上是空的，那么你的程序将会因为试图在null上调用Equals方法而导致NullReferenceException崩溃。还要意识到，如果你特别想检查引用是否相等，你应该使用Object。方法，因为你永远不知道Equals是如何实现的。

你的程序崩溃是因为dataList是空的，因为你从来没有初始化它。

不是data为空，而是dataList为空。

你需要创建一个

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

更好的是:既然它是一个字段，就把它设为私有。如果没有什么可以阻止你，也把它设置为只读。只是很好的练习。

一边

检查是否为空的正确方法是if(data != null)。这种检查对于引用类型是普遍存在的;even Nullable<T>重写了相等运算符，以更方便地表示Nullable。检查是否为null时使用HasValue。

如果你执行If (!data. equals (null))，那么如果data == null，你将得到一个NullReferenceException。这有点滑稽，因为避免这种异常是首要目标。

你也在这样做:

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

这绝对不好。我可以想象，您把它放在那里只是为了在仍然在方法内部的情况下进入调试器，在这种情况下忽略这段。否则，不要无缘无故地捕获异常。如果你这样做了，请使用throw;重新抛出它们。

从c# 8开始，你可以使用'empty'属性模式(带模式匹配)来确保对象不为空:

if (obj is { })
{
    // 'obj' is not null here
}

这种方法意味着“如果对象引用了某个对象的实例”(即它不是空的)。

你可以把它看作是:if (obj is null)....的对立面当对象没有引用某个对象的实例时，返回true。

有关c# 8.0模式的更多信息，请阅读这里。

你的dataList是空的，因为它还没有被实例化，从你发布的代码判断。

Try:

    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;
    }
    return success;
}

不，你应该用!=。如果数据实际上是空的，那么你的程序将会因为试图在null上调用Equals方法而导致NullReferenceException崩溃。还要意识到，如果你特别想检查引用是否相等，你应该使用Object。方法，因为你永远不知道Equals是如何实现的。

你的程序崩溃是因为dataList是空的，因为你从来没有初始化它。

c# 6有单值空检查:)

之前:

if (points != null) {
    var next = points.FirstOrDefault();
    if (next != null && next.X != null) return next.X;
}   
return -1;

后:

var bestValue = points?.FirstOrDefault()?.X ?? -1;