ARCが有効なコードで「このブロックで[オブジェクト]を強くキャプチャすると保持サイクルが発生する可能性が高い」という警告を修正

ARCが有効なコードで、ブロックベースのAPIを使用しているときに、保持サイクルの可能性に関する警告を修正するにはどうすればよいですか？

警告：
Capturing 'request' strongly in this block is likely to lead to a retain cycle

このコードスニペットによって生成されます。

ASIHTTPRequest *request = [[ASIHTTPRequest alloc] initWithURL:...

[request setCompletionBlock:^{
    NSDictionary *jsonDictionary = [[CJSONDeserializer deserializer] deserialize:request.rawResponseData error:nil];
    // ...
    }];

警告はrequest、ブロック内のオブジェクトの使用に関連しています。

自分に返信：

ドキュメンテーションについての私の理解では、キーワードblockを使用し、ブロック内で使用した後に変数をnilに設定することは問題ないはずですが、それでも警告が表示されます。

__block ASIHTTPRequest *request = [[ASIHTTPRequest alloc] initWithURL:...

[request setCompletionBlock:^{
    NSDictionary *jsonDictionary = [[CJSONDeserializer deserializer] deserialize:request.responseData error:nil];
    request = nil;
// ....

    }];

更新：「_ block」の代わりに「 _weak」というキーワードを使用し、一時変数を使用するようにしました。

ASIHTTPRequest *_request = [[ASIHTTPRequest alloc] initWithURL:...
__weak ASIHTTPRequest *request = _request;

[request setCompletionBlock:^{
    NSDictionary *jsonDictionary = [[CJSONDeserializer deserializer] deserialize:request.responseData error:nil];
    // ...
    }];

iOS 4もターゲットにする場合は、の__unsafe_unretained代わりにを使用します__weak。同じ動作ですが、ポインターはオブジェクトが破棄されたときに自動的にnilに設定されるのではなく、ぶら下がったままになります。

この問題は、リクエストへの強い参照を持つリクエストにブロックを割り当てているために発生します。ブロックは自動的に要求を保持するため、元の要求はサイクルのために割り当て解除されません。理にかなっていますか？

リクエストオブジェクトに__blockのタグを付け、それ自体を参照できるようにするので、それは奇妙です。これと一緒に弱参照を作成することで、これを修正できます。

ASIHTTPRequest *request = [[ASIHTTPRequest alloc] initWithURL:...];
__weak ASIHTTPRequest *wrequest = request;

[request setCompletionBlock:^{
    NSDictionary *jsonDictionary = [[CJSONDeserializer deserializer] deserialize:wrequest.rawResponseData error:nil];
    // ...
    }];

それはブロック内に自己を保持するために発生します。ブロックは自己からアクセスされ、自己はブロックで参照されます。これにより、保持サイクルが作成されます。

の弱い参照を作成してこれを解決してみてください self

__weak typeof(self) weakSelf = self;

operationManager = [[AFHTTPRequestOperation alloc] initWithRequest:request];
operationManager.responseSerializer = [AFJSONResponseSerializer serializer];
[operationManager setCompletionBlockWithSuccess:^(AFHTTPRequestOperation *operation, id responseObject) {

    [weakSelf requestFinishWithSucessResponseObject:responseObject withAFHTTPRequestOperation:operation andRequestType:eRequestType];

} failure:^(AFHTTPRequestOperation *operation, NSError *error) {
    [weakSelf requestFinishWithFailureResponseObject:error withAFHTTPRequestOperation:operation andRequestType:eRequestType];
}];
[operationManager start];

場合によっては、xcodeコンパイラーが識別子の保持サイクルに問題を抱えているため、completementBlockを保持していないことが確実な場合は、次のようにコンパイラーフラグを設定できます。

#pragma clang diagnostic push
#pragma clang diagnostic ignored "-Warc-retain-cycles"
#pragma clang diagnostic ignored "-Wgnu"

-(void)someMethod {
}

Guillaumeが提供するソリューションを試すと、デバッグモードでは問題ありませんが、リリースモードではクラッシュします。

ターゲットはiOS 4.3であるため、__ weakではなく__unsafe_unretainedを使用しないでください。

オブジェクト「リクエスト」でsetCompletionBlock：が呼び出されるとコードがクラッシュします：リクエストの割り当てが解除されました...

したがって、このソリューションはデバッグモードとリリースモードの両方で機能します。

// Avoiding retain cycle :
// - ASIHttpRequest object is a strong property (crashs if local variable)
// - use of an __unsafe_unretained pointer towards self inside block code

self.request = [ASIHttpRequest initWithURL:...
__unsafe_unretained DataModel * dataModel = self;

[self.request setCompletionBlock:^
{
    [dataModel processResponseWithData:dataModel.request.receivedData];        
}];

ASIHTTPRequest *request = [[ASIHTTPRequest alloc] initWithURL:...
__block ASIHTTPRequest *blockRequest = request;
[request setCompletionBlock:^{
    NSDictionary *jsonDictionary = [[CJSONDeserializer deserializer] deserialize:blockRequest.responseData error:nil];
    blockRequest = nil;
// ....

}];

__weak参照と__block参照の違いは何ですか？

Apple開発者向けWebサイトのドキュメントをご覧ください。https：//developer.apple.com/library/prerelease/ios/#documentation/General/Conceptual/ARCProgrammingGuide/Introduction.html#//apple_ref/doc/uid/TP40011029

ページの下部に保持サイクルに関するセクションがあります。