Android Kotlin中setOnClickListener失效问题求助

嘿，我帮你排查出代码里的几个关键问题了，这就是按钮没反应、逻辑出错的根源：

问题1：按钮点击事件根本没绑定上 #

你把solveBtn.setOnClickListener{...}写在solver函数里面了，但这个solver函数在onCreate里从来没被调用过！相当于按钮从来没被设置过点击监听，点它当然没反应啦。

问题2：获取用户输入的方式完全错了 #

var num = NumInput.inputType.toInt()这行代码大错特错！inputType是EditText的「输入类型配置」（比如数字输入、文本输入这类系统常量），不是用户输入的文本内容，你拿到的是个莫名其妙的常量值，根本不是用户输入的数字。

问题3：质数判断逻辑搞反了 #

你的solver函数里的判断逻辑完全颠倒了：质数的定义是只能被1和自身整除，你现在的逻辑会把质数判断成非质数，非质数判断成质数，完全搞反了。

修复后的完整代码 #

你的XML布局没问题，直接保留 #

<Button 
    android:id="@+id/solveBtn" 
    android:layout_width="113dp" 
    android:layout_height="65dp" 
    android:layout_marginBottom="184dp" 
    android:layout_marginEnd="8dp" 
    android:layout_marginStart="8dp" 
    android:layout_marginTop="8dp" 
    android:text="Solve" 
    app:layout_constraintBottom_toBottomOf="parent" 
    app:layout_constraintEnd_toEndOf="parent" 
    app:layout_constraintHorizontal_bias="0.501" 
    app:layout_constraintStart_toStartOf="parent" 
    app:layout_constraintTop_toBottomOf="@+id/NumInput" />

修复后的Kotlin代码 #

class MainActivity : AppCompatActivity() {
    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        setContentView(R.layout.activity_main)

        // 先拿到所有需要的控件实例
        val numInput = findViewById<EditText>(R.id.NumInput)
        val answerText = findViewById<TextView>(R.id.ansText)
        val solveBtn = findViewById<Button>(R.id.solveBtn)

        // 直接在onCreate里给按钮绑定点击事件，确保Activity启动就生效
        solveBtn.setOnClickListener {
            // 点击按钮时才去拿用户输入的内容
            val inputContent = numInput.text.toString().trim()
            
            // 先做输入合法性校验，避免崩溃或错误结果
            when {
                inputContent.isEmpty() -> {
                    answerText.text = "请输入一个正整数哦！"
                    return@setOnClickListener
                }
                inputContent.toIntOrNull() == null -> {
                    answerText.text = "输入的不是有效数字，请重新输入！"
                    return@setOnClickListener
                }
                inputContent.toInt() <= 1 -> {
                    answerText.text = "请输入大于1的正整数！"
                    return@setOnClickListener
                }
            }

            // 把输入转换成整数，调用质数判断函数
            val inputNum = inputContent.toInt()
            checkPrime(inputNum, answerText)
        }
    }

    // 独立的质数判断函数，职责单一，更易维护
    private fun checkPrime(num: Int, resultView: TextView) {
        var isPrime = true
        // 优化：只需要检查到num的平方根，提升效率
        val sqrtLimit = kotlin.math.sqrt(num.toDouble()).toInt()
        
        for (divisor in 2..sqrtLimit) {
            if (num % divisor == 0) {
                isPrime = false
                break // 找到因数就不用继续循环了
            }
        }

        // 根据判断结果设置文本
        resultView.text = if (isPrime) "$num 是质数" else "$num 不是质数"
    }
}

核心修复点解释 #

1. 按钮监听正确绑定：把点击事件的代码移到onCreate里，确保Activity一启动就给按钮挂上监听，点击就能响应。
2. 正确获取输入：用numInput.text.toString()拿到用户真正输入的内容，还加了输入校验，避免空输入、非数字导致的崩溃。
3. 修复质数逻辑：修正了判断逻辑，还优化了循环范围（只到平方根），比原来的遍历到num-1高效多了。
4. 代码结构优化：把质数判断拆成独立函数，代码更清晰，以后要改逻辑也方便。

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