多线程安全问题
2018-05-23 本文已影响78人
conowen
经典问题
@interface ViewController ()
@property (strong,nonatomic) NSString *str;
@end
@implementation ViewController
- (void)viewDidLoad {
[super viewDidLoad];
dispatch_queue_t queue_t_concurrent = dispatch_queue_create("com.test.concurrent", DISPATCH_QUEUE_CONCURRENT);
for (int i= 0; i < 1000000; i++) {
dispatch_async(queue_t_concurrent, ^{
self.str = [NSString stringWithFormat:@"str =%d",i];
}
});
}
}
因为是异步并行队列,而且str属性是非原子性的,系统会开启多条线程来执行Block代码,运行的时候,几率出现crash现象。
原因:因为对self.str进行重新赋值的时候,会调用内部的setter方法的release
方法,如果线程A调用release
方法的时候,对象已经释放了,如果线程B在这个时刻调用了release
方法,就出现crash现象。(访问一个已经释放的对象,会出现野指针问题EXC_BAD_ACCESS)
MRC的Setter与getter方法
//非原子性
- (void)setStr:(NSString *)str {
[str retain];
[_str release];
_str = str;
}
- (NSString *)str {
return _str;
}
//原子性
- (NSString *) str {
NSString *str = nil;
@synchronized(self) {
str = [[_str retain] autorelease];
}
return str;
}
- (void) setStr:(UITextField *)str {
@synchronized(self) {
[str retain];
[_str release];
_str = str;
}
}
就算上述的代码运行在ARC环境下,一样会导致crash现象,因为ARC的retain和release是由编辑器在编译的时候自动加上去的,最后一样会有retain和release方法。
解决方法
把self.str属性改成原子性的,或者给Block加锁,或者self.str属性改成weak属性。