Besoin de conseils avant de développer une mauvaise habitude

Question

J'ai un contrôleur appelé AuctionsController. J'y ai actions appelé Index() et AuctionCategoryListing():

//Used for displaying all auctions. 
public ActionResult Index() 
{ 
    AuctionRepository auctionRepo = new AuctionRepository(); 
    var auctions = auctionRepo.FindAllAuctions(); 
    return View(auctions); 
} 

//Used for displaying auctions for a single category. 
public ActionResult AuctionCategoryListing(string categoryName) 
{ 
    AuctionRepository auctionRepo = new AuctionRepository(); 
    var auctions = auctionRepo.FindAllAuctions() 
         .Where(c => c.Subcategory.Category.Name == categoryName); 
    return View("Index", auctions); 
} 

Comme vous pouvez le dire, ils ont tous deux invoquent le même point de vue (cette action est appelée « d'invoquer une vue » Qu'est-ce que c'est le nom propre. ?).

@model IEnumerable<Cumavi.Models.Auction> 

@{ 
    ViewBag.Title = "Index"; 
} 

<h2>Index</h2> 

<p> 
    @Html.ActionLink("Create New", "Create") 
</p> 
<table> 
    <tr> 
     <th></th> 
     <th> 
      IDSubcategory 
     </th> 
     <th> 
      IDCity 
     </th> 
     <th> 
      IDPerson 
     </th> 
     <th> 
      Title 
     </th> 
     <th> 
      TextBody 
     </th> 
     <th> 
      ContactNumber 
     </th> 
     <th> 
      AskingPrice 
     </th> 
     <th> 
      AddressDirection 
     </th> 
     <th> 
      LatestUpdateDate 
     </th> 
     <th> 
      VisitCount 
     </th> 
    </tr> 

@foreach (var item in Model) { 
    <tr> 
     <td> 
      @Html.ActionLink("Edit", "Edit", new { id=item.ID }) | 
      @Html.ActionLink("Details", "Details", new { id=item.ID }) | 
      @Html.ActionLink("Delete", "Delete", new { id=item.ID }) 
     </td> 
     <td> 
      @item.IDSubcategory 
     </td> 
     <td> 
      @item.IDCity 
     </td> 
     <td> 
      @item.IDPerson 
     </td> 
     <td> 
      @item.Title 
     </td> 
     <td> 
      @item.TextBody 
     </td> 
     <td> 
      @item.ContactNumber 
     </td> 
     <td> 
      @String.Format("{0:F}", item.AskingPrice) 
     </td> 
     <td> 
      @item.AddressDirection 
     </td> 
     <td> 
      @String.Format("{0:g}", item.LatestUpdateDate) 
     </td> 
     <td> 
      @item.VisitCount 
     </td> 
    </tr> 
} 

</table> 

Ils héritent tous les deux du même modèle. Ma question est la suivante: est-ce que je fais les choses de la bonne manière? Ou est-ce juste un hack j'ai réussi à gratter ensemble. Aide-moi avant que j'apprenne une mauvaise habitude.

Answer 1

Je modifier ce à:

public ActionResult Index(string categoryName) 
{ 

    AuctionRepository auctionRepo = new AuctionRepository(); 
    var auctions=auctionRepo.FindAllAuctions(); 

    if (!string.IsNullOrEmpty(categoryName)) 
    { 
     auctions = auctions.Where(c => c.Subcategory.Category.Name == categoryName); 
    } 

    return View(auctions); 
} 

Votre itinéraire pourrait alors ressembler à:

context.MapRoute(
     "auction_defalt", 
     "Auction/{categoryName}", 
     new { controller="Auction", action = "Index", categoryName = UrlParameter.Optional } 

Puisque les actions sont si semblables, je ne vois pas de raison de les séparer.

Answer 2

Vous devez faire des branchements quelque part, donc c'est probablement plus une question de préférence.

La façon dont je le manipulerais est d'avoir une seule méthode, et de prendre le nom de la catégorie comme paramètre. Puisque les chaînes sont nullables, si elles ne sont pas spécifiées, elles seront nulles. Ma méthode d'action serait probablement quelque chose comme:

public ActionResult Index(string categoryName) 
{ 
    AuctionRepository auctionRepo = new AuctionRepository(); 
    var auctions = auctionRepo.FindAllAuctions(); 

    if(String.IsNullOrEmpty(categoryName) == false) 
     auctions = auctions.Where(c => c.Subcategory.Category.Name == categoryName); 

    return View(auctions); 
} 

Answer 3

C'est une caractéristique de MVC que la vue et le contrôleur sont indépendants. Vous pouvez également utiliser des vues partagées ou partielles pour la même chose, si je peux dire que c'est une bonne chose parce que vous écrivez du code réutilisable et que vous l'utilisez.

Answer 4

Je préférerais le repo à faire des choses telles que le filtrage et l'intérêt pour la pagination de la performance et le concept SEC

public ActionResult Index(string categoryName) 
    { 
     AuctionRepository auctionRepo = new AuctionRepository(); 

    //Let the Repo Handle things like filtering and pagination, avoiding performance issues 

     var auctions = auctionRepo.FindAllAuctions(categoryName); 

     return View(auctions); 
    } 

DAL devrait être responsable de ces tâches.

Answer 5

Comme tout framework, ASP.NET MVC vous offre de nombreuses possibilités de prise de vue dans le pied. Sans réflexion préalable, la réutilisation des actions du contrôleur, des modèles de vue et des vues peut rapidement devenir un cauchemar de maintenance. Sans compter que sans pareille considération vos routes deviendront difficiles à lier ensemble. En suivant les principes de la convention sur la configuration, vous pouvez résoudre votre problème en utilisant des actions distinctes mais en réutilisant une vue partielle. Pour moi, l'action d'index du AuctionsController devrait être responsable de la liste de toutes les enchères dans le système. Je n'appellerais pas l'action de ma catégorie AuctionCategoryListing, mais je l'appellerais simplement Category. Grâce à la convention cela a pour effet agréable de jeter les routes comme:

• site.com/auctions/ pour l'indice
• site.com/auctions/category/CATEGORYNAME pour la catégorie.

La route est facilement compréhensible par l'utilisateur et vous permet de comprendre clairement ce que chacun fait.(À ce stade Omar fournit a good suggestion in his answer pour laisser votre référentiel gérer la pagination, le filtrage, etc.)

En ce qui concerne ce que chaque action devrait retourner, vous avez plusieurs options. Ma préférence serait de renvoyer des vues séparées contenant chacune une référence à un partiel commun. Cela vous donne la possibilité de créer différentes vues entourant le partiel mais permet de réutiliser la pièce commune.

Pour en savoir plus qui pourraient être utiles:

• One ViewModel per View
• ASP.NET MVC terminology is tripping me up - why 'ViewModel'?
• when should i use partial views in asp.net mvc?