Asp.net Core 2.0禁用用户代码正确性及视图实现咨询

一、你的控制器代码存在的问题及修正方案 #

首先，你目前的代码有几个明显的问题：

1. 参数类型错误：控制器方法接收的是ApplicationUser类型的参数，但实际请求中通常只会传递用户ID（字符串类型），而不是整个用户实体，这样模型绑定会直接失败。
2. 代码逻辑不完整：你的代码只写到if(Id == nu，既没有闭合判断逻辑，也没有调用SetLockoutEnabledAsync实现核心锁定功能。

下面是修正后的完整控制器代码：

// 记得在控制器构造函数中注入UserManager<ApplicationUser>
private readonly UserManager<ApplicationUser> _userManager;

public YourControllerName(UserManager<ApplicationUser> userManager)
{
    _userManager = userManager;
}

[HttpPost]
public async Task<IActionResult> Disable(string userId)
{
    if (string.IsNullOrEmpty(userId))
    {
        return BadRequest("用户ID不能为空");
    }

    var user = await _userManager.FindByIdAsync(userId);
    if (user == null)
    {
        return NotFound("未找到指定用户");
    }

    // 启用锁定功能（核心操作：仅限制登录，完全保留用户所有数据）
    var lockoutResult = await _userManager.SetLockoutEnabledAsync(user, true);
    if (lockoutResult.Succeeded)
    {
        // 可选：设置永久锁定，让用户立即无法登录
        await _userManager.SetLockoutEndDateAsync(user, DateTimeOffset.MaxValue);
        return RedirectToAction("Index"); // 重定向回用户列表页
    }

    // 如果锁定操作失败，返回错误信息
    foreach (var error in lockoutResult.Errors)
    {
        ModelState.AddModelError(string.Empty, error.Description);
    }
    return View(user);
}

说明：

• SetLockoutEnabledAsync(user, true)是完全符合你需求的核心方法：它只是开启用户的锁定开关，不会删除任何用户数据，仅限制用户登录系统。
• 额外调用SetLockoutEndDateAsync是为了立即触发锁定效果，如果只是开启锁定功能但暂不生效，可以去掉这一步。

二、对应的视图实现方案 #

通常我们会在用户列表视图中添加锁定操作按钮，或者单独做一个确认视图，这里提供两种常用实现方式：

方式1：用户列表页直接添加锁定按钮（表单提交） #

在用户列表的每个条目后，嵌入一个表单提交到Disable方法，点击按钮时触发确认提示：

@model IEnumerable<ApplicationUser>

<table class="table">
    <thead>
        <tr>
            <th>用户名</th>
            <th>邮箱</th>
            <th>状态</th>
            <th>操作</th>
        </tr>
    </thead>
    <tbody>
        @foreach (var user in Model)
        {
            <tr>
                <td>@user.UserName</td>
                <td>@user.Email</td>
                <td>@(user.LockoutEnabled ? "<span class='text-danger'>已锁定</span>" : "<span class='text-success'>正常</span>")</td>
                <td>
                    @if (!user.LockoutEnabled)
                    {
                        <form asp-action="Disable" method="post" style="display:inline;">
                            <input type="hidden" name="userId" value="@user.Id" />
                            <button type="submit" class="btn btn-sm btn-danger" onclick="return confirm('确定要锁定该用户吗？此操作不会删除用户数据');">锁定</button>
                        </form>
                    }
                </td>
            </tr>
        }
    </tbody>
</table>

方式2：单独的锁定确认视图（适合需要额外提示的场景） #

如果需要更正式的确认流程，可以创建Disable.cshtml视图：

@model ApplicationUser

<h3>确认锁定用户</h3>
<div class="alert alert-warning">
    你确定要锁定用户 <strong>@Model.UserName</strong> 吗？<br>
    此操作仅限制用户登录系统，不会删除该用户的任何数据。
</div>

<form asp-action="Disable" method="post">
    <input type="hidden" name="userId" value="@Model.Id" />
    <button type="submit" class="btn btn-danger">确认锁定</button>
    <a asp-action="Index" class="btn btn-secondary ms-2">取消</a>
</form>

此时需要给控制器添加一个GET请求的Disable方法，用于返回确认视图：

[HttpGet]
public async Task<IActionResult> Disable(string userId)
{
    if (string.IsNullOrEmpty(userId))
    {
        return BadRequest("用户ID不能为空");
    }

    var user = await _userManager.FindByIdAsync(userId);
    if (user == null)
    {
        return NotFound("未找到指定用户");
    }

    return View(user);
}

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