Vous pensez pouvoir rendre ce code plus joli?

Question

Voici une simple vue MVC qui affiche toutes les zones dans une base de données et liste toutes les Carreras dans chaque zone sous l'en-tête.

<h2>Listado General de Carreras</h2> 

    <% foreach (var Area in (List<string>)ViewData["Areas"]) 
     { %> 

     <p><span class="titulo"><%: Area%></span></p> 

     <% foreach (var carrera in Model) 
      { 
       if (carrera.Area.Nombre == Area) 
       { %>  

       <p><%=Html.ActionLink(carrera.Nombre, "Details", new { id = carrera.ID })%></p> 

      <% } 

      } 
     }%> 

Pensez-vous que vous pouvez rendre ce code un peu plus joli/efficace?

Answer 1

Vous pouvez vous débarrasser de l'if(crrera.Area.Nombre == Area) avec LINQ. De l'apparence de celui-ci votre Model est un certain type de IEnumerable de sorte que votre boucle intérieure deviendrait

<% foreach var carrera in Model.Where(c => c.Area.Nombre == Area)) 
{ %> 
    <p><%= Html.ActionLink(carrera.Nombre, "Details", new { id = carrera.ID })%></p> 
<% } %> 

Cela aiderait pas vraiment avec « efficacité », mais il est certainement « plus joli » à mon humble avis

Couple de MVC 3 caractéristiques aideraient également sur le front de joliesse. D'abord la syntaxe Razor, et deuxième la nouvelle propriété ViewModel est de type dynamique et vous souhaitez vous débarrasser de la recherche de dictionnaire méchant sur ViewData et son casting associé-à-dire votre boucle extérieure deviendrait

<% foreach (var Area in View.Areas) 
} %> 

Vous pouvez obtenir quelque chose de similaire en créant une vue modèle spécifique et le clouage de propriété sur les zones à ce modèle

Answer 2

Je ne pense pas que vous devez passer les zones dans le ViewData. Triez le Model par région. Ensuite, vous avez juste besoin d'une seule boucle à travers Model et vous pouvez vérifier si carrera.Area.Nombre est différent du dernier que vous avez vu, et si c'est le cas, imprimez l'en-tête pour cela.

Espérons que logique, mais laissez-moi savoir si vous avez besoin d'un petit coup de pouce avec le code.

Answer 3

un peu difficile à dire sans savoir ce qui est exactement Area. Mais si c'est possible, je déplacerais des zones dans le modèle de vue, et rendrais chaque objet Area collection de carreras.

également de faire le lien d'action plus jolie que vous pouvez utiliser T4MVC.

Vous pourriez finir avec le code comme ceci:

<h2>Listado General de Carreras</h2> 

<% foreach (var area in Model.Areas) { %> 

    <p><span class="titulo"><%:area %></span></p> 

    <% foreach (var carrera in area) { %> 
     <p><%:Html.ActionLink(carrera.Nombre, MVC.Carrera.Details(carrera.ID)) %></p> 
    <% } %> 

<% } %>