Tuesday, September 27, 2016

ASP.NET MVC View Model Patterns

Copyright from: stevemichelotti.com
Since MVC has been released I have observed much confusion about how best to construct view models. Sometimes this confusion is not without good reason since there does not seem to be a ton of information out there on best practice recommendations.  Additionally, there is not a "one size fits all" solution that acts as the silver bullet. In this post, I'll describe a few of the main patterns that have emerged and the pros/cons of each. It is important to note that many of these patterns have emerged from people solving real-world issues.
Another key point to recognize is that the question of how best to construct view models is not unique to the MVC framework.  The fact is that even in traditional ASP.NET web forms you have the same issues.  The difference is that historically developers haven't always dealt with it directly in web forms – instead what often happens is that the code-behind files end up as monolithic dumping grounds for code that has no separation of concerns whatsoever and is wiring up view models, performing presentation logic, performing business logic, data access code, and who knows what else.  MVC at least facilitates the developer taking a closer look at how to more elegantly implement Separation of Concerns.
Pattern 1 – Domain model object used directly as the view model
Consider a domain model that looks like this:
?
1
2
3
4
5
6
7
public class Motorcycle 
{
    public string Make { getset; }
    public string Model { getset; }
    public int Year { getset; }
    public string VIN { getset; }
}
When we pass this into the view, it of course allows us to write simple HTML helpers in the style of our choosing:
?
1
2
<%=Html.TextBox("Make") %>
<%=Html.TextBoxFor(m => m.Make) %>
And of course with default model binding we are able to pass that back to the controller when the form is posted:
?
1
public ActionResult Save(Motorcycle motorcycle) 
While this first pattern is simple and clean and elegant, it breaks down fairly quickly for anything but the most trivial views. We are binding directly to our domain model in this instance – this often is not sufficient for fully displaying a view.
Pattern 2 – Dedicated view model that *contains* the domain model object
Staying with the Motorcycle example above, a much more real-world example is that our view needs more than just a Motorcycle object to display properly.  For example, the Make and Model will probably be populated from drop down lists. Therefore, a common pattern is to introduce a view model that acts as a container for all objects that our view requires in order to render properly:
?
1
2
3
4
5
6
public class MotorcycleViewModel 
{
    public Motorcycle Motorcycle { getset; }
    public SelectList MakeList { getset; }
    public SelectList ModelList { getset; }
}
In this instance, the controller is typically responsible for making sure MotorcycleViewModel is correctly populated from the appropriate data in the repositories (e.g., getting the Motorcycle from the database, getting the collections of Makes/Models from the database).  Our Html Helpers change slightly because they refer to Motorcycle.Make rather than Make directly:
?
1
<%=Html.DropDownListFor(m => m.Motorcycle.Make, Model.MakeList) %>
When the form is posted, we are still able to have a strongly-typed Save() method:
?
1
public ActionResult Save([Bind(Prefix = "Motorcycle")]Motorcycle motorcycle) 
Note that in this instance we had to use the Bind attribute designating "Motorcycle" as the prefix to the HTML elements we were interested in (i.e., the ones that made up the Motorcycle object).
This pattern is simple and elegant and appropriate in many situations. However, as views become more complicated, it also starts to break down since there is often an impedance mismatch between domain model objects and view model objects.
Pattern 3 – Dedicated view model that contains a custom view model entity
As views get more complicated it is often difficult to keep the domain model object in sync with concerns of the views.  In keeping with the example above, suppose we had requirements where we need to present the user a checkbox at the end of the screen if they want to add another motorcycle.  When the form is posted, the controller needs to make a determination based on this value to determine which view to show next. The last thing we want to do is to add this property to our domain model since this is strictly a presentation concern. Instead we can create a custom "view model entity" instead of passing the actual Motorcycle domain model object into the view. We'll call it MotorcycleData:
?
1
2
3
4
5
6
7
8
public class MotorcycleData 
{
    public string Make { getset; }
    public string Model { getset; }
    public int Year { getset; }
    public string VIN { getset; }
    public bool AddAdditionalCycle { getset; }
}
This pattern requires more work and it also requires a "mapping" translation layer to map back and forth between the Motorcycle and MotorcycleData objects but it is often well worth the effort as views get more complex.  This pattern is strongly advocated by the authors of MVC in Action (a book a highly recommend).  These ideas are further expanded in a post by Jimmy Bogard (one of the co-authors) in his post How we do MVC – View Models. I strongly recommended reading Bogard's post (there are many interesting comments on that post as well). In it he discusses approaches to handling this pattern including using MVC Action filters andAutoMapper (I also recommend checking out AutoMapper).
Let's continue to build out this pattern without the use of Action filters as an alternative. In real-world scenarios, these view models can get complex fast.  Not only do we need to map the data from Motorcycle to MotorcycleData, but we also might have numerous collections that need to be populated for dropdown lists, etc.  If we put all of this code in the controller, then the controller will quickly end up with a lot of code dedicated just to building the view model which is not desirable as we want to keep our controllers thin. Therefore, we can introduce a "builder" class that is concerned with building the view model.
?
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
public class MotorcycleViewModelBuilder 
{
    private IMotorcycleRepository motorcycleRepository;
  
