29

I have been searching a lot about my current problem but I could not find a real answer to solve that issue.

I am trying to build a LINQ Query that produces the following SQL:

SELECT * FROM TABLE WHERE (Field1 = X, Field2 = Y ... ) or (Field3 = Z)

In a normal situation I would just do this:

Object.Where(c => (c.Field1 == X && c.Field2 == Y) || (c.Field3 == Z))

I cannot use this approach because the query is build by using multiple .Where() calls.

Having an example:

// This is a short example, the real world situation has 20 fields to check and they are all connected with an AND.
if (model.Field1.HasValue) 
{
    Query = Query.Where(c => c.Field1 == X)
}

if (model.Field2.HasValue) 
{
    Query = Query.Where(c => c.Field2 == X)
}

[...] like 20 more of these .Where() calls.

and that is how it gets complicated for me. All these .Where() calls are building a Linq Query that is connected with AND, which is fine.

How do I let them execute with Parenthese and add a simple OR now using the API?

Is there a way to save the predicate in some variables so I can make something like:

Query = Query.Where(c => previousPredicates || c.Field3 == X)

or how to solve that problem?

I think there must be a good solution for that particual problem and I am not the only one who needs it, but I am absolute unsure how to achieve it.

P.S: I can't really remove the multiple .Where() calls and writing direct SQL is neither an option.

EDIT StackOverflow wants me to say why my question is different from others. Well, the thing is about Parentheses. I do not want to connect all .Where() with a single OR clause, I want to leave them with AND and add another OR clause while all the AND queries are being parenthesied.

10
  • 4
    Possible duplicate of How can I build Linq query with dynamic OR statements?
    – GSerg
    Commented May 18, 2019 at 8:35
  • 1
    No it is not a duplicate. While the other link is helpful it does not really explain how to achieve Parentheses, since it only explains how to replace all ` AND ` connections with ` OR `. Commented May 18, 2019 at 8:44
  • Yes, it is a duplicate. In the Jon Skeet's code, the someOtherCondition is the expression that may contain several ANDs. These AND blocks are then gradually ORed together.
    – GSerg
    Commented May 18, 2019 at 8:47
  • The other answer tells you how to combine predicates. Just replace the || with &&. Then you can do exactly what you want Query = Query.Where(c => previousPredicates || c.Field3 == X)
    – adjan
    Commented May 18, 2019 at 8:52
  • Assuming the Fields are nullable, the example can be shortened to : Query = Query.Where(c => c.Field1 ?? X == X && c.Field2 ?? X == X). If it is entity framework query, add tag because some cases are not applicable
    – Slai
    Commented May 18, 2019 at 8:52

4 Answers 4

18

If you want to build your query programmatically and have it execute on your SQL server instead of fetching all records and querying in memory, you need to use the set of static methods on the Expression class and build your query using those. In your example:

public class Query // this will contain your 20 fields you want to check against
{
    public int? Field1; public int? Field2; public int? Field3; public int Field4;
}

public class QueriedObject // this is the object representing the database table you're querying
{
    public int QueriedField;
}

public class Program
{
    public static void Main()
    {
        var queryable = new List<QueriedObject>().AsQueryable();
        var query = new Query { Field2 = 1, Field3 = 4, Field4 = 2 };

        // this represents the argument to your lambda expression
        var parameter = Expression.Parameter(typeof(QueriedObject), "qo");

        // this is the "qo.QueriedField" part of the resulting expression - we'll use it several times later
        var memberAccess = Expression.Field(parameter, "QueriedField");

        // start with a 1 == 1 comparison for easier building - 
        // you can just add further &&s to it without checking if it's the first in the chain
        var expr = Expression.Equal(Expression.Constant(1), Expression.Constant(1));

        // doesn't trigger, so you still have 1 == 1
        if (query.Field1.HasValue)
        {
            expr = Expression.AndAlso(expr, Expression.Equal(memberAccess, Expression.Constant(query.Field1.Value)));
        }
        // 1 == 1 && qo.QueriedField == 1
        if (query.Field2.HasValue)
        {
            expr = Expression.AndAlso(expr, Expression.Equal(memberAccess, Expression.Constant(query.Field2.Value)));
        }
        // 1 == 1 && qo.QueriedField == 1 && qo.QueriedField == 4
        if (query.Field3.HasValue)
        {
            expr = Expression.AndAlso(expr, Expression.Equal(memberAccess, Expression.Constant(query.Field3.Value)));
        }

        // (1 == 1 && qo.QueriedField == 1 && qo.QueriedField == 4) || qo.QueriedField == 2
        expr = Expression.OrElse(expr, Expression.Equal(memberAccess, Expression.Constant(query.Field4)));

        // now, we combine the lambda body with the parameter to create a lambda expression, which can be cast to Expression<Func<X, bool>>
        var lambda = (Expression<Func<QueriedObject, bool>>) Expression.Lambda(expr, parameter);

        // you can now do this, and the Where will be translated to an SQL query just as if you've written the expression manually
        var result = queryable.Where(lambda);       
    }
}
1
  • Yep, that is the only answer out of presented so far that will work, even though it is pretty inconvenient to use.
    – GSerg
    Commented May 18, 2019 at 14:32
