LiveData异常后不走回调

前言:测试大佬突然发现页面上有一块UI在某些情况下无内容展示,但是接口有下发对应字段。这种偶现的bug就很头秃了,排查一波也没发现啥异常,就是个简单的接口请求,然后在LiveData回调中更新UI。这样看来问题出在LiveData了?正好本地也复现了这个问题,疯狂刷新调用接口,UI也没发生变化。

赶紧debug一下,发现LiveData.setValue()此时卡在了dispatchingValue()方法开头处,mDispatchingValue为true,直接return,没有走后续的observer.onChange()回调。mDispatchingValue字段顾名思义,标记正在分发事件,此时继续setValue()不会立即更新。mDispatchingValue是个全局变量,找一下引用发现赋值处全在dispatchingValue()方法中。

    void dispatchingValue(@Nullable ObserverWrapper initiator) {
        if (mDispatchingValue) {
            mDispatchInvalidated = true;
            return;
        }
        mDispatchingValue = true;
        do {
            mDispatchInvalidated = false;
            if (initiator != null) {
                considerNotify(initiator);
                initiator = null;
            } else {
                for (Iterator, ObserverWrapper>> iterator =
                        mObservers.iteratorWithAdditions(); iterator.hasNext(); ) {
                    considerNotify(iterator.next().getValue());
                    if (mDispatchInvalidated) {
                        break;
                    }
                }
            }
        } while (mDispatchInvalidated);
        mDispatchingValue = false;
    }

所以说正常流程下mDispatchingValue初始值应该为false,然后在do的入口处置为true,标记正在分发事件,分发事件后跳出循环再次置为fasle。有可能出现问题的地方就在do中的分发事件,considerNotify()就是我们的回调,回调中如果出现异常,执行逻辑跳到外部catch块,mDispatchingValue就没法在最后重置为false,那么后续的setValue()都会被return掉。

这里有的同学就会问了,LiveData回调中出现异常不就崩溃了吗,崩溃了就能找到问题解决了啊,还哪有后续不分发事件的坑呢。but基础框架大概率会把接口调用整个流程try catch住,我这边用的协程,整个协程体都被try catch了所以不会崩溃。我这边的case可以等同于把setValue()方法try catch住,然后应该是某个接口字段的问题抛出异常,进而导致后续事件不再分发。

思考了一下,找到接口字段fix掉貌似是个不错的选择,但是吧,我们能相信后端吗?就算和后端大佬遵守了彼此的约定,那我们能保证回调中永远不出现异常吗?回调中的业务代码崩溃了其实也还好,起码能快速定位然后解决,就怕被外部catch住了,然后一脸懵逼不走回调。

先来个兜底方案吧,在你确定回调被catch住不会崩溃进而引发这个bug的时候,不妨自己catch处理。

fun  MutableLiveData.safeObserve(owner: LifecycleOwner, onChange: (T) -> Unit) {
    this.observe(owner) {
        try {
            onChange(it)
        } catch (e: Exception) {
            e.printStackTrace()
        }
    }
}

思路很简单,套一层try catch,即使回调中产生异常,dispatchingValue()方法也会走完。

当然啦,该改的还是要改,具体是哪个字段没下发有可能为空还是要保护一下的。我应该是被data class坑了,解析json反射给字段赋值不会走构造函数的默认值,最好都写成可空类型。

如果你不希望发生崩溃,也不希望LiveData后续永远不走回调,可以尝试上面这个方案。搬砖式的把observe都改为safeObserve也很难受啊,ASM替换一下调用比较舒服,这个没什么难度也就不展开了。ASM统一替换的话不要太暴力,搞个注解啥的做成可配才是王道。

你可能感兴趣的:(LiveData异常后不走回调)