Immer schwache Referenz des Selbst in Block in ARC übergeben?

Ich bin etwas verwirrt über die Blockverwendung in Objective-C. Ich verwende derzeit ARC und habe ziemlich viele Blöcke in meiner App, die sich derzeit immer auf selfdie schwache Referenz beziehen. selfKann dies die Ursache dafür sein, dass diese Blöcke erhalten bleiben und nicht freigegeben werden? Die Frage ist, sollte ich immer eine weakReferenz von selfin einem Block verwenden?

-(void)handleNewerData:(NSArray *)arr
{
    ProcessOperation *operation =
    [[ProcessOperation alloc] initWithDataToProcess:arr
                                         completion:^(NSMutableArray *rows) {
        dispatch_async(dispatch_get_main_queue(), ^{
            [self updateFeed:arr rows:rows];
        });
    }];
    [dataProcessQueue addOperation:operation];
}

ProcessOperation.h

@interface ProcessOperation : NSOperation
{
    NSMutableArray *dataArr;
    NSMutableArray *rowHeightsArr;
    void (^callback)(NSMutableArray *rows);
}

ProcessOperation.m

-(id)initWithDataToProcess:(NSArray *)data completion:(void (^)(NSMutableArray *rows))cb{

    if(self =[super init]){
        dataArr = [NSMutableArray arrayWithArray:data];
        rowHeightsArr = [NSMutableArray new];
        callback = cb;
    }
    return self;
}

- (void)main {
    @autoreleasepool {
        ...
        callback(rowHeightsArr);
    }
}

Es hilft, sich nicht auf die strongoder einen weakTeil der Diskussion zu konzentrieren. Konzentrieren Sie sich stattdessen auf den Fahrradteil .

A beibehält Zyklus eine Schleife ist, was geschieht , wenn das Objekt A Objekt B behält, und Objekt B behält Objekt A. In dieser Situation , wenn ein Objekt freigegeben wird:

• Objekt A wird nicht freigegeben, da Objekt B einen Verweis darauf enthält.
• Objekt B wird jedoch niemals freigegeben, solange Objekt A einen Verweis darauf hat.
• Objekt A wird jedoch niemals freigegeben, da Objekt B einen Verweis darauf enthält.
• Ad infinitum

Somit bleiben diese beiden Objekte während der gesamten Laufzeit des Programms im Speicher, obwohl sie freigegeben werden sollten, wenn alles ordnungsgemäß funktioniert.

Wir machen uns also Sorgen, Zyklen beizubehalten , und es gibt nichts an Blöcken an und für sich, die diese Zyklen erzeugen. Dies ist zum Beispiel kein Problem:

[myArray enumerateObjectsUsingBlock:^(id obj, NSUInteger idx, BOOL *stop){
   [self doSomethingWithObject:obj];
}];

Der Block behält den Block bei, behält ihn selfjedoch selfnicht bei. Wenn der eine oder andere freigegeben wird, wird kein Zyklus erstellt und alles wird freigegeben, wie es sollte.

Wo Sie in Schwierigkeiten geraten, ist so etwas wie:

//In the interface:
@property (strong) void(^myBlock)(id obj, NSUInteger idx, BOOL *stop);

//In the implementation:
[self setMyBlock:^(id obj, NSUInteger idx, BOOL *stop) {
  [self doSomethingWithObj:obj];     
}];

Jetzt hat Ihr Objekt ( self) einen expliziten strongVerweis auf den Block. Und der Block hat einen impliziten starken Bezug zu self. Das ist ein Zyklus, und jetzt wird keines der Objekte ordnungsgemäß freigegeben.

Da in einer solchen Situation self per Definition bereits ein strongVerweis auf den Block vorhanden ist, ist es normalerweise am einfachsten, einen explizit schwachen Verweis auf selfden zu verwendenden Block zu erstellen:

__weak MyObject *weakSelf = self;
[self setMyBlock:^(id obj, NSUInteger idx, BOOL *stop) {
  [weakSelf doSomethingWithObj:obj];     
}];

Dies sollte jedoch nicht das Standardmuster sein, dem Sie beim Umgang mit aufrufenden Blöcken folgenself ! Dies sollte nur verwendet werden, um einen Haltezyklus zwischen self und dem Block zu unterbrechen. Wenn Sie dieses Muster überall anwenden würden, laufen Sie Gefahr, einen Block an etwas zu übergeben, das ausgeführt wurde, nachdem selfdie Zuordnung aufgehoben wurde.

//SUSPICIOUS EXAMPLE:
__weak MyObject *weakSelf = self;
[[SomeOtherObject alloc] initWithCompletion:^{
  //By the time this gets called, "weakSelf" might be nil because it's not retained!
  [weakSelf doSomething];
}];

