What am i doing wrong with asp.net-mvc dropdownlist?

Question

I use a dropdownlist in one of my create.aspx but it some how doesnt seem to work...

public IEnumerable<SelectListItem> FindAllMeasurements()
    {
        var mesurements = from mt in db.MeasurementTypes
                          select new SelectListItem
                          {
                             Value = mt.Id.ToString(),
                             Text= mt.Name
                          };
        return mesurements;
    }

and my controller,

 public ActionResult Create()
    {
      var mesurementTypes = consRepository.FindAllMeasurements().AsEnumerable();
     ViewData["MeasurementType"] = new SelectList(mesurementTypes,"Id","Name");
     return View();
    }

and my create.aspx has this,

<p>
  <label for="MeasurementTypeId">MeasurementType:</label>
    <%= Html.DropDownList("MeasurementType")%>
     <%= Html.ValidationMessage("MeasurementTypeId", "*") %>
   </p>

When i execute this i got these errors,

DataBinding: 'System.Web.Mvc.SelectListItem' does not contain a 
 property with the name 'Id'.

Darin Dimitrov · Accepted Answer · 2010-05-03 06:32:46Z

7

In your controller you are creating a new SelectList from IEnumerable<SelectListItem> which is not correct because you've already specified the Value and Text properties.

You have two options:

public ActionResult Create()
{
    var mesurementTypes = consRepository.FindAllMeasurements();
    ViewData["MeasurementType"] = mesurementTypes;
    return View();
}

or:

public ActionResult Create()
{
    ViewData["MeasurementType"] = new SelectList(db.MeasurementTypes, "Id", "Name");
    return View();
}

There's also a third and preferred way using strongly typed view:

public ActionResult Create()
{
    var measurementTypes = new SelectList(db.MeasurementTypes, "Id", "Name");
    return View(measurementTypes);
}

and in the view:

<%= Html.DropDownList("MeasurementType", Model, "-- Select Value ---") %>

edited May 3, 2010 at 6:32

answered May 3, 2010 at 6:21

Darin Dimitrov

1.0m276 gold badges3.3k silver badges3k bronze badges

Sign up to request clarification or add additional context in comments.

7 Comments

ACP Over a year ago

@Ya darin that worked... How to add "Select" as 0th index in that list?

ACP Over a year ago

@PieterG How to add "Select" as 0th index in that list?

Darin Dimitrov Over a year ago

@Pandiya, you could use the proper extension method to add an optional label: <%= Html.DropDownList("MeasurementType", "-- Please Select a Value ---")%>

ACP Over a year ago

@Darin how to set value "0" to that "Select"

ACP Over a year ago

@when i inspected my select <option value="">Select</option>... I want to set 0 as value...

|

Community · Accepted Answer · 2017-05-23 10:32:56Z

1

As the error message implies, you need an IEnumerable<SelectList>, not an IEnumerable<Materials>.

The constructor for SelectList has an overload that takes an IEnumerable. See .net MVC, SelectLists, and LINQ

edited May 23, 2017 at 10:32

CommunityBot

11 silver badge

answered May 3, 2010 at 5:08

Robert Harvey

182k48 gold badges349 silver badges516 bronze badges

1 Comment

Robert Harvey Over a year ago

@Pandiya: Well, it's a completely different question now. I see that you found SelectList.

Collectives™ on Stack Overflow

What am i doing wrong with asp.net-mvc dropdownlist?

2 Answers 2

7 Comments

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

7 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related