ASP.NET MVC大型嵌套复选框列表Form POST提交过慢求助

看起来你遇到的是默认ModelBinder处理大量嵌套集合时的性能瓶颈——64个Group加853个User（还有重复的User实例），默认绑定逻辑要做大量反射、集合实例化和ModelState验证，30秒的耗时完全符合这个场景。下面是几个经过验证的可行方案，按推荐优先级排序：

1. 最有效：精简提交数据，改用轻量DTO #

你当前的ViewModel包含了很多不需要回传的字段（比如Name、DistinguishedName、SamAccountName这些），这些字段在服务器端完全可以通过Guid重新获取，没必要让前端提交。只保留提交必须的字段，能大幅减少表单数据量和ModelBinder的处理工作：

步骤： #

• 创建一个只包含提交必要字段的DTO：

public class ByActivityEditSubmitDto
{
    public int ActivityId { get; set; }
    public string Path { get; set; }
    public List<GroupSubmitDto> Groups { get; set; } = new List<GroupSubmitDto>();
}

public class GroupSubmitDto
{
    public Guid? Guid { get; set; }
    public bool IsAllowed { get; set; }
    public List<UserSubmitDto> Users { get; set; } = new List<UserSubmitDto>();
}

public class UserSubmitDto
{
    public Guid? Guid { get; set; }
    public bool IsAllowed { get; set; }
}

• 修改控制器Action接收这个DTO：

[HttpPost]
public ActionResult Edit(ByActivityEditSubmitDto submitDto)
{
    // 从数据库/缓存获取完整的Group和User数据，再结合submitDto的IsAllowed状态更新
    var fullViewModel = _activityService.GetEditViewModel(submitDto.ActivityId);
    foreach (var groupDto in submitDto.Groups)
    {
        var matchingGroup = fullViewModel.Groups.First(g => g.Guid == groupDto.Guid);
        matchingGroup.IsAllowed = groupDto.IsAllowed;
        foreach (var userDto in groupDto.Users)
        {
            var matchingUser = matchingGroup.Users.First(u => u.Guid == userDto.Guid);
            matchingUser.IsAllowed = userDto.IsAllowed;
        }
    }

    // 手动验证必要字段（比如ActivityId是否有效）
    if (submitDto.ActivityId <= 0)
    {
        ModelState.AddModelError("ActivityId", "无效的活动ID");
    }

    if (ModelState.IsValid)
    {
        // 业务逻辑
    }
}

• 同步修改视图的编辑器模板，只保留需要提交的Hidden字段：比如去掉Name、DistinguishedName等不需要的HiddenFor，只保留Guid和IsAllowed相关的控件。

2. 次优：自定义ModelBinder跳过冗余逻辑 #

如果不想修改ViewModel结构，可以写一个针对ByActivityEditViewModel的自定义ModelBinder，直接从Request.Form中解析数据，跳过默认ModelBinder的通用反射和验证逻辑：

public class ByActivityEditViewModelBinder : IModelBinder
{
    public object BindModel(ControllerContext controllerContext, ModelBindingContext bindingContext)
    {
        var request = controllerContext.HttpContext.Request;
        var viewModel = new ByActivityEditViewModel
        {
            ActivityId = int.Parse(request.Form["ActivityId"]),
            Path = request.Form["Path"],
            Groups = new List<Group>()
        };

        // 解析Groups集合：Request.Form的键会是类似Groups[0].Guid、Groups[0].IsAllowed这样的格式
        int groupIndex = 0;
        while (!string.IsNullOrEmpty(request.Form[$"Groups[{groupIndex}].Guid"]))
        {
            var group = new Group
            {
                Guid = Guid.Parse(request.Form[$"Groups[{groupIndex}].Guid"]),
                IsAllowed = bool.Parse(request.Form[$"Groups[{groupIndex}].IsAllowed"])
            };

            // 解析当前Group下的Users
            int userIndex = 0;
            group.Users = new List<User>();
            while (!string.IsNullOrEmpty(request.Form[$"Groups[{groupIndex}].Users[{userIndex}].Guid"]))
            {
                group.Users.Add(new User
                {
                    Guid = Guid.Parse(request.Form[$"Groups[{groupIndex}].Users[{userIndex}].Guid"]),
                    IsAllowed = bool.Parse(request.Form[$"Groups[{groupIndex}].Users[{userIndex}].IsAllowed"])
                });
                userIndex++;
            }

            viewModel.Groups.Add(group);
            groupIndex++;
        }

