Checking if an object is null in C#

It’s not data that is null, but dataList.

You need to create one with

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

Even better: since it’s a field, make it private. And if there’s nothing preventing you, make it also readonly. Just good practice.

Aside

The correct way to check for nullity is if(data != null). This kind of check is ubiquitous for reference types; even Nullable<T> overrides the equality operator to be a more convenient way of expressing nullable.HasValue when checking for nullity.

If you do if(!data.Equals(null)) then you will get a NullReferenceException if data == null. Which is kind of comical since avoiding this exception was the goal in the first place.

You are also doing this:

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

This is definitely not good. I can imagine that you put it there just so you can break into the debugger while still inside the method, in which case ignore this paragraph. Otherwise, don’t catch exceptions for nothing. And if you do, rethrow them using just throw;.

Leave a Comment