说到单例，在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地址在: kisekied/dispatch_once_lock_demo
以上是该次问题的总结，欢迎大家指教。
网友评论