Question

The ‘RenderPartial()’ method in ASP.NET MVC offeres a very low level of functionality. It does not provide, nor attempt to provide a true ‘sub-controller’ model *.

I have an increasing number of controls being rendered via ‘RenderPartial()’. They fall into 3 main categories :

1) Controls that are direct descendants of a specific page that use that page’s model

2) Controls that are direct descendants of a specific page that use that page’s model with an additional key of some type. Think implementation of ‘DataRepeater’.

3) Controls that represent unrelated functionality to the page they appear on. This could be anything from a banner rotator, to a feedback form, store locator, mailing list signup. The key point being it doesn’t care what page it is put on.

Because of the way the ViewData model works there only exists one model object per request – thats to say anything the subcontrols need must be present in the page model.

Ultimately the MVC team will hopefully come out with a true ‘subcontroller’ model, but until then I’m just adding anything to the main page model that the child controls also need.

In the case of (3) above this means my model for ‘ProductModel’ may have to contain a field for ‘MailingListSignup’ model. Obviously that is not ideal, but i’ve accepted this at the best compromise with the current framework – and least likely to ‘close any doors’ to a future subcontroller model.

The controller should be responsible for getting the data for a model because the model should really just be a dumb data structure that doesn’t know where it gets its data from. But I don’t want the controller to have to create the model in several different places.

What I have begun doing is creating a factory to create me the model. This factory is called by the controller (the model doesn’t know about the factory).

public static class JoinMailingListModelFactory {          public static JoinMailingListModel CreateJoinMailingListModel() {              return new JoinMailingListModel()             {                 MailingLists = MailingListCache.GetPartnerMailingLists();             };         }     }    

So my actual question is how are other people with this same issue actually creating the models. What is going to be the best approach for future compatibility with new MVC features?

---

• NB: There are issues with RenderAction() that I won’t go into here – not least that its only in MVCContrib and not going to be in the RTM version of ASP.NET-MVC. Other issues caused sufficent problems that I elected not to use it. So lets pretend for now that only RenderPartial() exists – or at least that thats what I’ve decided to use.

Answer 1

One approach I’ve seen for this scenario is to use an action-filter to populate the data for the partial view – i.e. subclass ActionFilterAttribute. In the OnActionExecuting, add the data into the ViewData. Then you just have to decorate the different actions that use that partial view with the filter.