C# While循环角色输入验证异常：仅User输入可被正常接受

你写的C#程序里，只有输入User时能被正常接受，Administrator和Manager都会被判定为无效，核心问题出在while循环里的条件判断逻辑错误。

错误原因分析 #

原代码里的判断条件：

if (!(role.Trim().ToLower().Equals("administrator") && (role.Trim().ToLower().Equals("manager")) || (role.Trim().ToLower().Equals("user"))))

因为&&的优先级比||高，这段逻辑实际等价于!((A && B) || C)——也就是只有当输入同时是Administrator和Manager（不可能发生），或者是User时，条件才为假，才会跳出循环；其他所有输入都会触发重新输入的提示，这就是为什么只有User能通过。

修复后的完整代码 #

Console.WriteLine("请输入以下角色之一：Administrator、Manager 或 User");
string? roleInput;
while (true)
{
    roleInput = Console.ReadLine();
    // 处理空输入或全空格的情况
    if (string.IsNullOrWhiteSpace(roleInput))
    {
        Console.WriteLine("输入不能为空，请重新输入：Administrator、Manager 或 User");
        continue;
    }
    // 统一标准化输入格式，避免重复调用方法
    string normalizedRole = roleInput.Trim().ToLower();
    // 判断输入是否属于有效角色，无效则提示重新输入
    if (!(normalizedRole == "administrator" || normalizedRole == "manager" || normalizedRole == "user"))
    {
        Console.WriteLine("输入无效，请重新输入：Administrator、Manager 或 User");
        continue;
    }
    // 输入有效，跳出循环
    break;
}

// 循环结束时roleInput一定非空，用!抑制空值警告
string acceptedRole = roleInput!.Trim().ToLower();
Console.WriteLine($"输入的角色 {acceptedRole} 已被接受");

关键修复点 #

• 修正逻辑判断：把有效角色用||连接后取反，明确判断“输入不在有效列表中”时触发重新输入
• 优化代码效率：提前将输入标准化（Trim+ToLower）并赋值给变量，避免重复调用相同方法
• 增加空值处理：用string.IsNullOrWhiteSpace处理用户直接回车或输入全空格的情况，避免空引用异常
• 简化代码结构：去掉冗余的else分支，逻辑更清晰

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