ASP.NET Core MVC用ViewBag留存表单历史输入是否为不良实践？

你遇到的表单值重置问题，本质是两个原因：一是返回视图时没有结构化传递用户提交的查询条件，二是手动逐字段给input赋值的做法效率极低。ASP.NET Core MVC 本身内置了表单值自动回填机制，完全不需要靠手动写ViewBag赋值实现需求。

具体实现步骤 #

• 首先创建页面专用ViewModel，将查询条件、查询结果两类数据封装在一起，避免零散传值：

public class CustomerListPageViewModel
{
    // 存储用户提交的所有筛选条件
    public QueryModel Query { get; set; } = new QueryModel();
    // 存储筛选后返回的客户数据列表
    public IEnumerable<Customer> Customers { get; set; } = Enumerable.Empty<Customer>();
}

• 修改控制器Action，组装统一的ViewModel后传给视图，不需要写任何单字段赋值代码：

[HttpPost]
public async Task<IActionResult> Query(QueryModel query)
{
    var filteredCustomers = await _context.Customers
        .AsQueryable()
        .FilterCustomerList(query);
    
    var pageModel = new CustomerListPageViewModel
    {
        Query = query,
        Customers = filteredCustomers
    };
    return View("Index", pageModel);
}

• 修改Index视图，顶部声明强类型模型，表单输入框使用asp-for Tag Helper做绑定，不需要手动写value属性：

@model CustomerListPageViewModel

<!-- 表单区域 -->
<div class="col-md-4">
    <!-- asp-for会自动生成name、id、value属性，值自动从绑定的模型中读取 -->
    <input asp-for="Query.Name" type="text" placeholder="First name" class="form-control">
</div>

<!-- 结果列表区域，遍历Model.Customers渲染即可 -->

这个方案完全规避了ViewBag的缺陷：所有绑定都是强类型的，属性名写错在编译期就会报错；不管表单有20个还是30个字段，只需要把整个QueryModel传入ViewModel即可，不需要逐行写赋值代码；甚至模型验证失败时用户输入的非法格式内容也会被自动保留，不会出现值丢失的问题。

筛选、查询类操作属于幂等操作，本身就不应该用POST方法提交。你可以直接把表单的提交方法改成GET，会获得几个额外的收益：

• 所有筛选参数会拼接在URL上，用户可以直接收藏、分享筛选后的结果页，刷新、后退操作也不会触发浏览器的「表单重复提交」提示
• 不需要额外做值传递：GET请求提交的参数会自动进入ModelState，只要你用asp-for绑定表单字段，哪怕不把QueryModel显式传给视图，Tag Helper也会自动从ModelState里读取用户之前输入的值完成回填，代码更简洁。

修改方式非常简单：

1. 把表单标签的提交方法改为get：

<form asp-action="Query" method="get">
    <!-- 表单内容保持不变 -->
</form>

2. 把Action的请求特性从[HttpPost]改为[HttpGet]即可，原有筛选逻辑不需要做任何改动。

除了你提到的动态类型不安全、重复代码多的问题，ViewBag还有两个明显的缺陷：

• 无法回填模型验证失败时的用户输入：如果用户输入了不符合QueryModel类型定义的内容（比如本该填数字的字段填了文本），模型绑定阶段就会失败，错误值根本不会传入Action，用ViewBag自然也拿不到这个值，用户输入的内容会直接丢失
• 重定向场景下值会直接失效：如果后续要做PRG（Post-Redirect-Get）模式避免重复提交，ViewBag里的值在重定向后会被清空，需要额外用TempData转存，维护成本更高。

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