Meilleure pratique: plusieurs entités à entité unique

Question

Il s'agit d'une question très difficile à expliquer et j'espère que mon extrait de code en explique la majeure partie.

Disons que vous avez la conception de base de données suivante: musicstyle relations http://img190.yfrog.com/img190/2080/musicstylerelations.jpg

Et vous voulez construire une interface générique pour modifier les relations entre musicstyle les trois entités. Actuellement, j'ai créé un MusicStyleController qui nécessite le type d'entité auquel il est associé (membre, événement ou groupe). Je me rend malade en écrivant un tel code, mais je ne peux pas trouver un autre moyen, plus élégant.

Notez que cette fonction est appelée à l'aide de jquery.post().

La question

Comment décririez-vous factoriser ce code, et seriez-vous normaliser encore plus la base de données? Gardez à l'esprit que j'utilise Entity Framework comme modèle de données.

Answer 1

En supposant que id représente l'identifiant du membre, vous pouvez créer 3 fonctions distinctes pour gérer chaque type, séparant ainsi vos préoccupations plus qu'elles ne le sont actuellement.

Exemple:

[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(); 
} 

Ensuite, vous simplement modifier votre message jquery pour aller aux méthodes respectives en fonction de ce que vous essayez de faire.

Answer 2

Comment refactoriez-vous ce code?

1) Le code qui vérifie que l'utilisateur est connecté doit être déplacé:

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

C'est une préoccupation transversale, qui devrait être appliquée au moyen d'un cadre de sécurité, printemps vient à l'esprit comme Exemple. 2) J'éviterais d'utiliser un motif singleton pour représenter ces cas d'utilisation, ils pourraient rapidement se transformer en une collection de scripts qui, une fois grands, peuvent être difficiles à savoir où placer le code. Envisagez d'utiliser le modèle de commande à la place.

Ce modèle vous permettra de renvoyer les résultats au format JSON, XML ou tout autre format basé sur les interfaces que votre commande doit également respecter.

class DeleteMusicStyleByBandCommand : JsonResultModelCommand, XmlResultModelCommand { 

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

    public void execute() { 
    .. 
    } 

    public JsonResult getJsonResult() { .. } 

    public XmlResult getXmlResult() { .. } 
} 

Le modèle de commande à mon humble avis est beaucoup mieux à représenter des cas d'utilisation que de nombreuses méthodes dans un service ..