views:

996

answers:

3

I'm building a 4 layered ASP.Net web application. The layers are:

  1. Data Layer
  2. Entity Layer
  3. Business Layer
  4. UI Layer

The entity layer has my data model classes and is built from my entity data model (edmx file) in the datalayer using T4 templates (POCO). The entity layer is referenced in all other layers.

My data layer has a class called SourceKeyRepository which has a function like so:

public IEnumerable<SourceKey> Get(SourceKey sk)
{
    using (dmc = new DataModelContainer())
    {
        var query = from SourceKey in dmc.SourceKeys
                    select SourceKey;

        if (sk.sourceKey1 != null)
        {
            query = from SourceKey in query
                    where SourceKey.sourceKey1 == sk.sourceKey1
                    select SourceKey;
        }

        return query;
    }
}

Lazy loading is disabled since I do not want my queries to run in other layers of this application. I'm receiving the following error when attempting to access the information in the UI layer:

The ObjectContext instance has been disposed and can no longer be used for operations that require a connection.

I'm sure this is because my DataModelContainer "dmc" was disposed. How can I return this IEnumerable object from my data layer so that it does not rely on the ObjectContext, but solely on the DataModel?

Is there a way to limit lazy loading to only occur in the data layer?

+4  A: 

query is lazy evaluated so the data is not retreived from the database until you enumerate it.

If you do:

return query.AsEnumerable();

you will force the query to be executed and avoid the problem.

You are receiving the error message because when the caller enumerates the collection, the ObjectContext (dmc) is already disposed thanks to your using clause (which is good - dispose database related resources early!)

Anders Abel
Just tried it:'System.Linq.IQueryable<Entities.SourceKey>' does not contain a definition for 'ToEnumerable' and no extension method 'ToEnumerable' accepting a first argument of type 'System.Linq.IQueryable<Entities.SourceKey>' could be found (are you missing a using directive or an assembly reference?)
Chris Klepeis
Sorry, I was confusing the names - it should be `AsEnumerable` just as Tomas suggests. I've updated my answer.
Anders Abel
I adjusted it and now its throwing hte original message that the ObjectContext was disposed.
Chris Klepeis
It didnt like when I had the return within the using... I took it out and now its good to go. THanks
Chris Klepeis
+2  A: 

You could call some method on the query object, for example

return query.AsEnumerable();

That should make sure you execute the query, thus making sure you don't need the object context later.

Tomas Lycken
A: 

In my opinion, this scenario has no relevance with AsEnumerable() or AsQueryable(). Try this;

 public IEnumerable<SourceKey> Get(SourceKey sk, DataModelContainer dmc) {    

    var query = from SourceKey in dmc.SourceKeys
                select SourceKey;

    if (sk.sourceKey1 != null)
    {
        query = from SourceKey in query
                where SourceKey.sourceKey1 == sk.sourceKey1
                select SourceKey;
    }

    return query;

}

And you must get this property with

using (dmc = new DataModelContainer()) {
 // GET
}
Luindale Ainion