UITabbarController 偶现启动crash问题分析

问题背景

最近新版本发布后,出现了一个偶现的crash并且迅速增加为Top1,这里对该问题做一个分析。
报错内容如下:
NSException -[UITabBarController setSelectedViewController:] only a view controller in the tab bar controller's list of view controllers can be selected.
crash堆栈如下:

问题分析

APM分析

首先是查看APM提供的信息,没有系统聚集、没有机型聚集,是版本新增问题都是隐私界面(也就是新用户冷启场景)。

该问题在灰度有出现过,一位同事在排查过程中,发现另外一个类似问题是在UITabBarController的 _viewControllerForTabBarItem:方法出现异常,这个问题量级并不大,场景类似但是没有特别信息帮助定位。

多维分析

由于crash出现在系统的UITabBarController类,无法调试获取更多信息,逆向排查周期太长。这里可以通过Slardar的信息,结合日志和业务场景逐步缩小排查范围。
首先通过crash场景,我们猜测是在用户新用户冷启才会遇到,这里通过回捞日志和crash的pv/uv相比可以确定;
其次通过排查新用户冷启场景的特有逻辑,关注点放在新版本相关的代码和实验改动,发现在底tab在新用户冷启场景的底tab刷新逻辑有较大可疑。
结合crash信息only a view controller in the tab bar controller's list of view controllers can be selected以及crash堆栈里有viewWillAppear时机,合理猜测一个场景:是否tab切换时,导致某个vc不在tabbar的子vc里面。比如说,没有某个tab但是又指定跳到该vc,类似self.tabbarVC setSelectedViewController:self.xxxVC;又或者,某个子vc不在self.viewConrollers里面,但是又要跳转到该vc。

通过业务代码排查,业务并无直接设置setSelectedViewController的操作;在排查过程中发现只有setSelectedIndex的操作,从堆栈上来看,如果是setSelectedIndex触发crash,堆栈上应该会有这个方法。
于是重点排查子vc不存在的情况,在查看新用户切换tab的逻辑时,发现了有一个vc复用的逻辑,旧tabbarVC的vc会被复用到新的tabbarVC,结合ViewController只能有一个parentVC的限制,从逻辑上分析是有可能出现堆栈所描述的场景。
结合这个猜测,当vc被复用到新的tabbarVC时,加了一段代码让新的tabbarVC不添加到window,从而旧的tabbar继续触发viewWillAppear,问题可以复现。

反向分析

当问题可以稳定复现后,就可以进一步分析逻辑上的缺陷。
首先是vc的复用逻辑分析:
App在启动时就要初始化tabbarVC,并且在后续会刷新底tab的数量。由于我们使用了某个tabbarVC的组件,组件并不支持动态新增底tab,这里采用的是重新创建tabbarVC的方式。
而我们的vc复用逻辑就是将vc从旧的tabbarVC移到新的tabbarVC。
这里写了一个复用的模拟代码:

- (void)testAnotherTabbarVC {
    UITabBarController *anotherTabbarVC = [UITabBarController new];
    [anotherTabbarVC addChildViewController:self.tabVC.viewControllers.firstObject];
}

复用逻辑比较简单清晰,但是UIKit有一个限制:每个vc只能有一个parentVC。当我们给新tabbarVC设置子vc,其中复用vc已经有parentVC,此时因为复用到新的tabbarVC,parentVC也会从旧的tabbarVC变成新的tabbarVC。
当旧的tabbarVC触发viewWillAppear的时候,复用vc的parentVC已经变成新的tabbarVC(截图为nil是因为新的tabbarVC被释放了),但是没被复用的另外一个vc的parentVC仍然是旧tabbarVC。
此时出现了错误:
only a view controller in the tab bar controller's list of view controllers can be selected

问题解决

方案1:在viewWillAppear之前,不触发reloadTab,也就是等待展示之后再把旧的tabbarVC替换为新的tabbarVC;(这也是之前采用的方案)
方案2:在设置新的tabbarVC的viewController属性时,将复用vc从旧的tabbarVC的viewController移除;(这是UIKit的默认做法,但是需要修改tabbarVC的组件)
方案3:不复用vc,只复用数据源;(需要修改复用方案)

代码地址

为了验证分析没有出错,特意写了demo,问题可以复现,github地址

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

推荐阅读更多精彩内容