10

First, create some helper extension methods to easier combine two Func<T,bool> predicates:

 public static Func<T, bool> And<T>(this Func<T, bool> left, Func<T, bool> right) 
     => a => left(a) && right(a);

 public static Func<T, bool> Or<T>(this Func<T, bool> left, Func<T, bool> right)
     => a => left(a) || right(a);

Then you can use them to chain predicates:

var list = Enumerable.Range(1, 100);

Func<int, bool> predicate = v => true; // start with true since we chain ANDs first

predicate = predicate.And(v => v % 2 == 0); // numbers dividable by 2
predicate = predicate.And(v => v % 3 == 0); // numbers dividable by 3
predicate = predicate.Or(v => v % 31 == 0); // numbers dividable by 31

var result = list.Where(predicate);

foreach (var i in result)
    Console.WriteLine(i);

Output:

6
12
18
24
30
31
36
42
48
54
60
62
66
72
78
84
90
93
96
8
  • 8
    IQueryable.Where accepts Expression<Func<>>, that is the only thing that can be translated to SQL. IEnumerable.Where accepts Func<>, so passing a Func<> predicate to a EF / Linq2Sql query will result in the entire table fetched on the client and filtered with the predicate. The OP specifically wanted something that translates to SQL.
    – GSerg
    Commented May 18, 2019 at 9:27
  • 2
    See LINQKIT for a generic way of generatig predicates - among other things Commented May 18, 2019 at 9:51
  • 1
    This of course doesn't work with IQueryable<>. Take a look carefully of the type of the var result in your "working" example - it's IEnumerable<>, not IQueryable<>. Also even you make the final predicate expression, using delegates inside makes it non translatable.
    – Ivan Stoev
    Commented May 18, 2019 at 11:31
  • 3
    IQueryable<int> result = list.AsQueryable().Where(predicate); is a compile error, "Cannot implicitly convert type 'System.Collections.Generic.IEnumerable<int>' to 'System.Linq.IQueryable<int>'. An explicit conversion exists (are you missing a cast?)". Which is correct. The code you have on GDB is different, .Where(v => predicate(v)). That does compile (because it is technically an Expression, built for you by the compiler because of the =>), but it will not work in a data context, giving "Method 'System.Object DynamicInvoke(System.Object[])' has no supported translation to SQL."
    – GSerg
    Commented May 18, 2019 at 12:14
  • 1
    @Adrian Correct, and Jon Skeet has confirmed that it would not work. I pointed to it before I saw he was using Func<> instead of Expression<Func<>>.
    – GSerg
    Commented May 18, 2019 at 12:24
5

You can use Expression to create in one step like this:

Expression<Func<Model, bool>> exp = (model => 
                                    ((model.Field1.HasValue && c.Field1 == X) &&
                                    (model.Field2.HasValue && c.Field2 == X)) ||
                                     model.Field3 == X
                                    )

Once you have your predicates defined, it's very easy to use them in a query.

var result = Query.AsQueryable().Where(exp)

Check the code in this gist: my gist url

UPDATE 1: If You have to use steps to create your expression you can use this:

Expression<Func<Model, bool>> exp = c => true;
if (model.Field1.HasValue) 
{
    var prefix = exp.Compile();
    exp = c => prefix(c) && c.Field1 == X;
}

if (model.Field2.HasValue) 
{
    var prefix = exp.Compile();
    exp = c => prefix(c) && c.Field2 == X;
}

[...] like 20 more of these .Where() calls.
10
0

Ok you have had your own share of answer about linq.

Let me introduce a different approach using Dynamic.linq

// You could build a Where string that can be converted to linq.
// and do if sats and append your where sats string. as the example below
var query = "c => (c.Field1 == \" a \" && c.Field2 == Y) || (c.Field3 == \" b \")";
var indicator = query.Split('.').First(); // the indicator eg c
   // assume TABLE is the name of the class
var p = Expression.Parameter(typeof(TABLE), indicator);
var e = DynamicExpression.ParseLambda(new[] { p }, null, query);

// and simple execute the expression 
var items = Object.Where(e);

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Not the answer you're looking for? Browse other questions tagged or ask your own question.