Kotlin新手BMI计算应用输入校验问题求助

你的代码核心问题：只在App启动时获取一次输入值 #

不管是第一个还是第二个版本，你都是在onCreate方法里一次性读取了EditText的内容——但App刚启动时，输入框是空的，所以weight和height一开始就是null，导致后续逻辑完全不符合预期：

• 第一个版本：按钮永远被禁用，且点击事件从未被设置（因为if分支没执行）
• 第二个版本：启动时直接进入else弹出提示，之后用户输入内容后，按钮虽然可点击，但点击事件还是没被设置（因为启动时if分支没触发），所以点击后没反应

正确的实现思路 #

你需要实时监听输入框的文本变化，并且在按钮点击时才去获取当前最新的输入值并校验，而不是只在启动时读一次。

修正后的代码示例 #

class MainActivity : AppCompatActivity() {
    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        setContentView(R.layout.activity_main)
        
        val editTextWeight = findViewById<EditText>(R.id.weightEditText)
        val editTextHeight = findViewById<EditText>(R.id.heightEditText)
        val calculateButton = findViewById<Button>(R.id.calcButton)

        // 先禁用按钮
        calculateButton.isEnabled = false

        // 监听两个输入框的文本变化，实时更新按钮状态
        val textWatcher = object : TextWatcher {
            override fun beforeTextChanged(s: CharSequence?, start: Int, count: Int, after: Int) {}
            override fun onTextChanged(s: CharSequence?, start: Int, before: Int, count: Int) {
                // 每次文本变化时，获取当前的输入值
                val weight = editTextWeight.text.toString().toDoubleOrNull()
                val height = editTextHeight.text.toString().toDoubleOrNull()
                // 只有当两个值都不为空时，启用按钮
                calculateButton.isEnabled = weight != null && height != null
            }
            override fun afterTextChanged(s: Editable?) {}
        }

        // 给两个输入框添加文本监听
        editTextWeight.addTextChangedListener(textWatcher)
        editTextHeight.addTextChangedListener(textWatcher)

        // 始终设置按钮的点击事件，在点击时才处理计算逻辑
        calculateButton.setOnClickListener {
            val weight = editTextWeight.text.toString().toDoubleOrNull()
            val height = editTextHeight.text.toString().toDoubleOrNull()
            
            // 这里再次校验（防止极端情况，比如按钮启用后用户又清空输入）
            if (weight != null && height != null) {
                val bmi = weight / (height * height)
                Toast.makeText(this@MainActivity, "Your BMI is %.2f".format(bmi), Toast.LENGTH_LONG).show()
            } else {
                Toast.makeText(this@MainActivity, "Please enter both values correctly!", Toast.LENGTH_LONG).show()
            }
        }
    }
}

关于类似Angular的状态机制 #

Android里有专门的组件来分离UI初始化和业务逻辑，最常用的是ViewModel（属于Jetpack组件库）：

• ViewModel负责存储和处理业务逻辑（比如BMI计算、输入校验规则），它的生命周期独立于Activity，不会因为屏幕旋转等配置变化而销毁
• 你可以用LiveData或StateFlow来监听输入数据的变化，UI层（Activity/Fragment）只需要观察这些数据，更新UI状态（比如按钮是否可用）
• 这样就能把输入校验、计算逻辑从Activity中抽离出来，让UI层只专注于界面展示和用户交互

举个简单的ViewModel示例：

class BmiViewModel : ViewModel() {
    // 用MutableStateFlow存储输入值
    private val _weight = MutableStateFlow<Double?>(null)
    private val _height = MutableStateFlow<Double?>(null)
    
    // 对外暴露不可变的StateFlow
    val weight = _weight.asStateFlow()
    val height = _height.asStateFlow()
    
    // 校验是否可以计算BMI
    val canCalculate = combine(weight, height) { w, h ->
        w != null && h != null
    }.stateIn(viewModelScope, SharingStarted.WhileSubscribed(), false)

    // 更新体重输入
    fun updateWeight(input: String) {
        _weight.value = input.toDoubleOrNull()
    }

    // 更新身高输入
    fun updateHeight(input: String) {
        _height.value = input.toDoubleOrNull()
    }

    // 计算BMI
    fun calculateBmi(): Double? {
        val w = _weight.value
        val h = _height.value
        return if (w != null && h != null) {
            w / (h * h)
        } else {
            null
        }
    }
}

然后在Activity中使用这个ViewModel：

class MainActivity : AppCompatActivity() {
    private val viewModel: BmiViewModel by viewModels()

    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        setContentView(R.layout.activity_main)
        
        val editTextWeight = findViewById<EditText>(R.id.weightEditText)
        val editTextHeight = findViewById<EditText>(R.id.heightEditText)
        val calculateButton = findViewById<Button>(R.id.calcButton)

        // 监听输入框变化，通知ViewModel更新数据
        editTextWeight.addTextChangedListener(object : TextWatcher {
            override fun beforeTextChanged(s: CharSequence?, start: Int, count: Int, after: Int) {}
            override fun onTextChanged(s: CharSequence?, start: Int, before: Int, count: Int) {
                viewModel.updateWeight(s.toString())
            }
            override fun afterTextChanged(s: Editable?) {}
        })

        editTextHeight.addTextChangedListener(object : TextWatcher {
            override fun beforeTextChanged(s: CharSequence?, start: Int, count: Int, after: Int) {}
            override fun onTextChanged(s: CharSequence?, start: Int, before: Int, count: Int) {
                viewModel.updateHeight(s.toString())
            }
            override fun afterTextChanged(s: Editable?) {}
        })

        // 观察ViewModel中的canCalculate状态，更新按钮可用性
        lifecycleScope.launch {
            viewModel.canCalculate.collect { canCalc ->
                calculateButton.isEnabled = canCalc
            }
        }

        // 按钮点击时调用ViewModel的计算方法
        calculateButton.setOnClickListener {
            val bmi = viewModel.calculateBmi()
            if (bmi != null) {
                Toast.makeText(this@MainActivity, "Your BMI is %.2f".format(bmi), Toast.LENGTH_LONG).show()
            } else {
                Toast.makeText(this@MainActivity, "Please enter both values correctly!", Toast.LENGTH_LONG).show()
            }
        }
    }
}

这样就完美实现了UI和业务逻辑的分离，和Angular的状态管理思路类似，让代码更清晰、更易维护。

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