注册带角色选择的用户时出现奇怪的NullReferenceException

嘿，我看你在注册页面的角色下拉列表上栽跟头了——不绑定数据就空着，还触发了未处理异常，对吧？结合你贴出来的代码片段，我来给你捋捋怎么搞定这个问题：

问题根源 #

你当前的Register POST方法里没有给视图传递角色列表的数据源，导致下拉列表没有选项；如果尝试提交未绑定的角色选择，还可能因为找不到有效角色数据触发异常。另外，GET请求的Register方法也没提前加载角色数据，页面一开始就显示空下拉框。

分步解决方案 #

1. 先在GET请求的Register方法中加载角色数据 #

首先要在用户打开注册页面时，就把系统里的角色列表传递到视图，这样下拉框一开始就有选项：

public async Task<IActionResult> Register(string returnUrl = null)
{
    ViewData["ReturnUrl"] = returnUrl;
    // 从角色管理器获取所有角色
    var allRoles = await _roleManager.Roles.ToListAsync();
    // 把角色转成SelectList，传递到视图（用ViewData或者ViewModel都行）
    ViewData["RoleOptions"] = new SelectList(allRoles, "Id", "Name");
    // 如果习惯用ViewModel，就给RegisterViewModel加个Roles属性，赋值后传模型
    // var model = new RegisterViewModel { Roles = allRoles };
    // return View(model);
    return View();
}

2. 更新视图，绑定下拉列表 #

在Register.cshtml里，用ASP.NET Core的标签助手渲染下拉框，绑定我们传递的角色数据：

<div class="form-group">
    <label asp-for="SelectedRoleId" class="form-label">用户角色</label>
    <!-- 如果用ViewData传递的RoleOptions -->
    <select asp-for="SelectedRoleId" asp-items="@ViewBag.RoleOptions" class="form-control">
        <option value="">请选择一个角色</option>
    </select>
    <!-- 如果用ViewModel的Roles属性 -->
    <!-- <select asp-for="SelectedRoleId" asp-items="@(new SelectList(Model.Roles, "Id", "Name"))" class="form-control">
        <option value="">请选择一个角色</option>
    </select> -->
    <span asp-validation-for="SelectedRoleId" class="text-danger"></span>
</div>

注意：要给RegisterViewModel加一个SelectedRoleId属性（或者SelectedRoleName，看你用角色的ID还是名称），用来接收用户选中的角色值。

3. 在POST的Register方法中处理角色分配（并防异常） #

用户提交注册表单后，要给新创建的用户分配选中的角色，同时要做异常防护，避免因为角色不存在或者加载失败触发未处理异常：

public async Task<IActionResult> Register(RegisterViewModel model, string returnUrl = null)
{
    ViewData["ReturnUrl"] = returnUrl;
    // 重要：如果ModelState无效，返回视图时要重新传递角色数据，不然下拉框又空了
    var allRoles = await _roleManager.Roles.ToListAsync();
    ViewData["RoleOptions"] = new SelectList(allRoles, "Id", "Name");

    if (ModelState.IsValid)
    {
        var user = new ApplicationUser { UserName = model.Email, Email = model.Email };
        var createResult = await _userManager.CreateAsync(user, model.Password);
        
        if (createResult.Succeeded)
        {
            // 先检查选中的角色是否存在，避免分配不存在的角色触发异常
            bool roleExists = await _roleManager.RoleExistsAsync(model.SelectedRoleName);
            // 如果你用的是角色ID，就改成FindByIdAsync判断是否不为null
            // var targetRole = await _roleManager.FindByIdAsync(model.SelectedRoleId);
            // bool roleExists = targetRole != null;

            if (roleExists)
            {
                await _userManager.AddToRoleAsync(user, model.SelectedRoleName);
                // 这里加你的后续逻辑，比如登录、跳转等
                return RedirectToLocal(returnUrl);
            }
            else
            {
                ModelState.AddModelError("", "所选角色不存在，请重新选择");
                return View(model);
            }
        }
        // 把创建用户的错误反馈给页面
        foreach (var error in createResult.Errors)
        {
            ModelState.AddModelError(string.Empty, error.Description);
        }
    }
    // ModelState无效时返回视图，此时下拉框已有角色数据
    return View(model);
}

4. 额外的异常防护 #

为了彻底避免未处理异常，你可以给角色加载的代码加个try-catch，同时判断角色列表是否为空：

try
{
    var allRoles = await _roleManager.Roles.ToListAsync();
    if (!allRoles.Any())
    {
        ModelState.AddModelError("", "系统中暂无可用角色，请先创建角色后再注册");
        return View(model);
    }
    // 后续逻辑
}
catch (Exception ex)
{
    ModelState.AddModelError("", $"加载角色列表时出错：{ex.Message}");
    return View(model);
}

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