Android Studio用Java实现两组RadioButton勾选后启用Submit按钮问题求解

问题原因 #

• 逻辑运算符优先级错误：&&的运算优先级高于||，你原有判断条件没有加括号分组，实际执行逻辑和预期完全不符
• 条件判断逻辑写反：你需要的是「两组都有选中才启用按钮」，原有if条件的判断规则刚好和需求相反
• 冗余代码：重复定义了两个完全相同的checked/checkeds变量，手动实现RadioButton互斥的逻辑多余且容易出错

修复方案 #

快速修复（不改现有布局） #

直接替换你原有if判断部分的代码即可，不需要修改其他逻辑：

// 分别判断两组是否有选中项
boolean group1Selected = thirty.isChecked() || fourty.isChecked() || fifty.isChecked();
boolean group2Selected = geo.isChecked() || eng.isChecked() || country.isChecked() || movi.isChecked();
// 两组都选中才启用提交按钮
submit.setEnabled(group1Selected && group2Selected);

最优优化方案（推荐） #

在布局文件中将两组RadioButton分别用RadioGroup包裹，系统会自动处理同组RadioButton的互斥逻辑，不需要你手动写setChecked(false)的代码，判断选中状态也更简单：

1. 布局示例（省略无关属性）：

<!-- 第一组3个RadioButton -->
<RadioGroup
    android:id="@+id/rg_score"
    android:orientation="horizontal">
    <RadioButton android:id="@+id/thirty" .../>
    <RadioButton android:id="@+id/fourty" .../>
    <RadioButton android:id="@+id/fifty" .../>
</RadioGroup>

<!-- 第二组4个RadioButton -->
<RadioGroup
    android:id="@+id/rg_mode"
    android:orientation="horizontal">
    <RadioButton android:id="@+id/geo" .../>
    <RadioButton android:id="@+id/eng" .../>
    <RadioButton android:id="@+id/country" .../>
    <RadioButton android:id="@+id/movi" .../>
</RadioGroup>

2. 代码简化后：

// 只需要这三行即可完成按钮状态判断
boolean group1Selected = rg_score.getCheckedRadioButtonId() != -1;
boolean group2Selected = rg_mode.getCheckedRadioButtonId() != -1;
submit.setEnabled(group1Selected && group2Selected);

// 取值也不需要写大段switch，直接取选中的id即可
int selectScoreId = rg_score.getCheckedRadioButtonId();
if (selectScoreId == R.id.thirty) {
    scorelimit = 3;
} else if (selectScoreId == R.id.fourty) {
    scorelimit =4;
} else if (selectScoreId == R.id.fifty) {
    scorelimit =5;
}
// 第二组取值同理

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