Background: I have 4 dropdown lists on my page that all use one List of SelectListItem
to pull data from. All 4 of these dropdowns will always
The code you currently have is the best way to do it, and it should work just as it is. SelectListItem
does have a Selected
property, but this is most usually set by a SelectList
instance. Otherwise, you'd have to manually set this property somewhere on your own. Since you're using IEnumerable<SelectListItem>
instead of an instantiated SelectList
, all of the items should default to having Selected
as false. It's only when Razor renders the DropDownListFor
control does you enumerable get turned into an actual SelectList
and have its selected value set based on the model. Check your other code and make sure you're not setting the selected status manually anywhere.
FWIW, you don't need to add an empty item manually: the DropDownListFor
control has a means to add an empty item to the list.
@Html.DropDownListFor(x => x.SelectedItem1, Model.PossibleItems, string.Empty)
In your list, need to create different SelectList entities for each of the four drop down lists like so:
@Html.DropDownListFor(x => x.SelectedItem1,
new SelectList(Model.PossibleItems, "dataValue", "textValue", Model.SelectedItem1))
@Html.DropDownListFor(x => x.SelectedItem2, Model.PossibleItems)
new SelectList(Model.PossibleItems, "dataValue", "textValue", Model.SelectedItem2))
@Html.DropDownListFor(x => x.SelectedItem3, Model.PossibleItems)
new SelectList(Model.PossibleItems, "dataValue", "textValue", Model.SelectedItem3))
@Html.DropDownListFor(x => x.SelectedItem4, Model.PossibleItems)
new SelectList(Model.PossibleItems, "dataValue", "textValue", Model.SelectedItem4))
In this example, "dataValue" and "textValue" are the properties of the SelectListItem object that correspond to your value and text elements of the drop down options.
Instead of using a List<SelectListItem>
, use a SelectList
. Then you can reuse the list of items in DropDownListFor
.
Model:
public SelectList PossibleItems { get; set; }
Controller:
var items = new List<SelectListItem>();
model.PossibleItems = new SelectList(items, "Value", "Text");
View:
@Html.DropDownListFor(m => m.Item1, Model.PossibleItems)
@Html.DropDownListFor(m => m.Item2, Model.PossibleItems)
2 years ago but for anyone who finds this like I did looking for answers Feussy's comment above is the way to go. I'll try to explain why and what I found.
Using the OP example above. A couple conditions must be met in order to see the described behavior.
Put a breakpoint at @Html.DropDownListFor(x => x.SelectedItem1, Model.PossibleItems)
If you look in Model.PossibleItems you will see the list as expected. Let the code continue to the next line. Now if you look at Model.PossibleItems you will see that Html.DropDownListFor has modified your selectlist on the Model rather than just using it! The option for x.SelectedItem1 now has a 'selected' attribute not only in the HTML that goes to the client but in your model as well! This seems like bad behavior on Html.DropDownListFor but it definitely does it.
Now when you hit @Html.DropDownListFor(x => x.SelectedItem2, Model.PossibleItems)
Feussy's answer works because instead of having one selectList on the model that gets reused it is creating separate selectLists that are generated from some List/IEnumerable on the model that gets reused instead.
So the moral of the story is don't reuse a selectList with nullable values because Html.DropDownListFor has no problems altering the thing and won't clear out any selected attributes if the value is NULL.
Hope that was clear.