ASP.NET MVC

Question

を使用して一つに作成と編集ActionResultsマージ私は現在、4つのアクションメソッドがあります：

UserCreateを -
UserCreateを取得 - 投稿

UserEdit -
UserEditを取得 - 投稿

私の目標は、どういうわけかそれらを結合することだけですUserEdit。この2つの唯一の違いは、UserCreateがパスワードとConfirmPasswordをフォームに要求することです。 UserEditはありません。私はこのために2つの別々のViewModelを作成する必要がありました：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); 
} 

分離それらを保つことは結構です、より良い、よりMVC大会ですが、私はちょうどMVCのいくつかの機能は、私はそれを利用しておりませんということがないことを確認したい、私は唯一の2アクションメソッドを持って助けることができる場合、これに対する1つのビューモデル。

Answer 1

別々に保管してください。あなたはMVCの中で、これを「周りに」働くことを目的とした何かが欠けているわけではありません。確かにあなたはそれを動作させることができますが、それは彼らが似たような行動であると考えるために共通のAPIの実装から逸脱します。それが本当に悪いと感じるなら、同じビューモデルとコードバリデーションをページとサーバー側で使用することができます（1つのモデルに対して2つのシナリオが必要なので、モデルに依存することはできませんでした）。二。清潔です。

Answer 2

はい、あなたは確かにだけMVC機能を使用して2methods持つことができます。

あなたはPOSTタイプのためのメソッドを作成することができますGETタイプのため

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

を作成するための方法または編集するユーザーを作成することができます。ここではすべての属性を持つ新しいモデルのUserViewModelを作成します。

[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：ただし、この方法を使用することはお勧めしません。