重新认识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。第一种情况是发生了两次改变,先将之前的选中状态置为未选中,再将此次选中的按钮由未选中置为选中状态。

最后编辑于
©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 203,772评论 6 477
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 85,458评论 2 381
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 150,610评论 0 337
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 54,640评论 1 276
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 63,657评论 5 365
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 48,590评论 1 281
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 37,962评论 3 395
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 36,631评论 0 258
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 40,870评论 1 297
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 35,611评论 2 321
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 37,704评论 1 329
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 33,386评论 4 319
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 38,969评论 3 307
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 29,944评论 0 19
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 31,179评论 1 260
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 44,742评论 2 349
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 42,440评论 2 342

推荐阅读更多精彩内容

  • ¥开启¥ 【iAPP实现进入界面执行逐一显】 〖2017-08-25 15:22:14〗 《//首先开一个线程,因...
    小菜c阅读 6,350评论 0 17
  • Android 自定义View的各种姿势1 Activity的显示之ViewRootImpl详解 Activity...
    passiontim阅读 171,409评论 25 707
  • Spring Cloud为开发人员提供了快速构建分布式系统中一些常见模式的工具(例如配置管理,服务发现,断路器,智...
    卡卡罗2017阅读 134,590评论 18 139
  • 工厂模式类似于现实生活中的工厂可以产生大量相似的商品,去做同样的事情,实现同样的效果;这时候需要使用工厂模式。简单...
    舟渔行舟阅读 7,716评论 2 17
  • 蜗子阅读 152评论 0 0