Ce code MVC peut-il être refactored à l'aide d'un modèle de conception?

Peut-être que l'objet post devrait se mettre à jour et être passé un irpostage (qui a elle-même été transmise au contrôleur.) (Il s'agit d'injection de dépendance de base / IOC, et conserve le contrôleur SKINNIER)

//in controller:
var post = Mapper.Map<PostViewModel, Post>(viewModel);
post.Update(_postRepository);

//inside Post.cs:
public Update(IRepository rep){
//update db with the repo    
//give points
}

Vous pouvez utiliser Programmation orientée verso sur ce problème particulier. Le produit couramment utilisé dans le monde .net est postSharp .

L'idée serait que vous ajoutez un attribut au-dessus de la méthode. L'attribut indiquera quelles actions particulières doivent être effectuées (dans votre cache de cas de rafraîchissement, des points d'augmenter, etc.) et quand il devrait arriver (dans votre cas lors de la sortie de la méthode par exemple).

Vous pouvez également les séparer à différents attributs, de sorte que vous puissiez faire différents types de combinaisons.

Vous voudrez peut-être envisager d'implémenter un système de bus de service tel que NSERVICEBUS si vous avez affaire à un casse-cache ou à la vérification ( cela peut être mieux servi de manière asynchrone).

Avec un bus de messagerie, des messages d'événement peuvent être publiés sur n'importe quel nombre de gestionnaires d'événements (par exemple un gestionnaire de cache) de manière asynchrone, de sorte que votre application peut déclencher un message, puis continuer à servir rapidement des pages synchrones.

1. Je pense que vous devez envisager transaction forte>. Si vous essayez de mettre à jour les caches dans un autre fil et de renvoyer le résultat d'action immédiatement après le référentiel.Save. Il est difficile de faire rouler si les opérations de cache jettent une erreur (la plupart des caches sont globales, qui sont généralement confrontées à des problèmes de verrouillage / synchronisation). LI>

2. MVC est un motif de couche d'interface utilisateur. J'ai généralement placé une logique contextuelle HTTP dans le corps d'action et mettez la logique commerciale dans des couches inférieures, par exemple un service, qui gère le modèle de domaine. Dans votre cas, j'aimerais ajouter un iPostservice et mettre votre fonction de sauvegarde et votre fonction mise en cache. Qui est quelque chose comme: p>

   //codes in your PostService which implements IPostService
   [CacheEvent("POST")]
   [Transaction]
   public void Save(Post post) //care about domain model instead of view model
   {
      // Save.
      _postReposity.Save(post);
   }
   

3. De plus, je pense qu'il est bon d'utiliser AOP pour déclencher un événement et d'utiliser un conteneur de COI pour injecter la logique de cache (ainsi que la logique de transaction). Quelque chose comme: p>

   [HttpPost]
   public ActionResult Save(PostViewModel viewModel)
   {
      // Map.
      var post = Mapper.Map<PostViewModel, Post>(viewModel);
   
      // Save.
      _postService.Save(post);
   
      // Redirect.
      return RedirectToAction("Index");
   }
   

La réponse de @ jgauuffine a accepté, mais je pensais ajouter une réponse séparée depuis que j'utilise Structuremap, et il avait besoin une étape supplémentaire .

J'ai suivi ce que @JGAuffin spécifié, mais ce n'est pas " t tisser mon gestionnaire d'événements.

Alors j'ai ajouté ceci à mon structureMap config: xxx

alors ça marche.

i ' M Devinant l'alternative à ce sujet est de spécifier manuellement chaque auditeur enregistré, qui obtient un peu fou.

Quelqu'un qui utilise Structuremap me dire s'il s'agit de la bonne approche?