ASP.NET MVC application controller constructor problem

Question

I've created an application controller abstract class that my controllers derive from (As described in the following article)

The following is an example of what my code looks like

public abstract class ApplicationController : Controller
{
    private ProjectDataContext datacontext = new ProjectDataContext();

    protected ProjectDataContext DataContext
    {
        get { return datacontext; }
    }

    public ApplicationController()
    {
        ViewData["OpenTasks"] = DataContext.Tasks.Where(t => t.UserId == this.UserId).Count();
    }
}

This produces the following error which i have determined is due to the "Where" lamda expression:

If the controller doesn't have a controller factory, ensure that it has a parameterless public constructor.

this error is produced whichever way i write the LINQ query and the only way to compile the application is by removing the "Where" clause as follows.

ViewData["OpenTasks"] = DataContext.Tasks.Count();

any ideas what the problem is or how to resolve this as i need to execute the query against the user and not return all entries.

thanks in advance

Show implementation of UserId property

AnthonyWJones
– AnthonyWJones

2009-01-13 13:19:02 +00:00
Commented Jan 13, 2009 at 13:19 — AnthonyWJones
– AnthonyWJones, Commented Jan 13, 2009 at 13:19

AnthonyWJones · Accepted Answer · 2009-01-13 13:22:44Z

10

Try this instead of using the constructor:-

public abstract class ApplicationController : Controller
{
    private ProjectDataContext datacontext = new ProjectDataContext();

    protected ProjectDataContext DataContext
    {
        get { return datacontext; }
    }

    protected override void Initialize(System.Web.Routing.RequestContext requestContext)
    {
        base.Initialize(RequestContext);
        ViewData["OpenTasks"] = DataContext.Tasks.Where(t => t.UserId == this.UserId).Count();
    }
}

Its quite likely that the current user ID is dependand on the RequestContext

answered Jan 13, 2009 at 13:22

AnthonyWJones

190k35 gold badges236 silver badges307 bronze badges

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

Comments

Kieron · Accepted Answer · 2009-01-13 13:19:55Z

1

It could be that the call is crashing in the constructor because the UserId (this.UserId) hasn't been initialised yet.

answered Jan 13, 2009 at 13:19

Kieron

27.2k16 gold badges81 silver badges124 bronze badges

Collectives™ on Stack Overflow

ASP.NET MVC application controller constructor problem

2 Answers 2

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related