为 NSThread.isMainThread 正言

缘起

一个线上 Crash

image.png

Crash 最终定位到 MBWebManager 一个方法中:

image.png

其中 MBWebView_dispatch_main_sync_safe 宏的定义为:

image.png

这段代码是为了兼容外部在非主线程调用,其本质目的是想确保自己在 主线程 中执行,由于该方法需要同步返回一个 web 实例,因此使用了 dispatch_sync

分析

怎么崩溃的?

通过查看几个 Crash 的堆栈,发现这个 Crash 的堆栈都是一致的业务流程(是从一个 小贷 的业务流程调用过去的),看到了「必现」的曙光~

最终重现了Crash

image.png

Crash 原因

在一个「串行」队列的「同步」任务中,提交另一个「同步」任务,导致两个任务相互等待,死锁!

翻车过程

1、调用方所处一个「自定义串行队列」,但这个队列中的任务被“绑定”到主队列同步执行:


image.png

image.png

2、由于当前队列「不是主队列」,所以逻辑最终走到了 else 里面:


image.png

3、虽然当前队列「不是主队列」,但是当前任务被“绑定”到主队列同步执行,因此,在这个同步执行任务中 提交另一个同步任务到主队列,就造成了死锁。

问题1:为什么通过 dispatch_set_target_queue()“绑定”到 主队列 的任务是「同步执行」?

按照网上的示例,做了一个实验:


image.png
image.png

只是通过实验证实了,但其内部原理,有同学了解的话,望不吝指教,万分感谢🙏

问题2:为什么要判断当前是否是主队列?

这是本文的重点!!!

分析当时编写 MBWebManager 的初心,我只是想确保任务在「主线程」中执行,那么,为什么,最后条件判断写的是「判断当前是否是主队列」呢?

真是一个灵魂拷问~

是的,我随大流了……

因为大家都这么写,这么写是「公认的更安全的判断方法」

但是,它为什么更安全呢?

为何 NSThread.isMainThread 被认为不够安全?

Goolge了很多,大致得到了以下原因:

1、在主线程中执行的任务,不一定就是在主队列中

我找到两种写法可以实现这个结论:

(1)使用 dispatch_sync()

image.png

(2)使用 dispatch_set_target_queue()

image.png

2、在主队列中执行的任务,也不一定在主线程中

问题示例:

[NSThread isMainThread] is probably not what you want!

image.png

这篇文章里的示例可能有悖常识:主队列中的任务竟然不是在主线程中执行!!!

不要激动,导致这种原因最关键的是这行代码dispatch_main() ,就是这货让主队列的任务在非主线程运行。

这个方法苹果官方文档这样解释的:

/*!
 * @function dispatch_main
 *
 * @abstract
 * Execute blocks submitted to the main queue.
 * 执行提交给主队列的任务blocks
 *
 * @discussion
 * This function "parks" the main thread and waits for blocks to be submitted
 * 
 * to the main queue. This function never returns.
 * 这个函数会阻塞主线程并且等待提交给主队列的任务blocks完成,这个函数永远不会返回
 * 
 * Applications that call NSApplicationMain() or CFRunLoopRun() on the
 * main thread do not need to call dispatch_main().
 *
 */
API_AVAILABLE(macos(10.6), ios(4.0))
DISPATCH_EXPORT DISPATCH_NOTHROW DISPATCH_NORETURN
void
dispatch_main(void);

意思是这个方法会阻塞主线程,然后在其它线程中执行主队列中的任务,这个方法永远不会返回(意思会卡住主线程)

如果去掉dispatch_main()这行代码,就会正常在主线程里执行任务

3、“据说”某些组件的某些方法,必须在 主队列 中执行

问题示例:

24025596: MapKit: It’s not safe to call MKMapView’s addOverlay on a non-main-queue, even if it is executed on the main thread #7053

这个文章里面说的好像靠谱一些,但是我做了相同的实验,结果仍然与之不吻合:


image.png

网络上说「不在主队列中执行就会 crash 」的,除了上面的 MapKit 中的 addOverlay,还有一些,例如:VektorKit(原谅我的孤陋寡闻,没用过,所有就没有再继续实验),但我发现,这些文章大多都是 15年 或者 16年 发表的,所以,我猜测,这些组件后来修复了这个「BUG」了吧(是的,我更愿意把它理解成一个 BUG,因为如果组件功能设计某个 API 只能在 主队列 中执行,就应该以明确的文档告知外部使用者,如果没有文档说明,但在 非主队列 中执行就 crash,我想,更多的过错应该是这个组件的开发者吧,即使这个开发者是苹果内部人士,哈哈哈哈哈哈)~

真相

所以,通过上面的实验,和分析,其实并不是 NSThread.isMainThread 不安全,而是掉进了某些组件的 BUG 里面~

这篇文章说得还是比较详细:GCD’s Main Queue vs. Main Thread

心法

1、网络上的文章,不可尽信,还是要时刻保有一颗敢于质疑和求证的心态;

2、明确自己的初心,就像我这个例子里,就需要明确:到底是需要 主线程 还是 主队列?

知识点

1、多个 队列,通过 dispatch_set_target_queue()"绑定"到 一个 target queue 后,其中的任务是「同步执行」的;

2、通过 dispatch_sync() 执行的任务,会在「当前线程」中执行(如果当前线程是主线程,即便 当前队列 不是 主队列,任务也会在 主线程中执行);

3、使用 dispatch_sync 时,要非常小心,很容易掉进死锁问题中(这个只能具体问题,具体分析了)。

补充

回到这个 crash 问题,其解决方案就是:

将:

if (strcmp(dispatch_queue_get_label(DISPATCH_CURRENT_QUEUE_LABEL), dispatch_queue_get_label(dispatch_get_main_queue())) == 0) {
        block();
    } else {
        dispatch_sync(dispatch_get_main_queue(), ^{
            block();
        });
    }

改成:

if ([NSThread isMainThread]) {
        block();
    } else {
        dispatch_sync(dispatch_get_main_queue(), ^{
            block();
        });
    }

有同学说,可以改成下面这样,但前面第一个条件是冗余的,因为「主队列中的任务,一般情况下都是在 主线程 中执行」

if (strcmp(dispatch_queue_get_label(DISPATCH_CURRENT_QUEUE_LABEL), dispatch_queue_get_label(dispatch_get_main_queue())) == 0 
    || [NSThread isMainThread]) {
        block();
    } else {
        dispatch_sync(dispatch_get_main_queue(), ^{
            block();
        });
    }

谢谢阅读,如有纰漏,望不吝指教🙏

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

推荐阅读更多精彩内容