Use same actionresultmethod for multiple viewmodels?

Question

I have three viewmodels to edit the same domain object. I use AutoMapper to map to viewmodels from the domain object.

GET

if (Roles.IsUserInRole("administrators"))
{
    viewName = "EditAdmin";
    editOrder = Mapper.Map<Order, ViewModels.Order.Admin_ViewModel>(order);
}
else if (Roles.IsUserInRole("administrators"))
{
    viewName = "EditTechnician";
    editOrder = Mapper.Map<Order, ViewModels.Order.Technician_ViewModel>(order);
}
else if (Roles.IsUserInRole("clients"))
{
    viewName = "EditClient";
    editOrder = Mapper.Map<Order, ViewModels.Order.Client_ViewModel>(order);
}

RedirectToRoute(viewName, editOrder);

POST Is it possible to use the same method for all viewmodels? Possibly by letting the ViewModels inherit from "ViewModelbase"? Tried this with no success :(

Many thanks in advance!

EDIT: This is what my method looks like now:

public ActionResult EditAdmin(ViewModels.Order.Admin_ViewModel model) {...}

Any ideas?

UPDATE: I wasn't able to understand how to use composition (tried it for a few hours). So I went with this to clean up a bit. These are the handlers for my three viewmodels:

[HttpPost]
[Authorize(Roles = "administrators")]
public ActionResult EditAdmin(Admin_ViewModel model)
{
    return SaveViewModel(model);
}

[HttpPost]
[Authorize(Roles = "technicians")]
public ActionResult EditTechnician(Technician_ViewModel model)
{
    return SaveViewModel(model);
}

[HttpPost]
[Authorize(Roles = "clients")]
public ActionResult EditClient(Client_ViewModel model)
{
    return SaveViewModel(model);
}

SaveViewModel looks like this:

protected ActionResult SaveViewModel(dynamic model)
{ ... }

I don't feel to good about this solution. Could you please give me some pointers?

Rajeesh · Accepted Answer · 2012-05-09 13:34:30Z

1

IMHO - ViewModel should plain and simple. If you introduce inheritance, it is going get complicated.

I suggest you to use composition to build the ViewModel you want. So if the user is in a particular role, objects which are not related to the Role will be null when the page is posted. With that you can stick to one POST method rather than many.

Update : If you really want to have the inheritance in ViewModel. Custom model binder is the solution, if the default one is not working

edited May 9, 2012 at 13:34

answered May 9, 2012 at 13:26

Rajeesh

4,5154 gold badges28 silver badges29 bronze badges

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

1 Comment

Rajeesh Over a year ago

@JimH Could you please share the three ViewModels, then I could give a try.

Collectives™ on Stack Overflow

Use same actionresultmethod for multiple viewmodels?

1 Answer 1

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related