关于dispatch_once的坑及注意点-白红宇

说到单例，在Objective-C中我们很容易就能想到用dispatch_once来构建一个单例的对象，然而最近因为给一个目前维护的老项目增加新的功能的时候，却不小心踩到了dispatch_once的坑里面去了。简单的说明一下遇到的问题：公司测试在安装完APP一段时间后，重新进入会一直黑屏闪退，并且发生时间不确定，测试的时候也只是一个机型会发生这样的情况。当时我导出了崩溃日志，发现崩溃前是这样的（公司的项目，稍微打了一下码）：

看到上面显示trying to lock recursively，说明是被死锁了。然后我注意到最靠近崩溃的符号化信息是NBSDispatch_once，这是某第三方监测app的库的一个方法，然后我替换了最新的SDK，发现新版本的SDK已经去除了这个方法，测试发现没再出现类似的情况，于是松了一口气，PM让发布到APP Store。

本以为这件事就这样解决了，然而在我前一天晚上发布到APP Store后，有客户反馈会出现进入不了APP的问题，但是这次跟上次的又略有不同，上面那个是打开APP后一直黑屏然后闪退，而这个是在APP加载了Launch界面后出现的问题。

让我们再看一下上面的崩溃日志，有一个方法[AppViewControllerManager getAppViewControllerManager]这个方法出现多次，AppViewControllerManager是负责统管整个项目试图控制器的类，而这个方法是获取这个管理类的单例对象。在该单例方法内部的dispatch_once里面又包含了后续执行方法，在后续执行方法中有一段会再次调用[AppViewControllerManager getAppViewControllerManager]，从而造成了死锁，又因为造成死锁的方法是异步网络请求，在返回结果过快的时候，第一次的dispatch_once的block还没有执行结束的就再次进入该dispatch_once的block，导致死锁。

下面让我们构造一个类似的Demo工程进行说明，为了保证出现该死锁，我们全部用同步线程执行。（注：模拟器上死锁不会crash，真机上因为系统保护机制才会强制crash）。

编译环境：系统: macOS 10.12 Xcode版本：8.0

我们在Demo中新建一个管理类AppManager，在里面添加一个单例方法：

+ (AppManager *)getAppManager {    static AppManager *manager = nil;    static dispatch_once_t onceToken;    dispatch_once(&onceToken, ^{        manager = [[AppManager alloc] init];                ViewController *vc = [[ViewController alloc] init];        UIWindow *window = [UIApplication sharedApplication].keyWindow;        window.rootViewController = vc;        [vc doSomeThing];    });    return manager;}复制代码

在AppDelegate.m中：

- (BOOL)application:(UIApplication *)application didFinishLaunchingWithOptions:(NSDictionary *)launchOptions {    self.window = [[UIWindow alloc] initWithFrame:[UIScreen mainScreen].bounds];    [self.window makeKeyWindow];        _manager = [AppManager getAppManager];        [self.window makeKeyAndVisible];        return YES;}复制代码

我们在ViewController.m中添加方法doSomeThing:

- (void)doSomeThing {    [AppManager getAppManager];    self.view.backgroundColor = [UIColor cyanColor];}复制代码

不考虑其他因素，按步骤下来，预期屏幕背景应该是[UIColor cyanColor]；然而实际运行结果是:

我们查看运行的堆栈信息：

发现app进程停在[ViewController doSomeThing]中，并且在调用了[AppManager getAppManager]后进入信号等待，因为此时[AppManager getAppManager]中dispatch_once的block并未执行完成，处于lock状态，等待完全执行完成该block，而在该block中又进行了一次[AppManager getAppManager]的调用，从而造成死锁。

我们注释掉[ViewController doSomeThing]中的[AppManager getAppManager];后发现背景色如期变成[UIColor cyanColor]:

如何避免这类问题的发生？

1.慎用单例模式; 2.在dispacth_once包裹的block中，尽量避免与其他类的耦合。

推荐写法：

将dispatch_once中与其他类耦合的地方移出：

+ (AppManager *)getAppManager {    static AppManager *manager = nil;    static dispatch_once_t onceToken;    dispatch_once(&onceToken, ^{        manager = [[AppManager alloc] init];    });    return manager;}复制代码

放置到完全执行完初始化单例的后续方位执行：

- (BOOL)application:(UIApplication *)application didFinishLaunchingWithOptions:(NSDictionary *)launchOptions {    self.window = [[UIWindow alloc] initWithFrame:[UIScreen mainScreen].bounds];    [self.window makeKeyWindow];        _manager = [AppManager getAppManager];        ViewController *vc = [[ViewController alloc] init];    UIWindow *window = [UIApplication sharedApplication].keyWindow;    window.rootViewController = vc;    [vc doSomeThing];        [self.window makeKeyAndVisible];        return YES;}复制代码

Demo地址在: / 以上是该次问题的总结，欢迎大家指教。