performBatchUpdatesの親UIViewControllerへの強い参照がアクティビティをリークするのはなぜですか？

Question

私はちょうどを呼び出した後でもUIViewControllerがdeallocされないように、非常に厄介なUIViewControllerのデバッグを終了しました。私はperformBatchUpdatesに電話をかけると、すぐにdismissViewControllerAnimated、のUIViewControllerが漏洩しますと、そのUIViewControllerのdeallocメソッドを決して呼び出していない場合は、

self.dataSource.doNotAllowUpdates = YES; 

    [self.collectionView performBatchUpdates:^{ 
     [self.collectionView reloadItemsAtIndexPaths:@[indexPath]]; 
    } completion:^(BOOL finished) { 
     self.dataSource.doNotAllowUpdates = NO; 
    }]; 

基本的には：

は、私は次のコードブロックに問題を突き止め呼び出される。 UIViewControllerは永遠にハングアップします。

誰かがこの現象を説明できますか？私は、がある時間間隔、例えば500ミリ秒で実行されると仮定しているので、上記の間隔の後に、これらのメソッドを呼び出し、deallocをトリガすると仮定します。

修正はこのように表示されます。

self.dataSource.doNotAllowUpdates = YES; 

    __weak __typeof(self)weakSelf = self; 

    [self.collectionView performBatchUpdates:^{ 
     __strong __typeof(weakSelf)strongSelf = weakSelf; 

     if (strongSelf) { 
      [strongSelf.collectionView reloadItemsAtIndexPaths:@[indexPath]]; 
     } 
    } completion:^(BOOL finished) { 
     __strong __typeof(weakSelf)strongSelf = weakSelf; 

     if (strongSelf) { 
      strongSelf.dataSource.doNotAllowUpdates = NO; 
     } 
    }]; 

注BOOLメンバ変数、doNotAllowUpdatesつまり、私はperformBatchUpdatesへの呼び出しが実行されている間、そのデータソース/ collectionView更新のいずれかの種類を防ぐ追加の変数です。

performBatchUpdatesにweakSelf/strongSelfパターンを使用する必要があるかどうかについては、オンラインでディスカッションしましたが、特にこの質問には何も見つかりませんでした。

私はこのバグの底に到達できたことをうれしく思いますが、私は見ているこの動作を私に説明するためにスマートなiOS開発者を愛するでしょう。

Answer 1

あなたが把握したように、weakが使用されていない場合、保持サイクルが作成されます。

は、collectionViewを強く参照しており、collectionViewは、selfを強く参照しています。

非同期ブロックが実行される前に、常にselfが割り当て解除されている可能性があります。安全にこれを処理するために二つのことを実行する必要があります。

は

1. 常に
2. は常にweakSelfがnunnull のparam

としてそれを渡す前に存在することを確認selfへの弱参照（またはIVAR自体）を使用します更新日：

ロギングの少しを入れてperformBatchUpdatesは多くを確認します：

- (void)logPerformBatchUpdates { 
    [self.collectionView performBatchUpdates:^{ 
     NSLog(@"starting reload"); 
     [self.collectionView reloadItemsAtIndexPaths:[self.collectionView indexPathsForVisibleItems]]; 
     NSLog(@"finishing reload"); 
    } completion:^(BOOL finished) { 
     NSLog(@"completed"); 
    }]; 

    NSLog(@"exiting"); 
} 

プリント：

starting reload 
finishing reload 
exiting 
completed 

これは完了ブロックは、それが非同期バックメインスレッドにディスパッチされることを意味現在のスコープを出た後に焼成することを示しています。

バッチ更新を実行した後すぐにView Controllerを終了すると記載されています。私はこれがあなたの問題の根源だと思う：

いくつかのテストの後、私がメモリリークを再現できる唯一の方法は、解雇の前に仕事を派遣することでした。これは、ロングショットですが、あなたのコードは、偶然、このように見えるん？：

- (void)breakIt { 
    // dispatch causes the view controller to get dismissed before the enclosed block is executed 
    dispatch_async(dispatch_get_main_queue(), ^{ 
     [self.collectionView performBatchUpdates:^{ 
      [self.collectionView reloadItemsAtIndexPaths:[self.collectionView indexPathsForVisibleItems]]; 
     } completion:^(BOOL finished) { 
      NSLog(@"completed: %@", self); 
     }]; 
    }); 
    [self.presentationController.presentingViewController dismissViewControllerAnimated:NO completion:nil]; 
} 

上記のコードはdeallocビューコントローラ上で呼び出されていないにつながります。

既存のコードを使用して、単にdismissViewControllerコールを送信すると、問題を修正する可能性があります（またはperformSelector：after :)。

Answer 2

これはUICollectionViewのバグのようです。 APIユーザーは、タスクの実行を超えてシングルランブロックパラメータが保持されることを期待しないでください。したがって、参照サイクルを防止することは問題ではありません。

UICollectionViewは、バッチ更新プロセスが終了すると、またはバッチ更新プロセスが中断された場合（たとえば、コレクションビューが画面から削除された場合など）、ブロックへの参照をすべて消去する必要があります。

更新プロセス中にコレクションビューがオフスクリーンで表示されても完了ブロックが呼び出されるので、コレクションビューではその完了ブロックまでの参照をすべて埋める必要がありますコレクションビューの現在の状態にかかわらず、再び呼び出されることはありません。