        // 跳过自动ModelState验证，减少耗时
        bindingContext.ModelState.Clear();
        return viewModel;
    }
}

然后在ViewModel上注册这个Binder：

[ModelBinder(typeof(ByActivityEditViewModelBinder))]
public class ByActivityEditViewModel { ... }

或者在Action上指定：

[HttpPost]
public ActionResult Edit([ModelBinder(typeof(ByActivityEditViewModelBinder))] ByActivityEditViewModel viewModel)
{ ... }

3. 前端优化：只提交选中项而非全部数据 #

如果前端可以修改，那最好的方式是只提交选中的Group和User的Guid，而不是所有的项。这样表单数据量会骤减，ModelBinder处理起来毫无压力：

前端实现示例（jQuery）： #

$('form').submit(function(e) {
    e.preventDefault();
    
    // 收集选中的Group和User
    var selectedGroupGuids = [];
    var selectedUserGuids = [];
    
    // 遍历Group复选框，收集选中的Guid
    $('input[type="checkbox"][name*="Groups"][name*=".IsAllowed"]:checked').each(function() {
        // 找到对应的Group Guid隐藏字段
        var groupGuidInput = $(this).closest('div').find('input[name*=".Guid"]').first();
        if (groupGuidInput.length > 0) {
            selectedGroupGuids.push(groupGuidInput.val());
        }
    });
    
    // 遍历User复选框，收集选中的Guid
    $('input[type="checkbox"][name*="Users"][name*=".IsAllowed"]:checked').each(function() {
        var userGuidInput = $(this).closest('li').find('input[name*=".Guid"]').first();
        if (userGuidInput.length > 0) {
            selectedUserGuids.push(userGuidInput.val());
        }
    });
    
    // 构建提交数据
    var submitData = {
        ActivityId: $('#ActivityId').val(),
        Path: $('#Path').val(),
        SelectedGroupGuids: selectedGroupGuids,
        SelectedUserGuids: selectedUserGuids
    };
    
    // AJAX提交
    $.post($(this).attr('action'), submitData, function(response) {
        // 处理响应，比如跳转或提示
        window.location.href = response.redirectUrl;
    });
});

然后控制器接收这个简化的DTO：

public class ActivitySelectionDto
{
    public int ActivityId { get; set; }
    public string Path { get; set; }
    public List<Guid> SelectedGroupGuids { get; set; } = new List<Guid>();
    public List<Guid> SelectedUserGuids { get; set; } = new List<Guid>();
}

[HttpPost]
public ActionResult Edit(ActivitySelectionDto selectionDto)
{
    // 获取完整的ViewModel，然后根据选中的Guid设置IsAllowed
    var viewModel = _activityService.GetEditViewModel(selectionDto.ActivityId);
    
    foreach (var group in viewModel.Groups)
    {
        group.IsAllowed = selectionDto.SelectedGroupGuids.Contains(group.Guid.Value);
        foreach (var user in group.Users)
        {
            user.IsAllowed = selectionDto.SelectedUserGuids.Contains(user.Guid.Value);
        }
    }
    
    // 业务逻辑
}

这个方案的性能提升最明显，因为提交的数据量从原来的上千条变成了几十条（只选中的项）。

4. 临时缓解：禁用不必要的ModelState验证 #

如果以上方案暂时无法实施，可以尝试跳过ModelState的自动验证，因为默认ModelBinder会为每个字段创建ModelState条目，这在数据量大时非常耗时：

[HttpPost]
public ActionResult Edit(ByActivityEditViewModel viewModel)
{
    // 清除自动生成的ModelState，跳过验证
    ModelState.Clear();
    
    // 手动验证必要字段（比如ActivityId是否有效）
    if (viewModel.ActivityId <= 0)
    {
        ModelState.AddModelError("ActivityId", "无效的活动ID");
    }
    
    if (ModelState.IsValid)
    {
        // 业务逻辑
    }
}

这个方法只能临时缓解，不能从根本上解决ModelBinder的耗时问题，但可以减少一部分等待时间。

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