LINQ where condition filtering

user_1984610 0
Viewed 95215
String Sex = getSex(); // return M or F
String[] members = getMembers(); // return member codes in array or null
//if members array is null, no filtering for member codes
var query = from tb in MemberTable
            where tb.sex.Equals(Sex) && 
                  (members != null ? members.Contains(tb.membercode) : true)
            select tb;

The code doesn't return correct result. It returns all members no matter what members[] is.

Actually the original LINQ is complex so if there are any other possible solutions, I do not want to write the following:

if (members == null){ /*LINQ1*/ }
else { /*LINQ2*/ }

which is not a good coding style. Any suggestion for solving this problem?

linq c#
3 Answers
var query = MemberTable.Where(x=>x.sex.Equals(Sex))

if (members != null)
     query = query.Where(x=>members.Contains(x.membercode))

//use your query
query.ToList();

OR

var query = from tb in MemberTable
        where tb.sex.Equals(Sex) && 
              (members == null || members.Contains(tb.membercode))
        select tb;

I prefer the first.

user_3388460

Since || short-circuits, you should be able to do this:

var query = from tb in MemberTable
    where tb.sex.Equals(Sex) && 
          (members == null || members.Contains(tb.membercode))
    select tb;

The (members == null || members.Contains(tb.membercode)) subexpression will be true if members is null, so Contains would not be evaluated.

user_3358580 
var list = new List<ModelName>();
list = ctx.MemberTable
    .Where(c => c.sex==Sex)
    .Where(c => c.membercode==true)
    .ToList();
user_83183540
Related