EDIT: I made something much better to fill and read data from a view using ViewModels, called it ValueInjecter. http://valueinjecter.codeplex.com/documentation
using the ViewModel to store the mapping logic was not such a good idea because there was repetition and SRP violation, but now with the ValueInjecter I have clean ViewModels and dry mapping code
I made a ViewModel pattern for editing stuff in asp.net mvc
this pattern is usefull when you have to make a form for editing an entity and you have to put on the form some dropdowns for the user to choose some values
public class OrganisationViewModel
{
//paramterless constructor required, cuz we are gonna get an OrganisationViewModel object from the form in the post save method
public OrganisationViewModel() : this(new Organisation()) {}
public OrganisationViewModel(Organisation o)
{
Organisation = o;
Country = new SelectList(LookupFacade.Country.GetAll(), "ID", "Description", CountryKey);
}
//that's the Type for whom i create the viewmodel
public Organisation Organisation { get; set; }
#region DropDowns
//for each dropdown i have a int? Key that stores the selected value
public IEnumerable<SelectListItem> Country { get; set; }
public int? CountryKey
{
get
{
if (Organisation.Country != null)
{
return Organisation.Country.ID;
}
return null;
}
set
{
if (value.HasValue)
{
Organisation.Country = LookupFacade.Country.Get(value.Value);
}
}
}
#endregion
}
and that's how i use it
public ViewResult Edit(int id)
{
var model = new OrganisationViewModel(organisationRepository.Get(id));
return View(model);
}
[AcceptVerbs(HttpVerbs.Post)]
public ActionResult Edit(OrganisationViewModel model)
{
organisationRepository.SaveOrUpdate(model.Organisation);
return RedirectToAction("Index");
}
and the markup
<p>
<label for="Name">
Name:</label>
<%= Html.Hidden("Organisation.ID", Model.Organisation.ID)%>
<%= Html.TextBox("Organisation.Name", Model.Organisation.Name)%>
<%= Html.ValidationMessage("Organisation.Name", "*")%>
</p>
<p>
...
<label for="CountryKey">
Country:</label>
<%= Html.DropDownList("CountryKey", Model.Country, "please select") %>
<%= Html.ValidationMessage("CountryKey", "*") %>
</p>
so tell me what you think about it
