2017-12-05 14 views
5

私はこのような珍しいネストされたシーケンスを持つコードを継承しました。通常のパラダイムでは、UIを更新するためにメインキューへの単一のディスパッチがあります。以下に示すコードは、メインキューへの別のディスパッチ内のメインキューへのディスパッチをネストします。dispatch_async(dispatch_get_main_queue()^ {})をネストする目的は何ですか?

- (void)viewDidLoad 
{ 
// Setup some data 
// Adjust UI 

dispatch_async(myBackgroundQueue, ^{ 
    while(Do_some_time_consuming_work) { 

    // Time consuming work goes here 

    if (things_are_going_slowly) { 

     dispatch_async(dispatch_get_main_queue(), ^{ // <- one of these two seems redundant 
      dispatch_async(dispatch_get_main_queue(), ^{ // <- one of these two seems redundant 

      stillWorkingLabel.hidden = NO; //Let user know the work is still ongoing 
      }); 
     }); 
    ) 

    // Finish time-consuming work 
    } 

    }); 

} 

ネストの目的は何ですか。dispatch_async(dispatch_get_main_queue()?このネストされたシーケンスは、アプリのいくつかの場所に表示されます。メイン・キューへのディスパッチは1回だけ必要です。

ここでは関連するトピックに関するすべての質問をGoogle検索で読んだと思いますが、2つの同一のディスパッチをネストすることを提案している人は見つかりませんでした。

このアプリはうまくいき、上の例とコード内の他の場所で予想通りにUIが更新されます。

アプリのコードのほとんどは、上記のスキームの通常のネストされていないバージョンを使用しています。もちろん、うまく動作します。

これらのネストされたコールを単一のディスパッチに置き換えることになりましたが、ここで何か不足している可能性があります。アドバイスをいただければ幸いです。

答えて

2

私はこれを行うことに単一の利点を考えることはできませんし、そうでない理由を考えることができます。それは、内側の閉鎖の実行を遅らせるだけでなく、少量の追加リソースを取ることになります。 (それは、アプリケーションが実行される前に、イベントループを少なくとも2回通過することをアプリケーションに強制することになります)。

私はネストされた呼び出しを削除するのが適切だと思います。どんな意味がありませんメインための2つのdispatch_queuesをネスト

0

、まっすぐ進む方法があること

dispatch_async(dispatch_get_global_queue(DISPATCH_QUEUE_PRIORITY_DEFAULT, 0), ^{ 

// perform long task here 

dispatch_async(dispatch_get_main_queue(), ^(){ 

    //Add method, task you want perform on mainQueue 
//Control UIView, IBOutlet all here 

}); }); 
関連する問題