fortement typées Vues modèle sur un RedirectToAction

Question

J'ai deux méthodes d'action qui ressemblent à ce

[HttpPost] 
    public ActionResult Search(Models.InputModel input) 
    { 
     if (!IsSearchCriteriaValid(input)) 
      return RedirectToAction("Index"); 

     TempData[TempDataSearchInput] = input; 

     return RedirectToAction("List"); 
    } 

    public ActionResult List() 
    { 
     var input = TempData[TempDataSearchInput] as Models.InputModel; 

     if (!IsSearchCriteriaValid(input)) 
      return RedirectToAction("Index"); 

     var result = new List<MyDTO>(); 

     AutoMapper.Mapper.Map(_repository.GetBy(input), results); 

     var model = new Models.DisplayListModel { Result = result }; 
     return View("List", model); 
    } 

Est-il possible les meilleures pratiques standard pour faire quelque chose comme ça?

Answer 1

Oui, vous devez simplement supprimer l'action de recherche en tant que son code de duplication juste (même code que la liste), puis rediriger vers la liste! Donc, le code est redondant. Si vous le souhaitez, vous pouvez renommer votre action combine pour donner un sens plus significatif à votre code maintenant que vous les avez combinés, peut-être SearchToList() ou quelque chose. Ne pas utiliser les redirections lorsqu'elles ne sont pas nécessaires est la meilleure pratique. Faites-moi savoir si cela répond à votre question ou si je peux faire quelque chose de plus pour vous, merci.

Answer 2

Erx_VB.NExT.Coder est correct. Le code dans l'action de recherche n'est pas nécessaire. Je suppose que vous l'avez fait, parce que votre formulaire est posté sur/[controller]/Search? Vous pouvez toujours utiliser la vue search.aspx si vous le souhaitez et pointer simplement le formulaire vers/[controller]/List comme ci-dessous.

<% using (Html.BeginForm("foo", "bar", FormMethod.Post, new { id = "myID" })) 
     { %> 
    <%} %> 
will result in the following HTML: 

<form action="/bar/foo" id="myID" method="post"></form> 

Answer 3

Je suis d'accord que vous devriez les combiner en un seul ActionResult. Une autre façon de le faire serait de créer un itinéraire personnalisé. Donc, dans votre fichier Global.asax, ajoutez ce qui suit à votre fonction RegisterRoutes:

routes.MapRoute("MySearch", "MyController/Search", 
    new { controller = "MyController", action = "List" } 
); 

qui tracera automatiquement tous les appels à la recherche à la liste, et supprimera la nécessité d'avoir les deux ActionMethods définis dans votre code