Android扫码应用崩溃排查：RadioButton空指针异常问题分析

嘿，我帮你梳理下这个问题：你遇到的NullPointerException本质是表单检查函数checkForms()的逻辑漏洞，导致部分未选中的RadioGroup没被检测到，最终在output()里调用RadioButton.getText()时触发崩溃。

问题根源拆解 #

先看报错信息：

2020-02-08 20:50:56.692 21553-21553/com.example.scoutingapp3250 E/AndroidRuntime: FATAL EXCEPTION: main Process: com.example.scoutingapp3250, PID: 21553 java.lang.NullPointerException: Attempt to invoke virtual method 'java.lang.CharSequence android.widget.RadioButton.getText()' on a null object reference

这说明你在调用getText()时，RadioButton对象是null。为什么会这样？因为对应的RadioGroup没有选中任何选项，getCheckedRadioButtonId()返回-1，findViewById(-1)自然返回null。

而checkForms()本该拦截这种情况，但它的循环逻辑有致命问题：

switch (scanState) {
    case scanning:
        if (element instanceof EditText) {
            // 检查EditText...
            break;
        }
        if (element instanceof RadioGroup) {
            // 检查RadioGroup...
        }
        scanState = ScanState.stopped; // 这里是关键漏洞！
        break;
}

当第一个元素是已填写的EditText时：

1. 执行完EditText的判断后break出switch
2. 紧接着把scanState设为stopped
3. 后续所有循环元素都会进入case stopped:，直接跳过检查！

这就导致第一个EditText之后的所有RadioGroup都没被检查，如果这些RadioGroup没选中选项，checkForms()会错误返回false，允许跳转到QR页面，最终触发NPE。

修复步骤 #

1. 重构checkForms()函数

去掉冗余的状态机，确保每个表单元素都被检查：

public boolean checkForms() {
    boolean isEmpty = false;
    ScrollView scrollView = findViewById(R.id.scroll);
    LinearLayout linearLayout = findViewById(R.id.linearLayout);
    final int childCount = linearLayout.getChildCount();
    
    // 找到空值就停止循环，提升效率
    for (int i = 0; i < childCount && !isEmpty; i++) {
        View element = linearLayout.getChildAt(i);
        
        if (element instanceof EditText) {
            EditText editText = (EditText) element;
            // 用trim()过滤空格，避免用户输入空格被误判为已填写
            if (editText.getText().toString().trim().isEmpty()) {
                scrollView.scrollTo(0, editText.getTop() + 10);
                isEmpty = true;
            }
        } else if (element instanceof RadioGroup) {
            RadioGroup radioGroup = (RadioGroup) element;
            if (radioGroup.getCheckedRadioButtonId() == -1) {
                scrollView.scrollTo(0, radioGroup.getTop());
                isEmpty = true;
            }
        }
    }
    return isEmpty;
}

2. 给output()加安全防护（可选但推荐）

即使修复了检查逻辑，为了避免意外漏检，建议在获取RadioButton文本时做null检查：

// 示例：安全获取RadioGroup选中值
RadioGroup alliance = findViewById(R.id.alliance);
int allianceCheckedId = alliance.getCheckedRadioButtonId();
String allianceText = allianceCheckedId != -1 
    ? ((RadioButton) findViewById(allianceCheckedId)).getText().toString() 
    : ""; // 也可以设置默认提示文本

把所有RadioButton的获取都改成这种方式，从根源避免NPE。

3. 完善QR生成页面的空值处理

最后，在QR生成页面也要处理data为空的情况：

Intent intent = getIntent();
String test = intent.getStringExtra("data");
// 处理null或空字符串，避免生成QR时出错
String data_in_code = test != null ? test.trim() : "";

这样就能彻底解决空值导致的应用崩溃问题了。

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