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