Kotlin简易计算器等于按钮失效：条件为真却不执行分支

我帮你找到了两个关键问题，咱们一步步拆解：

核心问题1：操作符按钮的输入获取逻辑错误 #

你在加法（及其他操作符）按钮的点击事件外部定义了val input = expression.text.toString().trim()——这个变量只会在页面初始化时获取一次expression的文本内容，完全不会响应你点击按钮时的实时输入。

举个例子：你输入5后点击加号，此时input还是初始化时的空值，代码会直接进入if (input.isNullOrBlank())分支，operator根本不会被设置为"sum"。最终等于按钮里的operator始终是初始值"n"，自然不会触发任何运算分支。

次要问题2：等于按钮的代码顺序逻辑错误 #

你在判断operator之前就执行了expression.text = operator，就算后续运算成功，也会先把文本改成"n"（因为operator没被正确设置），再被运算结果覆盖——这会导致短暂的错误显示，也不符合正常计算器的交互逻辑。

修复后的代码示例 #

1. 修正操作符按钮（以加法为例，减法/乘法/除法同理） #

plus.setOnClickListener {
    // 把input移到点击事件内部，实时获取当前输入内容
    val input = expression.text.toString().trim()
    if (input.isNullOrBlank()) {
        expression.text = ""
    } else {
        operator = "sum"
        value1 = input.toFloat() // 直接用已获取的input，避免重复读取控件
        expression.text = ""
    }
}

2. 修正等于按钮（优化逻辑+增加异常处理） #

equals.setOnClickListener {
    val input2 = expression.text.toString().trim()
    // 增加operator判断，确保已选择操作符且输入有效
    if (!input2.isNullOrBlank() && operator != "n") {
        try {
            value2 = input2.toFloat()
            val result = when(operator) {
                "sum" -> value1 + value2
                "sub" -> value1 - value2
                "div" -> {
                    // 增加除零判断，避免崩溃
                    if (value2 == 0f) {
                        expression.text = "Error"
                        operator = "n"
                        return@setOnClickListener
                    }
                    value1 / value2
                }
                "multi" -> value1 * value2
                else -> 0f
            }
            expression.text = result.toString()
            operator = "n" // 运算完成后重置操作符
        } catch (e: NumberFormatException) {
            // 处理非数字输入的情况
            expression.text = "Invalid"
            operator = "n"
        }
    } else {
        // 未输入有效内容或未选操作符时，清空输入框
        expression.text = ""
    }
}

额外优化建议 #

• 给所有操作符按钮统一提取输入校验逻辑，避免重复代码；
• 可以给输入框设置输入类型为numberDecimal，限制用户只能输入数字和小数点；
• 运算结果可以保留指定小数位数，提升显示美观度。

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