Kotlin when语句变量访问+新冠自评RadioButton分值求和代码修复求助

我来帮你解决这两个问题：

你遇到的核心问题是作用域限制：在when分支的代码块（比如if (checked) { ... }内部）定义的变量，只能在该块的范围内被访问，块外部的代码完全看不到这些变量。

这里有两种实用的解决思路：

• 思路1：提前在when外部声明变量
  如果你需要在when执行完后使用这些变量，可以先在when语句的外部声明它们，再在分支内部赋值。示例：

  // 提前在when外部声明变量并初始化
  var yes1 = 0
  var no1 = 0
  
  when (view.getId()) {
      R.id.yes_1 -> if (checked) {
          yes1 = 1
          Toast.makeText(this, "Yes 1", Toast.LENGTH_LONG).show()
      } else {
          yes1 = 0 // 取消选中时重置分值
      }
      // 其他分支同理处理
  }
  
  // 现在可以正常访问yes1等变量
  

• 思路2：将when作为表达式返回值
  如果每个分支的逻辑是返回一个固定值，可以把when当作表达式使用，直接将分支结果赋值给一个变量。示例：

  val score = when (view.getId()) {
      R.id.yes_1 -> if (checked) 1 else 0
      R.id.no_1 -> 0
      // 其他分支依次定义返回值
      else -> 0
  }
  

你的代码无法正确计算总分，主要有这几个问题：

1. 变量作用域错误：所有yes1、no1等变量都定义在分支的if (checked)块内部，外部的assesment_point根本访问不到这些变量；
2. 拼写错误：代码里的n01是笔误，应该是no1；
3. 逻辑漏洞：每组RadioButton（比如yes_1和no_1）是互斥关系，但你的代码没有处理“取消选中”的情况，会导致分值计算混乱；
4. 代码冗余：重复的Toast和赋值逻辑太多，可维护性差。

下面是修复并优化后的代码，我推荐用RadioGroup管理每组选项（这是Android RadioButton的标准用法，能自动保证同一组内只能选中一个选项）：

第一步：在Activity类中声明总分数变量 #

// 用来存储所有选项的总分
private var totalScore = 0
// 记录每组的当前分值，避免重复计算
private val groupScoreMap = mutableMapOf(
    R.id.rg_1 to 0, R.id.rg_2 to 0, R.id.rg_3 to 0,
    R.id.rg_4 to 0, R.id.rg_5 to 0, R.id.rg_6 to 0
)

第二步：给每个RadioGroup设置选中监听 #

假设你的布局里，每组yes/no按钮都放在对应的RadioGroup中（比如rg_1对应yes_1/no_1，rg_2对应yes_2/no_2）：

// 批量给所有RadioGroup设置监听
val radioGroupIds = listOf(R.id.rg_1, R.id.rg_2, R.id.rg_3, R.id.rg_4, R.id.rg_5, R.id.rg_6)
radioGroupIds.forEach { rgId ->
    findViewById<RadioGroup>(rgId).setOnCheckedChangeListener { _, checkedId ->
        // 根据选中的按钮获取当前组的分值
        val currentGroupScore = when (checkedId) {
            R.id.yes_1, R.id.yes_2, R.id.yes_3, R.id.yes_4 -> 1
            R.id.yes_5, R.id.yes_6 -> 5
            else -> 0 // 选中no选项或未选中
        }

        // 更新总分：先减去该组之前的分值，再加上新分值
        totalScore -= groupScoreMap[rgId] ?: 0
        groupScoreMap[rgId] = currentGroupScore
        totalScore += currentGroupScore

        // 显示对应的Toast提示
        val toastText = when (checkedId) {
            R.id.yes_1 -> "Yes 1"
            R.id.no_1 -> "No 1"
            R.id.yes_2 -> "Yes 2"
            R.id.no_2 -> "No 2"
            R.id.yes_3 -> "Yes 3"
            R.id.no_3 -> "No 3"
            R.id.yes_4 -> "Yes 4"
            R.id.no_4 -> "No 4"
            R.id.yes_5 -> "Yes 5"
            R.id.no_5 -> "No 5"
            R.id.yes_6 -> "Yes 6"
            R.id.no_6 -> "No 6"
            else -> ""
        }
        val toastDuration = if (checkedId.toString().contains("yes")) Toast.LENGTH_LONG else Toast.LENGTH_SHORT
        Toast.makeText(this, toastText, toastDuration).show()
    }
}

第三步：点击评估按钮时计算结果 #

// 假设你有一个id为btn_assess的评估按钮
findViewById<Button>(R.id.btn_assess).setOnClickListener {
    val assessmentResult = when {
        totalScore >= 8 -> "您的感染风险较高，建议尽快进行新冠检测"
        totalScore in 4..7 -> "您存在一定感染风险，建议密切关注身体状况"
        else -> "您的感染风险较低，暂无需检测"
    }
    Toast.makeText(this, "总分：$totalScore\n$assessmentResult", Toast.LENGTH_LONG).show()
}

优化说明： #

• 用RadioGroup自动处理互斥选中，避免手动维护按钮状态的冗余代码；
• 用groupScoreMap记录每组的当前分值，确保总分计算准确；
• 总分数totalScore在类级别声明，所有监听都能修改和访问它；
• 归类处理相同分值的选项，减少重复逻辑。

如果你坚持使用单个RadioButton的点击监听，也可以把所有分值变量提升到类级别，并手动处理互斥选中：

private var score1 = 0
private var score2 = 0
private var score3 = 0
private var score4 = 0
private var score5 = 0
private var score6 = 0

override fun onClick(view: View) {
    if (view is RadioButton) {
        val checked = view.isChecked
        when (view.id) {
            R.id.yes_1 -> {
                score1 = if (checked) 1 else 0
                Toast.makeText(this, "Yes 1", Toast.LENGTH_LONG).show()
                findViewById<RadioButton>(R.id.no_1).isChecked = false
            }
            R.id.no_1 -> {
                score1 = if (checked) 0 else 0
                Toast.makeText(this, "No 1", Toast.LENGTH_SHORT).show()
                findViewById<RadioButton>(R.id.yes_1).isChecked = false
            }
            // 其他yes/no分支同理处理...
        }
        // 现在可以正确计算总分
        val assessmentPoint = score1 + score2 + score3 + score4 + score5 + score6
    }
}

不过还是更推荐RadioGroup的实现方式，它更符合Android的设计规范，代码也更简洁可靠。

内容的提问来源于stack exchange，提问作者Aurelio Chelestino Mofuga