Sie müssen nicht immer eine schwache Referenz verwenden. Wenn Ihr Block nicht beibehalten, sondern ausgeführt und dann verworfen wird, können Sie sich selbst stark erfassen, da dadurch kein Aufbewahrungszyklus erstellt wird. In einigen Fällen möchten Sie sogar, dass der Block das Selbst bis zur Fertigstellung des Blocks hält, damit die Zuordnung nicht vorzeitig aufgehoben wird. Wenn Sie den Block jedoch stark und innerhalb von Capture Self erfassen, wird ein Aufbewahrungszyklus erstellt.

Ich stimme @jemmons voll und ganz zu:

Dies sollte jedoch nicht das Standardmuster sein, dem Sie beim Umgang mit Blöcken folgen, die sich selbst aufrufen! Dies sollte nur verwendet werden, um einen Haltezyklus zwischen self und dem Block zu unterbrechen. Wenn Sie dieses Muster überall anwenden würden, laufen Sie Gefahr, einen Block an etwas zu übergeben, das ausgeführt wurde, nachdem die Zuordnung von self aufgehoben wurde.

//SUSPICIOUS EXAMPLE:
__weak MyObject *weakSelf = self;
[[SomeOtherObject alloc] initWithCompletion:^{
  //By the time this gets called, "weakSelf" might be nil because it's not  retained!
  [weakSelf doSomething];
}];

Um dieses Problem zu lösen, kann man eine starke Referenz über das weakSelfInnere des Blocks definieren:

__weak MyObject *weakSelf = self;
[[SomeOtherObject alloc] initWithCompletion:^{
  MyObject *strongSelf = weakSelf;
  [strongSelf doSomething];
}];

Wie Leo betont, würde der Code, den Sie Ihrer Frage hinzugefügt haben, keinen starken Referenzzyklus vorschlagen (auch bekannt als Zyklus beibehalten). Ein betriebsbedingtes Problem, das einen starken Referenzzyklus verursachen könnte, wäre, wenn die Operation nicht freigegeben wird. Ihr Code-Snippet weist zwar darauf hin, dass Sie Ihren Vorgang nicht als gleichzeitig definiert haben, aber wenn Sie dies getan haben, wird er nicht freigegeben, wenn Sie nie gepostet isFinishedhaben oder wenn Sie zirkuläre Abhängigkeiten oder ähnliches hatten. Und wenn der Vorgang nicht freigegeben wird, wird auch der View Controller nicht freigegeben. Ich würde vorschlagen, einen Haltepunkt oder NSLogdie deallocMethode Ihres Vorgangs hinzuzufügen und zu bestätigen, dass dieser aufgerufen wird.

Du sagtest:

Ich verstehe den Begriff der Beibehaltungszyklen, bin mir aber nicht ganz sicher, was in Blöcken passiert, was mich ein wenig verwirrt

Die Probleme mit dem Aufbewahrungszyklus (starker Referenzzyklus), die bei Blöcken auftreten, sind genau wie die Probleme mit dem Aufbewahrungszyklus, mit denen Sie vertraut sind. Ein Block behält starke Verweise auf Objekte bei, die innerhalb des Blocks angezeigt werden, und gibt diese starken Verweise erst frei, wenn der Block selbst freigegeben wird. Wenn also Blockreferenzen selfoder sogar nur eine Instanzvariable von referenzieren self, die einen starken Bezug zu self beibehält, wird dies erst aufgelöst, wenn der Block freigegeben wird (oder in diesem Fall, bis die NSOperationUnterklasse freigegeben wird).

Weitere Informationen finden Sie im Abschnitt Vermeiden starker Referenzzyklen beim Erfassen von Selbst im Dokument Programmieren mit Objective-C: Arbeiten mit Blöcken .

Wenn Ihr View Controller immer noch nicht freigegeben wird, müssen Sie lediglich feststellen, wo sich die ungelöste starke Referenz befindet (vorausgesetzt, Sie haben bestätigt, dass die Zuordnung NSOperationaufgehoben wird). Ein häufiges Beispiel ist die Verwendung einer Wiederholung NSTimer. Oder ein benutzerdefiniertes delegateoder anderes Objekt, das fälschlicherweise eine strongReferenz verwaltet. Sie können häufig Instrumente verwenden, um festzustellen, wo Objekte ihre starken Referenzen erhalten, z.

Oder in Xcode 5:

Einige Erklärungen ignorieren eine Bedingung bezüglich des Aufbewahrungszyklus. [Wenn eine Gruppe von Objekten durch einen Kreis starker Beziehungen verbunden ist, halten sie sich gegenseitig am Leben, auch wenn es keine starken Referenzen von außerhalb der Gruppe gibt.] Weitere Informationen finden Sie im Dokument

So können Sie das Selbst innerhalb des Blocks verwenden:

// Aufruf des Blocks

 NSString *returnedText= checkIfOutsideMethodIsCalled(self);

NSString* (^checkIfOutsideMethodIsCalled)(*)=^NSString*(id obj)
{
             [obj MethodNameYouWantToCall]; // this is how it will call the object 
            return @"Called";


};