使用ASP.NET MVC将ActionResults创建和编辑合并为一个

Question

I currently have 4 action methods:

UserCreate - Get
UserCreate - Post

UserEdit - Get
UserEdit - Post

My goal is to somehow combine them do just be UserEdit. The only difference between the two is that UserCreate asks for Password and ConfirmPassword in the form. UserEdit does not. I ended up having to create 2 separate ViewModels for this: UserCreateViewModel, UserEditViewModel.

// GET: Users/Create
[Route("users/create")]
public IActionResult UserCreate()
{
    return View();
}

// POST: Users/Create
[Route("users/create")]
[HttpPost]
[ValidateAntiForgeryToken]
public async Task<IActionResult> UserCreate(UserCreateViewModel vm)
{        
}

// GET: Users/Edit/5
[Route("users/edit/{id}")]
public async Task<IActionResult> UserEdit(string id)
{
    return View(vm);
}

// POST: Users/Edit/5
[Route("users/edit/{id}")]
[HttpPost]
[ValidateAntiForgeryToken]
public async Task<IActionResult> UserEdit(UserEditViewModel vm)
{
    return View(vm);
}

If keeping them separate is better and more MVC convention that is fine, but I just want to make sure there isn't some feature of MVC that I am not utilizing that can help me have only 2 Action methods, One view model for this.

Answer 1

Keep them separate. You aren't missing anything in MVC that is meant for working 'around' this so to say. Sure you could make it work but it strays from common api implementation for the sake of thinking they are similar actions. If it feels really bad you could use the same view model and code validations into page and server side (you couldn't rely on it being on the model as validation would now require two scenarios for one model) but if also split it out into two. It's cleaner.

Answer 2

Yes you can certainly have only 2methods using mvc feature:

You can create a method for create or edit user for GET type

[HttpGet]
[Route("users/useraddoredit/{id?}")]
public async Task<IActionResult> UserEdit(string id)
{
  if(string.isNullorEmpty(id))
  {
    return View();
  }
  return View(vm);
}

You can create a method for POST type. Here create a new model UserViewModel which has all the attributes.

[Route("users/edit/{id}/{isUserEdit?}")]
[HttpPost]
[ValidateAntiForgeryToken]
public async Task<IActionResult> UserEdit(UserViewModel vm, bool isUserEdit)
{
    if(isUserEdit)
    {
       return View(vm);
    }    
    else
    {
        //logic for add user
    }
}

PS:But its not recommended to use this approach.