ASP.NET Core MVC中HttpGet后模型值被URL参数覆盖问题排查

问题现象 #

调用/ContactController/GetContactEdit?contactId=-1后，即使在控制器中手动将viewModel.ContactId设为99，视图渲染出的隐藏域值仍为-1。调试时可见@Model的ContactId确实是99，但Html.HiddenFor输出的却是URL参数里的-1。该问题在传统ASP.NET MVC中不存在。

相关代码示例 #

视图模型 #

public class ContactViewModel
{
    public int ContactId { get; set; }

    [Display(Name = "First")]
    public string FirstName { get; set; }
}

HttpGet接口 #

[HttpGet]
public PartialViewResult GetContactEdit(int? contactId)
{
    ContactViewModel viewModel = new ContactViewModel();

    if (contactId > 0)
    {
        viewModel = Contact.Get<ContactViewModel>(contactId.Value).Content;
    }

    // 手动修改值，但渲染时不生效
    viewModel.ContactId = 99;

    return PartialView("~/Views/AddressBook/PartialViews/_ContactEdit.cshtml", viewModel);
}

部分视图代码 #

@using (Html.BeginForm(null, null, FormMethod.Post, new { id = "contactEditForm" }))
{
    @Html.AntiForgeryToken()
    @Html.HiddenFor(m=>m.ContactId)
    <div>
        <div>
            @Html.LabelFor(m=>m.FirstName)
            @Html.TextBoxFor(m=>m.FirstName)
        </div>
    </div>
}

原因分析 #

ASP.NET Core MVC的HTML辅助方法（如HiddenFor、TextBoxFor）优先从ModelState中取值，而非直接使用Model的当前值。当GET请求的contactId参数绑定到Action参数后，该值会自动存入ModelState。即便后续修改了Model的属性值，辅助方法仍会读取ModelState里的原始参数值——这是Core版本新增的行为，用于保留用户输入的原始值以便验证回显。

而传统ASP.NET MVC中，GET请求的参数不会自动保留到ModelState中，所以辅助方法直接使用Model的当前值。

解决方法 #

方法1：清除ModelState中对应键的值 #

在修改Model属性后，移除ModelState中对应的键，让辅助方法读取Model的当前值：

[HttpGet]
public PartialViewResult GetContactEdit(int? contactId)
{
    ContactViewModel viewModel = new ContactViewModel();

    if (contactId > 0)
    {
        viewModel = Contact.Get<ContactViewModel>(contactId.Value).Content;
    }

    viewModel.ContactId = 99;
    // 移除对应键，避免参数值覆盖
    ModelState.Remove("ContactId");

    return PartialView("~/Views/AddressBook/PartialViews/_ContactEdit.cshtml", viewModel);
}

方法2：手动指定控件值（不依赖ModelState） #

改用Html.Hidden手动传入Model的当前值，绕过ModelState的取值逻辑：

@using (Html.BeginForm(null, null, FormMethod.Post, new { id = "contactEditForm" }))
{
    @Html.AntiForgeryToken()
    @Html.Hidden("ContactId", Model.ContactId)
    <div>
        <div>
            @Html.LabelFor(m=>m.FirstName)
            @Html.TextBoxFor(m=>m.FirstName)
        </div>
    </div>
}

方法3：清空整个ModelState #

如果不需要保留任何ModelState数据（比如无需验证），可以直接清空：

[HttpGet]
public PartialViewResult GetContactEdit(int? contactId)
{
    ContactViewModel viewModel = new ContactViewModel();

    if (contactId > 0)
    {
        viewModel = Contact.Get<ContactViewModel>(contactId.Value).Content;
    }

    viewModel.ContactId = 99;
    // 清空所有ModelState数据
    ModelState.Clear();

    return PartialView("~/Views/AddressBook/PartialViews/_ContactEdit.cshtml", viewModel);
}

方法4：避免参数名与Model属性名重复 #

将Action的参数名改为与Model属性名不同的名称，比如id，这样ModelState中就不会存入对应键值：

[HttpGet]
public PartialViewResult GetContactEdit(int? id)
{
    ContactViewModel viewModel = new ContactViewModel();

    if (id > 0)
    {
        viewModel = Contact.Get<ContactViewModel>(id.Value).Content;
    }

    viewModel.ContactId = 99;

    return PartialView("~/Views/AddressBook/PartialViews/_ContactEdit.cshtml", viewModel);
}

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