Jplane 2019-06-30
做客户端开发应当时刻考虑多线程问题。我最初是做前端开发的,在这方面考虑得往往不够。谨记。
单例的常见写法其实就两种
+ (id)sharedInstance { static testClass *sharedInstance = nil; @synchronized(self) { if (!sharedInstance) { sharedInstance = [[self alloc] init]; } } return sharedInstance; }
+ (id)sharedInstance { static testClass *sharedInstance = nil; static dispatch_once_t once; dispatch_once(&once, ^{ sharedInstance = [[self alloc] init]; }); return sharedInstance; }
dispatch_once的写法更推荐一些。一方面是性能上好一点,另一方面是语义上更直观。once,执行一次嘛。
不管是用锁还是dispatch_once,本质上都是为了避免单例创建过程出现线程安全问题。
更进一步,我们经常会有懒加载某些属性的写法:
- (id<InterfaceEngineA>)engineA { if (_engineA == nil) { _engineA = [EngineA new]; } return _engineA; }
其实跟单例的实现是类似的,这种时候要格外注意线程安全问题。如果存在多线程场景,一定要做好保护
- (id<InterfaceEngineA>)engineA { @synchronized(self) { if (_engineA == nil) { _engineA = [EngineA new]; } } return _engineA; }
多线程问题的表现可能是各种各样难以预料的。这里我遇到的是,_engineA在多线程场景下小概率被重复创建,其实例1在init时注册了网络层命令字cmd1
的回包,而这个网络层框架的实现是,只接受第一个注册这一命令字的对象。导致实例2注册失败。后面调用实例2发送请求,回包都被实例1接收了。从日志上看,一切都挺正常的。但是下次取数据就是取不到。
这个bug第一次提过来的时候,没分析出根本原因,只在表面上做了保护。结果第二次提过来才真正改掉。
丢人呐。还是要好好学习才是。