IEnumerable ...

  • Thread starter Thread starter shapper
  • Start date Start date
S

shapper

Hello,

I am working on a few repositories ...

I think when using repositories an IEnumerable should be returned
instead of IQueryable or IList, correct?

As far as I know, when using repositories, all the queries should be
placed inside the repositories methods and return only the
IEnumerable ...

Probably in terms of performance is also better.

Am I wrong?

Thanks,
Miguel
 
I am working on a few repositories ...

I think when using repositories an IEnumerable should be returned
instead of IQueryable or IList, correct?

As far as I know, when using repositories, all the queries should be
placed inside the repositories methods and return only the
IEnumerable ...

Probably in terms of performance is also better.

Am I wrong?
Click to expand...

It is not clear to me what type of repositories you are
talking about, so I can not comment on the substance.

But I will note that:
- IQueryable extends IEnumerable
- you will always return an instance of a concrete class
even though the method is declared to return an interface

Arne
 
It is not clear to me what type of repositories you are
talking about, so I can not comment on the substance.
Click to expand...

Basically the repositories query a SQL database using Linq to SQL or
Linq to Entities and return one or more entities.
Some methods are FindById, FindAll, FindByCity, etc
I am not sure if this is what you were asking.
But I will note that:
- IQueryable extends IEnumerable
Click to expand...
Yes, I know but that is why I say IEnumerable should be used.
Why having a repository to return a list and then query that list
again?
Afer using a Repository method, like FindByCity, I think the logic is
to display the data not to query it again before displaying it.
It makes more sense to create a new method in the repository if needed
for extra functionality.

Not?

Thanks,
Miguel
 
Basically the repositories query a SQL database using Linq to SQL or
Linq to Entities and return one or more entities.
Some methods are FindById, FindAll, FindByCity, etc
I am not sure if this is what you were asking.

Yes, I know but that is why I say IEnumerable should be used.
Why having a repository to return a list and then query that list
again?
Afer using a Repository method, like FindByCity, I think the logic is
to display the data not to query it again before displaying it.
It makes more sense to create a new method in the repository if needed
for extra functionality.

Not?
Click to expand...

IList also extends IEnumerable.

So if you return an IList then you will have all the
functionality of IEnumerable plus some extra functionality.

So I do not understand your argument.

Arne
 
shapper said:
Basically the repositories query a SQL database using Linq to SQL or
Linq to Entities and return one or more entities.
Some methods are FindById, FindAll, FindByCity, etc
I am not sure if this is what you were asking.

Yes, I know but that is why I say IEnumerable should be used.
Click to expand...

It *is* IEnumerable--by virtue of being IQueryable.
Why having a repository to return a list and then query that list
again?
Click to expand...

You don't have to query anything again. You have an object of a type
that implements IEnumerable, exactly as you say you would like. Since it
also implements IQueryable, you *could* query it to obtain a subset of
it if your application called for you to do so, but this is not an
obligation.
 
Back
Top