Do we need to use the Repository pattern when working in ASP.NET MVC with ORM solutions?

Question

I am bit curious as to what experience other developers have of applying the Repository pattern when programming in ASP.NET MVC with Entity Framework or NHibernate. It seems to

Answer 1

The answer is no if you do not need to be able to switch ORM or be able to test any class that has a dependency to your ORM/database.

If you want to be able to switch ORM or be able to easily test your classes which uses the database layer: Yes you need a repository (with an interface specification).

You can also switch to a memory repository (which I do in my unit tests), a XML file or whatever if you use repository pattern.

Update

The problem with most repository pattern implementations which you can find by Googling is that they don't work very well in production. They lack options to limit the result (paging) and ordering the result which is kind of amazing.

Repository pattern comes to it's glory when it's combined with a UnitOfWork implementation and has support for the Specification pattern.

If you find one having all of that, let me know :) (I do have my own, exception for a well working specification part)

Update 2

Repository is so much more than just accessing the database in a abstracted way such as can be done by ORM's. A normal Repository implementation should handle all aggregate entities (for instance Order and OrderLine). Bu handling them in the same repository class you can always make sure that those are built correctly.

But hey you say: That's done automatically for me by the ORM. Well, yes and no. If you create a website, you most likely want to edit only one order line. Do you fetch the complete order, loop through it to find the order, and then add it to the view?

By doing so you introduce logic to your controller that do not belong there. How do you do it when a webservice want's the same thing? Duplicate your code?

By using a ORM it's quite easy to fetch any entity from anywhere myOrm.Fetch(user => user.Id == 1) modify it and then save it. This can be quite handy, but also add code smells since you duplicate code and have no control over how the objects are created, if they got a valid state or correct associations.

The next thing that comes to mind is that you might want to be able to subscribe on events like Created, Updated and Deleted in a centralized way. That's easy if you have a repository.

For me an ORM provides a way to map classes to tables and nothing more. I still like to wrap them in repositories to have control over them and get a single point of modification.