MVC5のViewModelでModelクラスをインスタンス化する必要はありますか？

Question

私はMVC5を初めて使い、ViewModelsの使用を把握しようとしています。私の質問は、自分のViewModelのコントローラーでモデルクラスをインスタンス化する必要があるかどうかです。私は、請求書を整理するためのデータベースを作成しており、データベーステーブルにマップするCashモデルと、データを表示/編集するCashViewModelを持っています。ここで

は私のモデルである：ここでは

public class Cash 
{ 
    public int CashId { get; set; } 
    public decimal CashAmount { get; set; } 
    public int CashTypeId { get; set; } 
    public DateTime DateModified { get; set; } 
    public CashType CashType { get; set; } 
} 

は私のViewModelである：ここでは

public class CashViewModel 
{ 
    public int CashId { get; set; } 
    public decimal CashAmount { get; set; } 
    public int CashTypeId { get; set; } 
    public SelectList CashTypeSelectList { get; set; } 
} 

は私のコントローラです：

public ActionResult Create([Bind(Include = "CashAmount,CashTypeId")] CashViewModel cashVM) 
    { 
     if (ModelState.IsValid) 
     { 
      var cash = db.Cashes.Create(); 
      cash.DateModified = DateTime.Now; 
      cash.CashAmount = cashVM.CashAmount; 
      cash.CashTypeId = cashVM.CashTypeId; 
      db.Cashes.Add(cash); 
      db.SaveChanges(); 
      return RedirectToAction("Index"); 
     } 

     ViewBag.CashTypeId = new SelectList(db.CashTypes, "CashTypeId", "CashTypeName", cashVM.CashTypeId); 
     return View(cashVM); 
    } 

私は現金モデルのインスタンスを作成していますViewModelクラスをモデルクラスにマッピングします。これは正しいです？ 1つのViewModelに複数の属性を持つ複数のモデルがあるプロジェクトを開始すると、すぐに手を抜けてしまうようです。

私が読んだすべての質問と記事は、あまりにもあいまいであるか、あまりにも細かすぎて、特定の質問には何も見つかりません。

Answer 1

ViewModelsは基本的にビューとコントローラの間にDTOを提供することです。しかし、あなたのモデルは間違っているように見えます.CashTypeIdではなく、CashTypeのみである必要があります.CashTypeモデルでは、IdとCashには参照があります。両方を持つ問題は、それらが同期していないということです。実際にEFが信頼できるものを選んでいることを覚えていません。

それ以外はあなたが理解したようです。 ViewModelには、ビューをレンダリングする値と、クライアントが値を割り当てるためのCashTypeIdを持つSelectListがあります。プライマリエンティティにCashTypeIdがなければ、新しいCashTypeオブジェクトを作成し、適切なIDを割り当てる必要があります（Entity Frameworkの場合）。

最後のアドバイス1つ：コントローラでこれらのDB操作を実行しないでください。 repository patternをチェックし、それらの間にビジネスレイヤーを置くことさえできます。

Answer 2

私はCashモデルのインスタンスを作成し、ViewModelクラスをモデルクラスにマッピングしています。これは正しいです？ 1つのViewModelに複数の属性を持つ複数のモデルがあるプロジェクトを開始すると、すぐに手を抜けてしまうようです。

あなたの行っていることに何も問題はありません。ただし、ドメインモデルからビューのモデル（ビューモデル）にマッピングする場合は、ライブラリを使用してマッピングを行うことができます。私はAutoMapperを使用します。あなたは、この持っている想像して：あなたのApplication_Start() startメソッドで

public class Cash { 
    public string Prop1 { get; set; } 
} 

public class CashViewModel { 
    public string Prop1 { get; set; } 
} 

を、あなたはこのようなビューのためにあなたのモデルにドメインモデルからマッピングする方法AutoMapperを指示することができます。

protected void Application_Start() 
{ 
    // ReverseMap will do vice-versa mapping 
    // If property names are the same, then AutoMapper will take care of it 
    // for you. 
    Mapper.Initialize(x => x.CreateMap<Cash, CashViewModel>().ReverseMap()); 

    // Of course you will have many cases where the property names 
    // are different 
    // For that case you will need to do this 
    // Imagine your model has a property called Name and view model has 
    // FirstName 
    Mapper.CreateMap<CashViewModel, Cash>() 
      .ForMember(model => model.Name, map => map.MapFrom(vm => vm.FirstName)); 
} 

次に、あなたのコントローラであなたはビューモデルからドメインモデルに変換するにはこれを行います：

var model = Mapper.Map<Cash>(cashViewModel); 

このアプローチの美しさは、すべてのマッピングが1か所に収まることです。結論として、hereはあなたが探しているものの大部分をカバーする素晴らしい記事です。