メソッドを持つASP.NET MVC ViewModel - それは "合法"ですか？

Question

viewmodelsはメソッドではなくプロパティのみを持つように制限する必要がありますか？

私のビューにラジオボタンがあり、ラジオボタンをチェックする必要があるかどうかを確認したいとします。

私は完全に私の見解でこれを行うことができます

：のviewmodelにこの方法を使用して

@Html.RadioButton("radiobuttonName", "The value", Model.IsChecked(id) 

：：私はのviewmodelにこのロジックを移動することができ

@Html.RadioButton("radiobuttonName", "The value", (id == Model.PersonId)) 

または

public int PersonId { get;set;} 
    public bool IsChecked(int id) 
    { 
     return (id == PersonId); 
    } 

これは問題ありませんか、それとも完全にビューで行うべきか、その他の方法で行うべきですか？

Answer 1

ViewModelにはメソッドがあります。あなたが毎回計算したい単一の結果である場合は、Controllerに評価コードを追加し、結果をViewModelに保存することをお勧めします。ただし、メソッドをより動的に使用して評価する必要がある場合、プロパティではできませんこれはあなたのために、ViewModelでこれをやっているのは大丈夫でしょう。

上記の例では、ViewModelにこれを行うことをお勧めします。ViewModelは、これを複数回コピーしてViewに貼り付けるのではなく、1か所にロジックが含まれています。

Answer 2

論理の正しい配置に関する本当に良い質問です。あなたのアプローチは確かに法律ですが、それはMVCの精神に従っていますか？ :)

あなたのメソッドのロジックがこのView/ViewModelにのみ適用されるか、この基になるモデルタイプを扱う他のViewModelsに適用される可能性があるかどうかによって異なります（Person ）。

これは、この特定のViewModelの目的のための1回限りの計算である場合は、モデルに保持します。この計算をPersonオブジェクトで一般的に使用する場合は、静的なServiceクラスを使用することを検討してください。 PersonServiceをクリックし、そこにメソッドを配置します。