我有一个棘手的 block 内存管理问题，我试图确保我理解。我正在开发一个可以播放视频的应用程序，但需要先检查用户是否真的被允许播放它。有几个步骤需要验证，其中一些需要用户交互，所以我有一大块代码看起来像这样:

MyVideoPlayer *videoPlayer = [[[MyVideoPlayer alloc] init] autorelease];
[videoPlayer canPlayAsset:(MyVideoAsset *)asset completionBlock:^(BOOL isAssetPlayable) {
    if (isAssetPlayable) {
        [videoPlayer playAsset:asset];
        [self presentModalViewController:videoPlayer animated:YES];
    }
}];

此方法可以立即返回，也可以需要一些用户输入和一些网络调用，因此是实际呈现玩家的 block 。我注意到一些奇怪的行为，我发现视频播放器被泄露了。这是我认为正在发生的事情:

1. videoPlayer 是自动释放的。
2. videoPlayer 由 block 保留。
3. 该 block 执行，要么显示 videoPlayer，要么不显示。
4. block 被释放，并释放videoPlayer。
5. videoPlayer 被解除分配(立即，如果它没有出现，或者当模态视图被关闭时)。

相反，发生的事情是这样的:

1. videoPlayer 是自动释放的。
2. videoPlayer 由 block 保留。
3. 该 block 执行，要么显示 videoPlayer，要么不显示。
4. ????

现在，我注意到如果按如下方式修改 block ，我能够获得预期的行为:

MyVideoPlayer *videoPlayer = [[[MyVideoPlayer alloc] init] autorelease];
[videoPlayer canPlayAsset:(MyVideoAsset *)asset completionBlock:^(BOOL isAssetPlayable) {
    if (isAssetPlayable) {
        [videoPlayer playAsset:asset];
        [self presentModalViewController:videoPlayer animated:YES];
    }
    [videoPlayer autorelease];
}];

但我真的不想在不知道自己在做什么的情况下添加它。

我的理解是，这不是保留循环，因为videoPlayer 既不保留也不复制该 block 。我的理解是当 block 不再在范围内时将被释放是不正确的吗？谁能帮我理解正确的方法来做到这一点？

更新

只是一些更多的信息，MyVideoPlayer 的 canPlayAsset:completionBlock: 实现(为了保护无辜者删除了细节)看起来有点像这样:

- (void)canPlayAsset:(MyVideoAsset *)asset completionBlock:(void(^)(BOOL isAssetPlayable))completion {
    if (!asset) {
        completion(NO);
        return;
    }
    // user is a singleton object
    if (user.isGuest) {
        if ([user allowedRating:asset.rating]) {
            completion(YES);
        } else {
            // show alert
            completion(NO);
        }
    } else {
        if ([user allowedRating:asset.rating]) {
            completion(YES);
        } else {
            // prompt for pin
        }
    }
}

如您所见，我绝不会保留这个该死的 block 。 (让我们忽略 pin 部分，因为我担心它会使事情进一步复杂化。即使那部分代码不执行，问题仍然会出现。)如果 block 是一个自动释放的对象，为什么它不被释放这个函数什么时候结束执行？

更新 2

好的，我找到了。问题根本不在这个街区。正在泄露的视频播放器中有一个网络请求，其中包含一个引用了自身的 block 。

这是动画片的一部分“我今天学到了宝贵的一课......”

最佳答案

只需将第一行更改为:

__block MyVideoPlayer *videoPlayer = [[[MyVideoPlayer alloc] init] autorelease];

这样做的原因是因为:

In a reference-counted environment, by default when you reference an Objective-C object within a block, it is retained. This is true even if you simply reference an instance variable of the object. Object variables marked with the __block storage type modifier, however, are not retained.

请参阅有关 Object and Block Variables 的 Apple 文档

但为什么不一开始就发布呢？

简单的解释就是completionBlock执行后，并没有被释放。它稍后在 dealloc 中被释放，因此它保留的所有变量仍然保留。这是有道理的，因为可以根据需要多次执行 block ，直到它被释放。

我以前见过这种行为，在 block 中引用拥有 block 的对象并在 dealloc 中释放 block ，这会阻止对象被释放。 解决方案是弱引用拥有该 block 的对象。这样，拥有类型(如 MyVideoPlayer)到达 dealloc，随后释放 block (在此示例中为 completionBlock)。

使用 __block 关键字来避免保留类型的替代方法是将对象包装在 NSValue 中。通过使用 valueWithNonretainedObject:和 nonretainedObjectValue方法。例如:

MyVideoPlayer *videoPlayer = [[[MyVideoPlayer alloc] init] autorelease];
NSValue *videoPlayerRef = [NSValue valueWithNonretainedObject:videoPlayer];
[videoPlayer canPlayAsset:(MyVideoAsset *)asset 
          completionBlock:^(BOOL isAssetPlayable) {
              if (isAssetPlayable) {
                  MyVideoPlayer *tempVideoPlayer = 
                      (MyVideoPlayer*)[videoPlayerRef nonretainedObjectValue];
                  [tempVideoPlayer playAsset:asset];
                  [self presentModalViewController:tempVideoPlayer animated:YES];
              }
}];

编辑/对话

这部分将进一步解释为什么区 block 没有被释放。我将包括我假设 MyVideoPlayer 背后发生的事情。很高兴看到实际代码，但这应该足够了。

头文件...

typedef void(^MyVideoPlayerCompletionBlock)(BOOL isAssetPlayable);

@interface MyVideoPlayer : NSObject
@property(nonatomic, copy) MyVideoPlayerCompletionBlock completion;
... // other property definitions (or ivars)    

-(void)canPlayAsset:(MyVideoAsset*)asset 
    completionBlock:(MyVideoPlayerCompletionBlock)completion;
... // other methods defined
@end

实现...

@implementation MyVideoPlayer

@synthesize completion = _completion;

-(void)dealloc {
    // Note: Block is released in dealloc like any other property or variable
    [_completion release], _completion = nil;
    ...  // other variables and properties not shown here are released
    [super dealloc];
}

-(void)canPlayAsset:(MyVideoAsset*)asset 
    completionBlock:(MyVideoPlayerCompletionBlock)completion {
    ... // Does something with asset.  Plays it, stores it, whatever

    // Saves completion block to call at a later time.
    // Note that this code could alternatively look like
    // _completion = [completion copy];
    // Blocks are usually copied and not retained
    self.completion = completion;
}

...在某些时候，完成 block 会启动... (可能在事件处理程序或某种形式中)

-(void)SomeEventHandlerOrFuncThatCallsCompletionInMyVideoPlayer {
    // Time to call completion!
    if (self.completion) {
        self.completion(YES);  // OR no, doesn't matter
        //
        // WHOA! :: self.completion is not released
        //
        //     i.e. self.completion is not nil, and all
        //          variables inside it are still retained
        //          because calling a block doesn't also
        //          release the block.  To do that you would
        //          need to do:     self.completion = nil;
        //          AFTER calling:  self.completion(...);
        //
        //    SO...
        //
        //          Because the block was not released, it is
        //          still retaining variables (such as the current
        //          instance of MyViewPlayer).  Note that the
        //          Block will never be released until dealloc
        //          is called.  :(  So..., if you want the block
        //          to retain the current MyViewPlayer until
        //          execution of the completion block is over, you
        //          will want to call:  self.completion = nil;
        //          to release the block and all associated variables
        //          after calling the completion block.
    }
}

@end

关于ios - 为什么这个 Objective-C Block 会泄漏？，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/9490905/