Syntax to refer a method returning an Expression to another method?

Question

I found a piece of code of the following form:

public static Expression<Func<Invoice, CustomerContact>> GetCustomerContact()
{
   return i => new CustomerContact {
                 FirstName = i.Customer.FirstName,
                 LastName = i.Customer.LastName,
                 Email = i.Customer.Email,
                 TelMobile = i.Customer.TelMobile,
               };
}

In other parts of the code, I want to get the same lightweight CustomerContact object, only not from the Invoice, but from the Customer itself. So the obvious thing to do would be to have:

public static Expression<Func<Customer, CustomerContact>> GetCustomerContact()
{
   return c => new CustomerContact {
                 FirstName = c.FirstName,
                 LastName = c.LastName,
                 Email = c.Email,
                 TelMobile = c.TelMobile,
               };
}

and then change the Expression taking Invoice as input to refer to this method, i.e. something like this:

public static Expression<Func<Invoice, CustomerContact>> GetCustomerContact()
{
   return i => GetCustomerContact(i.Customer); // doesn't compile
}

What's the correct syntax for this?

@newStackExchangeInstance: "Method, delegate or event is expected" — Shaul Behr
– Shaul Behr, Commented Jun 2, 2013 at 9:10

Ani · Accepted Answer · 2013-06-02 09:09:50Z

3

You can use Expression.Invoke:

var paramExpr = Expression.Parameter(typeof(Invoice), "i");
var propertyEx = Expression.Property(paramExpr, "Customer");

var body = Expression.Invoke(GetCustomerContactFromCustomer(), propertyEx);

return Expression.Lambda<Func<Invoice, CustomerContact>>(body, paramExpr);

Do note that some LINQ providers have problems with such invocation-expressions.

The easiest way to work around this (and to give you more convenient syntax) is to use LINQKit:

var expr = GetCustomerContactFromCustomer();   
Expression<Func<Invoice, CustomerContact>> result = i => expr.Invoke(i.Customer);    
return result.Expand();

answered Jun 2, 2013 at 9:09

Ani

114k27 gold badges269 silver badges313 bronze badges

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

3 Comments

Shaul Behr Over a year ago

Ouch, that's quite inelegant. Maybe that is the best solution, but I was hoping there would be a simpler solution that doesn't involve either reflection or third party extensions...

Ani Over a year ago

I don't think there's any way easier than Expression.Invoke to compose expressions in this manner. The LINQKIt way though is quite simple and elegant and gives you back the ability to use language lambdas for this.

Shaul Behr Over a year ago

Well, turns out LinqKit is going to be useful for my project in other areas. So thanks, and answer credit to you! :-)

Ben Reich · Accepted Answer · 2013-06-02 09:27:11Z

0

Are you sure you need to use an Expression? If you don't need different Linq providers to convert code trees into queries, then consider using just Func, instead. If you just use Func so that the method signatures are:

public static Func<Customer, CustomerContact> GetCustomerContact();

and

public static Func<Customer, CustomerContact> GetCustomerContact();

Then your syntax would be fine for constructing the second Func off of the first one. Of course, this will only work for in-memory objects (with Linq-to-objects).

The problem is that in order to build an Expression, you have to explicitely build the evaluation tree, which can be quite hairy (using the various static methods on Expression). Because of this hairiness, there are several helper packages, including LINQKit.

answered Jun 2, 2013 at 9:27

Ben Reich

16.3k2 gold badges41 silver badges61 bronze badges

2 Comments

Shaul Behr Over a year ago

Yeah, I do need an Expression; I want the query to run SQL-side.

Ben Reich Over a year ago

Then go with Ani's solution!

Collectives™ on Stack Overflow

Syntax to refer a method returning an Expression to another method?

2 Answers 2

3 Comments

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

3 Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related