When you format a complex Linq query well enough (don't let it run on too many lines) it shouldn't be more difficult to read than a complex condition in a while
loop or if
statement.
The follow reads just fine to me:
var myList = MyThings.Where(o => o.ID == someID
&& o.Name == someName
&& IsAdd
|| /* other conditions */).ToList();
So does the version using an extracted equals
Func<T, bool>
.
However, usually when I see a Func
object like that, I make the assumption that it may not always be the same logic. I use a Func
when the action being performed needs to be passed in externally, or determined by some condition. In this case, your Func
is right next to the Linq expression itself, so it's easy to see that it's not the case, but it may be confusing if they were separated further.
Let's pretend this logic was being used in an if
condition. If the logic is complex enough that you would consider splitting it out into its own named Boolean variable, it's probably justifiable to do something similar and make a Func
. If you would write the condition directly inside the body of your if
, it's probably simple enough to understand inside the body of your Where
function.