MVC4 pass lambda from controller to repository

Question

I have a controller that calls a method in my repository that runs a linq to entities query to get a list of products for my application. I'd like to add the ability for a user to filter the results (something like standard price high to low, low to high, etc etc).

So I'm thinking I need to dynamically populate my orderby in the repository, but am having trouble with this. I'm shaky on generics in c#, so please bear with me.

Controller

model.Products = ProdRepo.GetProducts ((int)CategoryId, 0, 8);

Model

    //get a list of the products
    public List<ProductModel> GetProducts (int CategoryId, int Start, int End) 
    {
        List<ProductModel> Products = new List<ProductModel>();
        var products = (from p in db.products select p).AsEnumerable().OrderByWithDirection(x=> x.date_added, true).Where( x => x.category_id == CategoryId).Where((row, index) => index >= Start && index < End).ToList();
            if (products.Count > 0)
            {
                foreach (var item in products)
                {
                    ProductModel model = new ProductModel();
                    model.CategoryId = item.category_id;
                    model.Description = item.description;
                    model.Id = item.id;
                    model.Product = item.product1;
                    model.DateAdded = item.date_added;
                    model.Image = item.image;
                    model.Price = item.price;

                    Products.Add(model);
                }
            }
      }

So I'm thinking I need to pass a Func<TSource, TKey> from my controller, but I having trouble piecing together how to accomplish that. Any help would be greatly appreciated.

In order to do that, the controller would need to have access to the ProductModel type, is that possible? — Travis J
– Travis J, Commented Jan 27, 2014 at 17:48
I'm not sure I know you mean, @Travis, but it's a new development project, so anything is possible. — mituw16
– mituw16, Commented Jan 27, 2014 at 17:52
@DarthVader Can you explain what you mean? Passing lambdas is nearly essential for LINQ. — Kendall Frey
– Kendall Frey, Commented Jan 27, 2014 at 18:21

Travis J · Accepted Answer · 2014-01-27 17:54:32Z

4

You can do this by passing in a Func<IQueryable<T>, IOrderedQueryable<T>> which in this case would be

 public List<ProductModel> GetProducts (int CategoryId, int Start, int End, Func<IQueryable<ProductModel>, IOrderedQueryable<ProductModel>> order? = null)
 {
  //code shown in question

  if( order != null )
  {
   Products = order(Products).ToList();
  }
 }

And you could call it like this:

model.Products = ProdRepo.GetProducts ((int)CategoryId, 0, 8, order: q => q.OrderBy(prod => prod.SomeField));

answered Jan 27, 2014 at 17:54

Travis J

82.6k43 gold badges213 silver badges280 bronze badges

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

Comments

Zind · Accepted Answer · 2014-01-27 18:01:29Z

1

I'm assuming the struggle is coming in because the ordering element is not always going to be the same type. Rather than overcomplicating a fairly simple choice by passing around a genericized generic, why not just pass in the option and take it into account while building the linq query?

In the code at my workplace we'd probably achieve this with an enum, something like:

public enum OrderByValue
{
    DateAdded,
    Price,
    OtherChoice
}

Then, your query would just have a decision to make in the middle:

var products = (from p in db.products select p).AsEnumerable();
switch(orderByValue)
{
    case OrderByValue.DateAdded:
    products = products.OrderByWithDirection(x=> x.date_added, true);
    break;

    case OtherStuff:
    ...
}
products = products.Where( x => x.category_id == CategoryId)
    .Where((row, index) => index >= Start && index < End)
    .ToList();

If it is in fact just a binary choice you could also do the same without the enum and the switch, and just pass in a bool.

answered Jan 27, 2014 at 18:01

Zind

7611 gold badge7 silver badges17 bronze badges

1 Comment

Zind Over a year ago

Prior answer by Travis J is a bit more elegant, I think. Good thought! Not as complicated as I was thinking it would be.

Collectives™ on Stack Overflow

MVC4 pass lambda from controller to repository

2 Answers 2

Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related