MVC C#中提交按钮传递下拉框选中值至GetWACC参数问题

你的核心问题很明确：你错误地将整个下拉框的选项列表（List<SelectListItem>）直接转成字符串传给了GetWACC方法，而不是获取用户实际选中的选项值。这就是为什么参数返回System.Collection.Generic.List的原因——这是List类型的默认ToString()结果。

下面是分步修正方案：

1. 修正View中的下拉框绑定 #

你的Model已经定义了IndustrySelected和LocationSelected两个字段用来存储选中值，但View里的下拉框绑定错了目标字段（绑定到了不存在的LocID）。修改View代码如下：

@model WACC

@using (Html.BeginForm("WACC", "Home", FormMethod.Post))
{
    @Html.ValidationSummary(true, "", new { @class = "text-danger" })

    <table class='wacctable'>
        <tr>
            <td style="background-color:lightgrey">@Html.DisplayNameFor(x => x.IndustryList)</td>
            <td style="width: 200px">
                @Html.DropDownListFor(x => x.IndustrySelected, Model.IndustryList, "--Select One--", new { @class = "form-control" })
                @Html.ValidationMessageFor(x => x.IndustrySelected, "", new { @class = "text-danger" })
            </td>
        </tr>
        <tr>
            <td style="background-color:lightgrey">@Html.DisplayNameFor(x => x.LocationList)</td>
            <td>
                @Html.DropDownListFor(x => x.LocationSelected, Model.LocationList, "--Select One--", new { @class = "form-control" })
                @Html.ValidationMessageFor(x => x.LocationSelected, "", new { @class = "text-danger" })
            </td>
        </tr>
        <tr>
            <td style="background-color:lightgrey">@Html.DisplayNameFor(x => x.ValuationDate)</td>
            <td>
                @Html.TextBoxFor(x => x.ValuationDate, "{0:yyyy-MM-dd}", new { @class = "date-picker", type = "date" })
            </td>
        </tr>
        <tr>
            <td style="background-color:lightgrey">@Html.DisplayNameFor(m => m.FairValue)</td>
            <td>
                @Html.TextBoxFor(m => m.FairValue, new { @readonly = "readonly", @class = "form-control" })
            </td>
        </tr>
    </table>
    <br />
    <input type="submit" value='Submit' class="btn btn-primary" />
}

关键修改点：

• 用@Html.BeginForm包裹整个表单，确保提交时参数能正确传递到Controller
• 将下拉框绑定到IndustrySelected和LocationSelected字段
• 添加验证提示，方便用户知道未选中选项的情况

2. 修正HomeController的逻辑 #

原来的代码只处理了GET请求，需要新增POST方法来接收表单提交的选中值，并且要传递选中值而非整个列表：

public class HomeController : Controller
{
    // GET: 初始化页面，加载下拉选项
    public ActionResult WACC()
    {
        ViewBag.Message = "Your WACC page.";
        var objWACC = new WACC();
        objWACC.ValuationDate = new DateTime(2018, 11, 20);
        objWACC.IndustryList = WACC_Lib.GetIndustry();
        objWACC.LocationList = WACC_Lib.GetOpLocations();
        return View(objWACC);
    }

    // POST: 处理表单提交，计算WACC
    [HttpPost]
    public ActionResult WACC(WACC objWACC)
    {
        // 重新加载下拉列表（POST请求后列表不会自动保留）
        objWACC.IndustryList = WACC_Lib.GetIndustry();
        objWACC.LocationList = WACC_Lib.GetOpLocations();

        // 验证是否选中了必要参数
        if (ModelState.IsValid)
        {
            // 传入选中的地点、行业和日期，而非整个列表
            objWACC.FairValue = WACC_Lib.GetWACC(objWACC.LocationSelected, objWACC.IndustrySelected, objWACC.ValuationDate);
        }

        return View(objWACC);
    }
}

关键修改点：

• 新增[HttpPost]标记的Action方法，接收表单提交的Model
• 使用objWACC.LocationSelected和objWACC.IndustrySelected作为参数传递给GetWACC
• 重新加载下拉列表，避免POST后下拉框为空

3. 优化Model的验证（可选但推荐） #

在Model的选中值字段上添加[Required]特性，实现自动验证：

public class WACC
{
    [Key]
    [Display(Name = "Industry")]
    public List<SelectListItem> IndustryList { get; set; }

    [Required(ErrorMessage = "请选择行业")]
    public string IndustrySelected { get; set; }

    [Display(Name = "Location")]
    public List<SelectListItem> LocationList { get; set; }

    [Required(ErrorMessage = "请选择地点")]
    public string LocationSelected { get; set; }

    [Display(Name = "Valuation Date")]
    public DateTime ValuationDate { get; set; }

    [Display(Name = "WACC")]
    public double FairValue { get; set; }
}

核心问题总结 #

你之前的错误是把选项列表对象当成了参数传递，而实际上应该传递用户选中的选项值。通过修正View的绑定目标和Controller的参数获取逻辑，就能解决这个问题。

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