Capture selbst in Block (Retain-Zyklen), nicht immer?

Question

Der folgende Code stammt aus dem von Apple bereitgestellten LazyTableImages-Beispielcode (source here).

In ihrem Komplettierungsblock haben sie einen Verweis auf self, der einen Retain-Zyklus verursachen sollte ... Aber dafür bekomme ich keine Warnung in Xcode, während ich in ähnlichem Code von mir würde.

Ist das korrekt?

Vielleicht vermisse ich eine Feinheit davon.

- (void)startIconDownload:(AppRecord *)appRecord forIndexPath:(NSIndexPath *)indexPath 
{ 
    IconDownloader *iconDownloader = [self.imageDownloadsInProgress objectForKey:indexPath]; 
    if (iconDownloader == nil) 
    { 
     iconDownloader = [[IconDownloader alloc] init]; 
     iconDownloader.appRecord = appRecord; 
     [iconDownloader setCompletionHandler:^{ 

      UITableViewCell *cell = [self.tableView cellForRowAtIndexPath:indexPath]; 

      // Display the newly loaded image 
      cell.imageView.image = appRecord.appIcon; 

      // Remove the IconDownloader from the in progress list. 
      // This will result in it being deallocated. 
      [self.imageDownloadsInProgress removeObjectForKey:indexPath]; 

     }]; 
     [self.imageDownloadsInProgress setObject:iconDownloader forKey:indexPath]; 
     [iconDownloader startDownload]; 
    } 
} 

Answer 1

Der Aufbewahrungszyklus, den Sie zu sehen glauben, liegt daran, dass das Objekt den Downloader in einem Wörterbuch enthält.

Es ist wahr, dass es im Block einen starken Verweis auf self gibt, aber solange der Vervollständigungshandler immer ausgeführt wird, wird der Downloader aus dem Wörterbuch entfernt. Und schließlich wird dieses Wörterbuch leer sein, was bedeutet, dass es keine Objekte geben wird, die sich selbst festhalten und somit keinen Retain-Zyklus.

Answer 2

self keinen starken Zeiger auf iconDownloader haben. Es erstellt und scoped gerade zu dieser Methode:

IconDownloader *iconDownloader = [self.imageDownloadsInProgress objectForKey:indexPath]; 

Wenn iconDownloader eine starke Eigenschaft war (self.iconDownloader) dann Xcode würde einen starken Referenzzyklus erkennen.

Answer 3

Capture Selbst selbst ist kein Retain-Zyklus. Es ist eine einzige Referenz. Eine Referenz kann keinen Zyklus erstellen. Das übliche Antipattern ist, dass zusätzlich eine Referenz auf den Block in einer starken Eigenschaft des Selbst gespeichert wird. Dann gibt es zwei Referenzen, die einen Zyklus aufbauen.

Answer 4

Es gibt keine Warnung, da der Compiler noch nicht alle möglichen Aufbewahrungszyklen erkennen kann.

Zum Beispiel:

- (void)foo 
{ 
    _block =^{ [self done]; }; // Warning: Possible retain cycle 

    DSGenericBlock foo =^{ [self done] }; 
    _block = foo; // No warning. 
} 

Wenn Sie den Block direkt auf eine Instanz Variable der „Selbst“ zuzuordnen sind, würden Sie die „möglich beibehalten Zyklus“ Warnung. Stattdessen wird der Block einem anderen Objekt zugewiesen, das dann von self beibehalten wird, so dass der Compiler den Zyklus nicht erkennt (obwohl der Zyklus existiert).