ASP.NET MVC 3 List <T> do IEnumerable <SelectListItem>

Question

Obecnie nie jestem zadowolony ze sposobu, w jaki moje obiekty DropDownListFor() są wypełnione. Próbuję znaleźć jako ogólny sposób zapełniania IEnumerable, jak to możliwe. To jest to, co do tej pory miałem.

Helper:

public static List<SelectListItem> ToSelectList(IDictionary<string,string> dictionaryItems, string selectedValue, string noSelection, bool search = false) 
    { 
     List<SelectListItem> items = new List<SelectListItem>(); 

     if (search) 
     { 
      items.Add(new SelectListItem { Selected = true, Value = "-1", Text = string.Format("-- {0} --", noSelection) }); 
     } 

     foreach (var item in dictionaryItems) 
     { 
      items.Add(new SelectListItem 
      { 
       Text = item.Key, 
       Value = item.Value, 
       Selected = selectedValue == item.Value ? true : false 
      }); 
     } 

     return items 
      .OrderBy(l => l.Text) 
      .ToList(); 
    } 

Kontroler:

[HttpGet] 
    public ActionResult Index() 
    { 
     var model = new CreateModel(); 

     var parentOrganisations = _orgs.FindBy(o => o.OwningOrganisationID == Globals.OrganisationID || o.ID == Globals.OrganisationID) 
      .OrderBy(o => o.OrganisationName); 

     Dictionary<string, string> items = new Dictionary<string, string>(); 

     foreach (var item in parentOrganisations) 
     { 
      items.Add(item.OrganisationName, item.ID.ToString()); 
     } 

     model.Organisations = SelectLists.ToSelectList(items, "-1", "-- None -- ", true); 

     return View(model); 
    } 

Widok:

<div class="control-group"> 
     <label class="control-label">Parent Organisation</label> 
     <div class="controls"> 
      @Html.DropDownListFor(m => m.ParentOrganisationID, Model.Organisations, new { @class = "input-xlarge"}) 
      <p class="help-block">Select a parent organisation to create a branch</p> 
     </div> 
    </div> 

Nie wydaje się być dużo powtarzalnego kodu w kontrolerze. Zajmuje ogólną listę, dodaje wartość i tekst do słownika, a następnie używa go jako danych wejściowych dla pomocnika, który tworzy listę wyboru do wysłania jako część modelu.

Czy ktoś ma lepsze sposoby na osiągnięcie tego? Nienawidzę mieć nadęcia w moim kontrolerze, a kiedy dostaję kilka zrzutów na formularzu, to dokładnie to się stanie w tym przypadku.

Dzięki

EDIT - Dzięki metodzie pomocnika Kennetha, mam teraz konsolidowane całość w jednej rozmowy w kontrolerze:

  model.Organisations = _orgs.FindBy(o => o.OwningOrganisationID == Globals.OrganisationID || o.ID == Globals.OrganisationID) 
      .OrderBy(o => o.OrganisationName) 
      .ToList() 
      .ToSelectList(org => org.OrganisationName, org => org.ID.ToString(), "-1", "None", true); 

Answer 1

Można dostarczyć wywołania zwrotne, które pozwolą im klucz i wartości, a następnie użyj ich. Oprócz tego można utworzyć go jako metodę rozszerzenia:

metoda Extension:

public static List<SelectListItem> ToSelectList<T>(this List<T> Items, Func<T, string> getKey, Func<T, string> getValue, string selectedValue, string noSelection, bool search = false) 
    { 
     List<SelectListItem> items = new List<SelectListItem>(); 

     if (search) 
     { 
      items.Add(new SelectListItem { Selected = true, Value = "-1", Text = string.Format("-- {0} --", noSelection) }); 
     } 

     foreach (var item in Items) 
     { 
      items.Add(new SelectListItem 
      { 
       Text = getKey(item), 
       Value = getValue(item), 
       Selected = selectedValue == getValue(item) ? true : false 
      }); 
     } 

     return items 
      .OrderBy(l => l.Text) 
      .ToList(); 
    } 

Zastosowanie:

List<Org>() parentOrganisations = // fetch here 
model.Organisations = parentOrganisations.ToSelectList(org => org.ID.ToString(), 
                 org => org.OrganisationName, 
                 "-1", 
                 "-- None -- ", 
                 true); 

Uwaga: Wpisałem to w tak edytorze więc możesz mieć błędy składniowe (powinny być jednak łatwe do rozwiązania).

Answer 2

Można to zrobić w kontrolerze tylko tak: -

List<SelectListItem> dropdownItems = parentOrganisations 
    .Select(item => new SelectListItem 
    { 
     Value = item.ID.ToString(), 
     Text = item.OrganisationName, 
     Selected = "-1" == item.ID.ToString() ? true : false 
    }) 
    .ToList();