dispatch_async更新変数クラッシュ - スレッドセーフ

Question

私はラインで次のコードを複数回アプリのクラッシュを実行すると：

res.append（I）

エラーは致命的なエラーがある：負の数とUnsafeMutablePointer.destroy

をデバッグするmalloc_error_breakにブレークポイントを設定 か ポインタ解放されが割り当てられていない***はdispatch_async内のグローバル変数を更新するために、正しくないですか？

クラスのViewController：のUIViewController {

var result = Array<Int>() 


func method(completion: (inner:() throws -> String)->Void){ 
    let group:dispatch_group_t = dispatch_group_create(); 
    let queue = dispatch_get_global_queue(DISPATCH_QUEUE_PRIORITY_DEFAULT, 0) 
    var res = Array<Int>() 
    for i in 0..<4 { 
     dispatch_group_async(group,queue){ 
      res.append(i) 
      print(res) 
      print("Block\(i)");     
      var s = 0 
      for k in 0..<1000 { 
       s = 2+3 
      } 
      print("Block\(i)End"); 



     } 
    } 

    dispatch_group_wait(group, DISPATCH_TIME_FOREVER); 

    print("All background tasks are done!!"); 
    print(res) 
} 


override func viewDidLoad() { 
    super.viewDidLoad() 


    self.method() { (inner:() throws -> String) -> Void in 
     do { 
      let res = try inner() 
      print(res) 
     } catch let error { 
      print(error) 
     } 
    } 
} 

Answer 1

うん、Arrayは、スレッドセーフではないので、アレイへの書き込みは、atomicを確保する必要があるとき。

したがって、高性能ロック：dispatch_semaphore_tを追加できます。

func method(completion: (inner:() throws -> String)->Void){ 
    // add lock 
    let lock: dispatch_semaphore_t = dispatch_semaphore_create(1) 
    let group:dispatch_group_t = dispatch_group_create(); 
    let queue = dispatch_get_global_queue(DISPATCH_QUEUE_PRIORITY_DEFAULT, 0) 
    var res = Array<Int>() 
    for i in 0 ..< 5 { 
     dispatch_group_async(group,queue){ 
      // lock 
      dispatch_semaphore_wait(lock, DISPATCH_TIME_FOREVER) 
      res.append(i) 
      // unlock 
      dispatch_semaphore_signal(lock) 
      var s = 0 
      for k in 0..<1000 { 
       s = 2+3 
      } 
     } 
    } 

    dispatch_group_wait(group, DISPATCH_TIME_FOREVER); 

    print("All background tasks are done!!"); 
    print(res) 
} 

しかし、あなたの非同期タスクは、上記のような時間のかかる操作でない場合は、注意が必要スレッドスケジュールは時間がかかり、パフォーマンスの損失につながる可能性があるため、マルチスレッドを使用しないでください。