ベストプラクティス：複数のエンティティから単一のエンティティへ

Question

これは説明するのが非常に難しいので、コード抽出がそのほとんどを説明してくれることを願っています。 musicstyle relations http://img190.yfrog.com/img190/2080/musicstylerelations.jpg

そして、あなたはすべての3つのエンティティ間のmusicstyle関係を修正するための1つの汎用のインターフェイスを構築したい：

は、あなたが次のデータベースの設計を持っているとしましょう。現在、関連するエンティティ（メンバー、イベント、バンド）のタイプを必要とするMusicStyleControllerを作成しました。

[AcceptVerbs(HttpVerbs.Post)] 
public JsonResult DeleteMusicStyle(int id, string type, int typeid) 
{ 
    if (!(Session["MemberLoggedIn"] is Member)) return Json(string.Empty); 
    Member member = (Member)Session["MemberLoggedIn"]; 

    switch (type) { 
     case "member": 
      _memberService.DeleteMusicStyle(member, id); 
      break; 
     case "band": 
      Band band = _bandService.GetBand(typeid); 
      _bandService.DeleteMusicStyle(band, id); 
      break; 
     case "event": 
      Event @event = _eventService.GetEvent(typeid); 
      _bandService.DeleteMusicStyle(@event, id); 
      break; 
    } 
    return SelectedMusicStyles(); 
} 

私は自分自身で病気のようなコードを書いていますが、別のよりエレガントな方法を見つけることはできません。

この関数は、jquery.post（）を使用して呼び出されます。

質問

はどのようにこのコードをリファクタリングでしょう、そしてあなたも、より多くのデータベースを正規化するのでしょうか？私はデータモデルとしてEntity Frameworkを使用しています。

Answer 1

idがメンバーのIDを表していると仮定すると、それぞれのタイプを処理する3つの別々の関数を作成することができます。

例：

[AcceptVerbs(HttpVerbs.Post)] 
public JsonResult DeleteMusicStyleByMember(int id) 
{ 
    if (!(Session["MemberLoggedIn"] is Member)) return Json(string.Empty); 
    Member member = (Member)Session["MemberLoggedIn"]; 
    _memberService.DeleteMusicStyle(member, id); 
    return SelectedMusicStyles(); 
} 

[AcceptVerbs(HttpVerbs.Post)] 
public JsonResult DeleteMusicStyleByBand(int id, int typeid) 
{   
    Band band = _bandService.GetBand(typeid); 
    _bandService.DeleteMusicStyle(band, id); 
    return SelectedMusicStyles(); 
} 

[AcceptVerbs(HttpVerbs.Post)] 
public JsonResult DeleteMusicStyleByEvent 
    (int id, int typeid) 
{ 
    Event event = _eventService.GetEvent(typeid); 
    _bandService.DeleteMusicStyle(event, id); 
    return SelectedMusicStyles(); 
} 

次に、あなたはちょうどあなたがやろうとしているものに応じて、それぞれの方法に行くためにあなたのjqueryのポストを変更します。

Answer 2

このコードをどのようにリファクタリングしますか？

1）にログインしているユーザが移動すべきチェックコード：

if (!(Session["MemberLoggedIn"] is Member)) return Json(string.Empty); 
    Member member = (Member)Session["MemberLoggedIn"]; 

これはセキュリティフレームワークを使用して適用されるべき横断的関心事であり、春のように気にポップ例。

2）私は、このユースケースを表現するためにシングルトンパターンを使用することを避け、すぐに大規模になるとコードをどこに配置するのかを知るのが難しいスクリプトの集まりになります。代わりにコマンドパターンの使用を検討してください。

このパターンを使用すると、JSON、XML、またはコマンドに準拠させるインターフェイスに基づく他の形式で結果を返すことができます。

class DeleteMusicStyleByBandCommand : JsonResultModelCommand, XmlResultModelCommand { 

    public DeleteMusicStyleByBand(int id, int typeid) { 
    //set private members 
    } 

    public void execute() { 
    .. 
    } 

    public JsonResult getJsonResult() { .. } 

    public XmlResult getXmlResult() { .. } 
} 

Commandパターンの私見は、サービスの多くの方法よりもユースケースを表すではるかに優れています。..