So weisen Sie einer IEnumerable-Eigenschaft <SelectListItem> in einem Modell einen SelectList-Wert zu

Question

Ich versuche, die Art und Weise zu verbessern, in der ich SelectLists in meinem MVC-Modell und meiner View erstellt habe.

Jemand hat in einer früheren Version meines Codes kommentiert, dass ich zu viele Verwendungen von new hatte. Im Modell, das unten gezeigt wird, sehe ich die Verwendung von new zweimal in der Definition der SelectList. Wie auch immer, ich weiß nicht, wie ich es sonst tun soll, und es ist der einzige Weg, wie ich es in Beispielen definiert habe. Gibt es einen besseren Weg?

In der Modellklasse, ich den Wert SelectList der IEnumerable<StudentRoster> StudentRosters zuweisen, die dann in der Ansicht zugänglich ist.

So erstelle ich die DropDownList in der Ansicht.

<p>Select a campus: @Html.DropDownListFor(m=>m.SelectedCampus,Model.CampusList) 

So definiere ich die SelectList im Modell.

public class SampleViewModel 
{   
    // This IEnumerable is a SelectList in the Html.BeginForm() block 
    public IEnumerable<SelectListItem> CampusList { get; set; } 

    public string SelectedCampus { get; set; }   

    // This list is the "table" shown in the View that the SelectList filters 
    public IEnumerable<StudentRoster> StudentRosters { get; set; }   
} 

Controller:

public ActionResult FilterableIndex(string SelectedCampus="MRA") 
{ 
    StudentRosterViewModel vm = new StudentRosterViewModel(); 

    vm.StudentRosters = db.StudentRosters 
     .Where(m => m.Campus == SelectedCampus)  
     .ToList(); 

    vm.SelectedCampus = SelectedCampus; 
    vm.CampusList = new SelectList(new List<string> 
        {"CRA","DRA","MRA","PRA" }); 

    return View(vm); 
} 

Ist die richtige Codierung oder ist dies ein Beispiel für Kommissionierung eine schlechte Angewohnheit?

Answer 1

Ihr Ansichtsmodell, geben Sie die Elemente einfach dem Ansichtsmodell, wenn es erstellt wurde, und lassen Sie die SelectListItems daraus generieren. @Rubix_Revenge, das ist die Aufgabe des Controllers.

Wie auch immer, ist dieser carrect Weg

CampusList = new SelectList(new List<string> {"CRA","DRA","MRA","PRA" });