重新认识onCheckedChanged(RadioGroup group, int checkedId)方法

一：bug描述

RadioGroup在onCheckedChanged方法中调用RadioGroup.clearCheck()会导致StackOverflowError
形如这样：

rg.setOnCheckedChangeListener(new RadioGroup.OnCheckedChangeListener() {
            @Override
            public void onCheckedChanged(RadioGroup group, @IdRes int checkedId) {
                switch (checkedId){
                    case R.id.rb1:
                        type="0";
                        break;
                    case R.id.rb2:
                        type="1";
                        rg.clearCheck();
                        break;
                    case R.id.rb3:
                        type="2";
                        break;
                }

                Log.e("type",type);

            }

详细bug信息如下：

 E/AndroidRuntime: FATAL EXCEPTION: mainProcess: com.dzy, PID: 10454
java.lang.StackOverflowError: stack size 8MB
    at android.widget.RadioGroup.check(RadioGroup.java:167)
    at android.widget.RadioGroup.clearCheck(RadioGroup.java:209)
    at com.dzy.menu.menu_item.comment.KawsV5CommentListActivity$8.onCheckedChanged(KawsV5CommentListActivity.java:212)
    at android.widget.RadioGroup.setCheckedId(RadioGroup.java:173)
    at android.widget.RadioGroup.check(RadioGroup.java:167)
    at android.widget.RadioGroup.clearCheck(RadioGroup.java:209)
    at com.dzy.menu.menu_item.comment.KawsV5CommentListActivity$8.onCheckedChanged(KawsV5CommentListActivity.java:212)
    at android.widget.RadioGroup.setCheckedId(RadioGroup.java:173)
    at android.widget.RadioGroup.check(RadioGroup.java:167)
    at android.widget.RadioGroup.clearCheck(RadioGroup.java:209)
    at com.dzy.menu.menu_item.comment.KawsV5CommentListActivity$8.onCheckedChanged(KawsV5CommentListActivity.java:212)
    at android.widget.RadioGroup.setCheckedId(RadioGroup.java:173)
    at android.widget.RadioGroup.check(RadioGroup.java:167)
    at android.widget.RadioGroup.clearCheck(RadioGroup.java:209)

报错中可以看到有三个方法陷入了死循环。

setCheck()
ckeck()
clearCheck()

因此我们在onCheckedChanged()中调用clearCheck的时候可以先设置

setOnCheckedChangeListener(null);

以避免出现这样的循环调用。

二:bug本质汉语版

其实质还是自己对OnCheckedChangeListener理解出错。曾以为这个接口的唯一回调方法中的参数checkdId是返回的选中的按钮的id，其实并不是，返回的而是状态改变的那个按钮的id。

public void onCheckedChanged(RadioGroup group, @IdRes int checkedId) 

这里面分为两种情况：

• 当按钮组中已经选择了按钮A，此时我们选择按钮B，按钮A的状态由选中变成了未选中，按钮B的状态由未选中变成了选中，那么毫无疑问这个方法回调的checkId会是由未选中变为选中的按钮B的id。
• 当我们通过clearCheck()／check(-1)/通过for循环遍历按钮组等方法，将按钮组中的所有按钮全部置为未选中的时候，这个onCheckedChanged()还是会被调用。并且会返回刚刚选中的那个按钮的id，因为刚刚选中的那个按钮状态会发生改变，而其他按钮状态不会发生改变。因此返回了那个状态由选中变为未选中的按钮id，其他未选中的按钮状态还是未选中。

三：bug本质代码版

查看RadioGroup源码，可以发现clearCheck调用的是check方法。

 public void clearCheck() {
        check(-1);
    }

那么check方法呢？

public void check(@IdRes int id) {
        // don't even bother

        if (id != -1 && (id == mCheckedId)) {
            return;
        }

        if (mCheckedId != -1) {
            setCheckedStateForView(mCheckedId, false);
        }

        if (id != -1) {
            setCheckedStateForView(id, true);
        }

        setCheckedId(id);
    }

其中一个成员变量mCheckedId是之前选择的那个按钮id，如果之前没有以选中的按钮，那么值为-1。所以在调用clearCheck()的时候，实际上传入check的值为-1。那么第一个判断是不可能进入的。由于之前有选中的按钮，因此第二个判断会进入setCheckedStateForView()。

private void setCheckedStateForView(int viewId, boolean checked) {
        View checkedView = findViewById(viewId);
        if (checkedView != null && checkedView instanceof RadioButton) {
            ((RadioButton) checkedView).setChecked(checked);
        }
    }

实际上这个方法就是调用了RadioButton本身的setChecked()方法。第三个判断也不会进入。最后调用了setCheckedId()方法开始回调，这时候传入的id为-1,看起来好像问题。

private void setCheckedId(@IdRes int id) {
        mCheckedId = id;
        if (mOnCheckedChangeListener != null) {
            mOnCheckedChangeListener.onCheckedChanged(this, mCheckedId);
        }
    }

关键是进入了第二个判断，我们看看CompoundButton中的setCheck方法

if (mOnCheckedChangeListener != null) {
    mOnCheckedChangeListener.onCheckedChanged(this, mChecked);
}
if (mOnCheckedChangeWidgetListener != null) {
    mOnCheckedChangeWidgetListener.onCheckedChanged(this, mChecked);
}

这里面又调用了onCheckedChanged()，不过这里调用的并不是RadioGroup中的OnCheckedChangeListener，但是在RadioGroup的init()方法中我们可以看到

private void init() {
    mChildOnCheckedChangeListener = new CheckedStateTracker();
    mPassThroughListener = new PassThroughHierarchyChangeListener();
    super.setOnHierarchyChangeListener(mPassThroughListener);
}

而这里的变量mChildOnCheckedChangeListener就是

image.png

看到这里应该就能明白了我之前在汉语版中的陈述，也就是在onCheckedChanged中的参数checkedId，无论是我上面说的第一种情况还是第二种情况，都是会返回状态改变的那个按钮的id。第一种情况是发生了两次改变，先将之前的选中状态置为未选中，再将此次选中的按钮由未选中置为选中状态。