MVC ViewBag绑定下拉框行数正确但仅显示第一项问题求助

Hey there, let's tackle this dropdown issue you're facing. It's super frustrating when the number of options is right but every single one shows the first item—let's break down what's likely going wrong and fix it.

First, Check Your View Code (Most Common Culprit) #

The biggest reason for this behavior is usually a mistake in how you're rendering the dropdown in your Razor view. If you're manually looping through the ViewBag.ContractID collection but accidentally referencing the first element every time (instead of the current item in the loop), you'll get exactly this problem.

Fix 1: Use the Built-in Helper Method

The easiest and safest way to render the dropdown is with Html.DropDownList:

@Html.DropDownList("ContractID", (IEnumerable<SelectListItem>)ViewBag.ContractID, "Select a Contract", new { @class = "form-control" })

This helper will automatically iterate through your SelectListItem collection and render each option correctly.

Fix 2: Correct Manual Loop

If you prefer to write the HTML manually, make sure you use the loop variable (not the first element of the collection) for each option:

<select name="ContractID" class="form-control">
    <option value="">Select a Contract</option>
    @foreach (var contractItem in (IEnumerable<SelectListItem>)ViewBag.ContractID)
    {
        <option value="@contractItem.Value">@contractItem.Text</option>
    }
</select>

Notice we're using contractItem (the current item in the loop) instead of something like ViewBag.ContractID[0] which would repeat the first item every time.

Second, Optimize Your Controller Code (Avoid Unintended Duplicates) #

Looking at your controller code, the Distinct() call is applied to the entire v_xxx_yyyy_aaaa_contracts entity. If there are multiple rows with the same ContractID but different values in other fields, Distinct() won't filter them out (since it compares the whole entity). To properly deduplicate by ContractID, adjust your query:

// First get unique ContractIDs
var uniqueContractIDs = db.v_xxx_yyyy_aaaa_contracts
    .Where(m => m.GroupID.ToString() == @group)
    .Select(m => m.ContractID) // Only select the ContractID field
    .Distinct() // Now this deduplicates based on ContractID alone
    .OrderBy(id => id)
    .ToList();

// Convert to SelectListItem collection
List<SelectListItem> Contracts = uniqueContractIDs
    .Select(id => new SelectListItem 
    { 
        Value = id, 
        Text = id 
    })
    .ToList();

ViewBag.ContractID = Contracts;

This ensures you only get unique ContractID values, which makes your dropdown cleaner and avoids any hidden issues from duplicate entities.

Give these fixes a try—most likely the view loop was the main issue, but optimizing the controller query will help prevent similar problems down the line.

内容的提问来源于stack exchange，提问作者Marcos Ferreira