C#计算器运算符校验IsOperator布尔方法始终返回false如何解决

问题根因 #

• 输入参数取值错误
  调用IsOperator时传入的txtOperator.ToString()写法错误：如果txtOperator是文本框控件实例，ToString()默认返回的是控件的类名字符串（例如System.Windows.Forms.TextBox），而非用户输入的运算符内容，自然无法匹配switch分支。正确取值需要调用控件的Text属性。
• 未处理输入首尾空白
  用户输入运算符时可能误输入空格、制表符等空白字符，直接传入的字符串和case分支的常量值不匹配，也会导致校验失败。

修复代码 #

调用侧修改 #

// 取输入框内容、去除首尾空白后再传入校验
if (IsOperator(txtOperator.Text.Trim(), "Operator"))
{
    // 原有业务逻辑
}

校验方法优化（可选） #

可以给方法增加空值防护，同时简化判断逻辑：

public bool IsOperator(string input, string name)
{
    if (string.IsNullOrWhiteSpace(input))
    {
        MessageBox.Show(name + " must be a valid math operator.", "Entry Error");
        return false;
    }
    return input is "*" or "x" or "/" or "+" or "-";
}
// 低版本C#不支持模式匹配的话可以替换为：
// return new System.Collections.Generic.HashSet<string> { "*", "x", "/", "+", "-" }.Contains(input);

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