Compare nullable types in Linq to Sql

Question

I have a Category entity which has a Nullable ParentId field. When the method below is executing and the categoryId is null, the result seems null however there are categories which has null ParentId value.

What is the problem in here, what am I missing?

public IEnumerable<ICategory> GetSubCategories(long? categoryId)
{
    var subCategories = this.Repository.Categories.Where(c => c.ParentId == categoryId)
        .ToList().Cast<ICategory>();

    return subCategories;
}

By the way, when I change the condition to (c.ParentId == null), result seems normal.

Answer 1

Other way:

Where object.Equals(c.ParentId, categoryId)

or

Where (categoryId == null ? c.ParentId == null : c.ParentId == categoryId)

Answer 2

The first thing to do is to put on logging, to see what TSQL was generated; for example:

ctx.Log = Console.Out;

LINQ-to-SQL seems to treat nulls a little inconsistently (depending on literal vs value):

using(var ctx = new DataClasses2DataContext())
{
    ctx.Log = Console.Out;
    int? mgr = (int?)null; // redundant int? for comparison...
    // 23 rows:
    var bosses1 = ctx.Employees.Where(x => x.ReportsTo == (int?)null).ToList();
    // 0 rows:
    var bosses2 = ctx.Employees.Where(x => x.ReportsTo == mgr).ToList();
}

So all I can suggest is use the top form with nulls!

ie

Expression<Func<Category,bool>> predicate;
if(categoryId == null) {
    predicate = c=>c.ParentId == null;
} else {
    predicate = c=>c.ParentId == categoryId;
}
var subCategories = this.Repository.Categories
           .Where(predicate).ToList().Cast<ICategory>();

---

Update - I got it working "properly" using a custom Expression :

    static void Main()
    {
        ShowEmps(29); // 4 rows
        ShowEmps(null); // 23 rows
    }
    static void ShowEmps(int? manager)
    {
        using (var ctx = new DataClasses2DataContext())
        {
            ctx.Log = Console.Out;
            var emps = ctx.Employees.Where(x => x.ReportsTo, manager).ToList();
            Console.WriteLine(emps.Count);
        }
    }
    static IQueryable<T> Where<T, TValue>(
        this IQueryable<T> source,
        Expression<Func<T, TValue?>> selector,
        TValue? value) where TValue : struct
    {
        var param = Expression.Parameter(typeof (T), "x");
        var member = Expression.Invoke(selector, param);
        var body = Expression.Equal(
                member, Expression.Constant(value, typeof (TValue?)));
        var lambda = Expression.Lambda<Func<T,bool>>(body, param);
        return source.Where(lambda);
    }

Answer 3

You need to use operator Equals:

 var subCategories = this.Repository.Categories.Where(c => c.ParentId.Equals(categoryId))
        .ToList().Cast<ICategory>();

Equals fot nullable types returns true if:

• The HasValue property is false, and the other parameter is null. That is, two null values are equal by definition.
• The HasValue property is true, and the value returned by the Value property is equal to the other parameter.

and returns false if:

• The HasValue property for the current Nullable structure is true, and the other parameter is null.
• The HasValue property for the current Nullable structure is false, and the other parameter is not null.
• The HasValue property for the current Nullable structure is true, and the value returned by the Value property is not equal to the other parameter.

More info here Nullable<.T>.Equals Method

Answer 4

My guess is that it's due to a rather common attribute of DBMS's - Just because two things are both null does not mean they are equal.

To elaborate a bit, try executing these two queries:

SELECT * FROM TABLE WHERE field = NULL

SELECT * FROM TABLE WHERE field IS NULL

The reason for the "IS NULL" construct is that in the DBMS world, NULL.= NULL since the meaning of NULL is that the value is undefined, Since NULL means undefined, you can't say that two null values are equal. since by definition you don't know what they are.

When you explicitly check for "field == NULL", LINQ probably converts that to "field IS NULL". But when you use a variable, I'm guessing that LINQ doesn't automatically do that conversion.

Here's an MSDN forum post with more info about this issue.

Looks like a good "cheat" is to change your lambda to look like this:

c => c.ParentId.Equals(categoryId)

Answer 5

Or you can simply use this. It will also translate to a nicer sql query

Where((!categoryId.hasValue && !c.ParentId.HasValue) || c.ParentId == categoryId)

Answer 6

What about something simpler like this?

public IEnumerable<ICategory> GetSubCategories(long? categoryId)
{
    var subCategories = this.Repository.Categories.Where(c => (!categoryId.HasValue && c.ParentId == null) || c.ParentId == categoryId)
        .ToList().Cast<ICategory>();

    return subCategories;
}

Answer 7

Linq to Entities supports Null Coelescing (??) so just convert the null on the fly to a default value.

Where(c => c.ParentId == categoryId ?? 0)