    public MotorcycleViewModelBuilder(IMotorcycleRepository repository)
    {
        this.motorcycleRepository = repository;
    }
  
    public MotorcycleViewModel Build()
    {
        // code here to fully build the view model
        // with methods in the repository
    }
}
This allows our controller code to look something like this:
?
1
2
3
4
5
6
public ActionResult Edit(int id) 
{
    var viewModelBuilder = new MotorcycleViewModelBuilder(this.motorcycleRepository);
    var motorcycleViewModel = viewModelBuilder.Build();
    return this.View();
}
Our views can look pretty much the same as pattern #2 but now we have the comfort of knowing that we're only passing in the data to the view that we need – no more, no less.  When the form is posted back, our controller's Save() method can now look something like this:
?
1
2
3
4
5
6
7
public ActionResult Save([Bind(Prefix = "Motorcycle")]MotorcycleData motorcycleData) 
{
    var mapper = new MotorcycleMapper(motorcycleData);
    Motorcycle motorcycle = mapper.Map();
    this.motorcycleRepository.Save(motorcycle);
    return this.RedirectToAction("Index");
}
Conceptually, this implementation is very similar to Bogard's post but without the AutoMap attribute.  The AutoMap attribute allows us to keep some of this code out of the controller which can be quite nice.  One advantage to not using it is that the code inside the controller class is more obvious and explicit.  Additionally, our builder and mapper classes might need to build the objects from multiple sources and repositories. Internally in our mapper classes, you can still make great use of tools like AutoMapper.
In many complex real-world cases, some variation of pattern #3 is the best choice as it affords the most flexibility to the developer.
Considerations
How do you determine the best approach to take?  Here are some considerations to keep in mind:
Code Re-use – Certainly patterns #1 and #2 lend themselves best to code re-use as you are binding your views directly to your domain model objects. This leads to increased code brevity as mapping layers are not required. However, if your view concerns differ from your domain model (which they often will) options #1 and #2 begin to break down.
Impedance mismatch – Often there is an impedance mismatch between your domain model and the concerns of your view.  In these cases, option #3 gives the most flexibility.
Mapping Layer – If custom view entities are used as in option #3, you must ensure you establish a pattern for a mapping layer. Although this means more code that must be written, it gives the most flexibility and there are libraries available such as AutoMapper that make this easier to implement.
Validation – Although there are many ways to perform validation, one of the most common is to use libraries like Data Annotations. Although typical validations (e.g., required fields, etc.) will probably be the same between your domain models and your views, not all validation will always match.  Additionally, you may not always be in control of your domain models (e.g., in some enterprises the domain models are exposed via services that UI developers simply consume).  So there is a limit to how you can associate validations with those classes.  Yes, you can use a separate "meta data" class to designate validations but this duplicates some code similar to how a view model entity from option #3 would anyway. Therefore, option #3 gives you the absolute most control over UI validation.
Conclusion
The following has been a summary of several of the patterns that have emerged in dealing with view models. Although these have all been in the context of ASP.NET MVC, the problem with how best to deal with view models is also an issue with other frameworks like web forms as well. If you are able to bind directly to domain model in simple cases, that is the simplest and easiest solution. However, as your complexity grows, having distinct view models gives you the most overall flexibility.