Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Linq OrderBy breaks with navigation property being null

Tags:

Working with four tables.

Users -> has basic user info including a userid and a departmentid (int)
Groups -> basic group info including a groupid
GroupsMembers -> table that has the relationship between a group and it's members, many to many relationship, so groupid and userid are the columns
Departments -> basic department info including deptid

I have a fk from the departmentid in the users table to the deparmtnet id in the departments table.

FK from groups groupid to groupsmembers groupid FK from users userid to groupsmembers userid

This allows the groups in the edmx to have a users navigation property which will have all the members of the group.

var grp = grpSource.FirstOrDefault(g => g.GroupID == groupID) if (grp.GroupID > 0) {     var userQuery = from u in grp.Users                     where !u.Deleted                     select u;     userQuery = userQuery.OrderBy(u => u.Department.Name); } 

I am including Users.Department.

The problem comes because users do not have to have a department, so the departmentid column is nullable. If there are any users for which the departmentid is null, the orderby breaks and says u.Department is null. If no departmentids are null, it works great. I need a way to sort based on Department.Name even if there are null departmentids. Any suggestions?

like image 975
Steph Avatar asked Feb 18 '10 16:02

Steph


1 Answers

You can use the conditional operator to check if the department is null :

userQuery = userQuery.OrderBy(u => (u.Department != null) ? u.Department.Name : String.Empty); 

For improved clarity, I created the following extension method :

    public static TResult IfNotNull<TSource, TResult>(this TSource obj, Func<TSource, TResult> selector, TResult defaultValue)     {         if (obj != null)             return selector(obj);         return defaultValue;     } 

It can be used as follows :

userQuery = userQuery.OrderBy(u => u.Department.IfNotNull(d => d.Name, String.Empty)); 
like image 128
Thomas Levesque Avatar answered Oct 07 '22 00:10

Thomas Levesque