combining two lambda expression query

Question

I have the follwing two lambda expression. Can you please help me how can I combine them into one.

private bool CheckForNonUniqueRollNumbers(IEnumerable<EmployeeInfoDto> empDtos)
{

    var dtosToCheck = empDtos.Where(dto => dto.ExceptionIfAny == null).ToList();
    var allNumbersEmpty = empDtos.All(dto => dto.Identity.RollNumber == "");
    if (dtosToCheck.Any() && !allSerialNumbersEmpty)
    {

    }

I want to combine above two queries

I tried the following query, but I am not sure if this is correct

var dtosToCheck = empDtos.Where(dto => dto.ExceptionIfAny == null 
    || dto.Identity.RollNumber == "").ToList();
    if (dtosToCheck.Any())
     {

     }

Thanks.

is your main target to satisfy either of the condition or both of them? — vikscool
– vikscool, Commented May 25, 2018 at 5:15
Why do you want to combine them? Your queries are more readable, as it's currently written. IMO! — Salah Akbari
– Salah Akbari, Commented May 25, 2018 at 5:18
@vikscool: Yes. both the conditions should satisfy -S.Akbar: if I combine, then I can just have one variable sayinf "isDtoValid" — Suraj Rai
– Suraj Rai, Commented May 25, 2018 at 5:25
@SurajRai if both are required then the query that you have written is correct, only if you don't want two separate query results. — vikscool
– vikscool, Commented May 25, 2018 at 5:47

Tatranskymedved · Accepted Answer · 2018-05-25 05:52:31Z

1

If both conditions should be satisfied (together at same time), You should use && to logically connect them (logical AND):

var allNumbersEmpty = empDtos
                            .Where(dto => dto.ExceptionIfAny == null
                                   && dto.Identity.RollNumber == "")
                            .ToList();

Your query with || will make a logical OR, which will apply if one or the other condition applies.

answered May 25, 2018 at 5:52

Tatranskymedved

4,4313 gold badges29 silver badges51 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

combining two lambda expression query

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related