Question

I am considering refactoring a repository I have to improve its flexibility and reduce the method count.

Where there are the following methods:

Collection GetAllUsersByRole(Role role)
User GetUserByuserName(string userName)

…I would like to have a single method taking a Linq expression:

ICollection GetUsers(Expression e)
{
  //retrieve user collection from store
  //apply expression to collection and return
}

Is this a reasonable approach? Presumably I’d lose some efficiency because the full users collection would need to be retrieved and filtered every time, rather than retrieving a subset of users according to some hard-coded criteria?

Edit: NHibernate provides ORM in my implementation.

Answer 1

You really want to take an Expression as the argument to that method.

As far as performance, it really comes down to how far you want to go with it. The simplest method is bringing all the objects into memory and then filtering with the predicate expression.

On the other hand, you mention some sort of criteria. I have no idea what your back end data system is, but you can take these passed filters and transform them into your criteria. This is essentially what Linq to SQL and Linq to Entities does, but hopefully the range of possibilities you need to support is significantly smaller. If not, it might make sense to switch to one of the ORM tools if you want to take this approach.