self.completionBlock = ^{} 和 (void)(^completionBlock)(void) = ^{} 之间的区别

Question

最近在关注 Apple 文档后

我使用以下约定来避免保留周期问题。

__weak __typeof(self) weak_self = self;
void(^completionBlock)(void) = ^(){

    __typeof(self) strong_self = weak_self;
    if (strong_self) {

        if (strong_self->_completion != NULL) {
            strong_self->_completion();
        }
    }
};

但是发现这段代码崩溃了，因为 self 在调用块之前被释放了。

当我使用下面的方法时发现它是有效的。

__block __typeof(self) block_self = self;
void(^completionBlock)(void) = ^(){

     if (block_self->_completion != NULL) {
         block_self->_completion();
     }
};

现在我很困惑什么时候应该使用 __weak 引用。只有在“self.completionBlock”

的情况下

__weak __typeof(self) weak_self = self;
self.completionBlock = ^(){

     if (weak_self->_completion != NULL) {
         weak_self->_completion();
     }
};

关于此条件的任何说明对我都非常有用。

我的实现代码如下。

=========================================== ====

文件 MyViewController

@implementation MyViewController

//starting point 
- (void)myPushMethod {
    __weak __typeof(self) weak_self = self;
    MyViewControllerTransitioning *delegateObj = [[MyViewControllerTransitioning alloc] initWithCompletion:^{

        //resetting the delegate
        __typeof(self) strong_self = weak_self;
        if (strong_self) {
            strong_self.navigationController.delegate = nil;
        }
    }];
    self.navigationController.delegate = delegateObj;
    [self.navigationController pushViewController:someViewController animated:_animated];
    //it is found that delegateObj is getting deallocated while reaches this point
}

@end

=========================================== ====

文件 MyViewControllerTransitioning

@implementation MyViewControllerTransitioning

- (instancetype)initWithCompletion:(completionHandler)completionHandler
{
    if(self = [super init])
    {
        if (completionHandler != NULL) {
            _completion  = completionHandler;
        }
    }
    return self;
}

- (id <UIViewControllerAnimatedTransitioning>)navigationController:(UINavigationController *)navigationController
                                   animationControllerForOperation:(UINavigationControllerOperation)operation
                                                fromViewController:(UIViewController *)fromVC
                                                  toViewController:(UIViewController *)toVC
{
    id<UIViewControllerAnimatedTransitioning> animationTransitioning = nil;

    //crashes here
    __block __typeof(self) block_self = self;
    void(^completionBlock)(void) = ^(){

        if (block_self->_completion != NULL) {
            block_self->_completion();
        }
    };

    //showing presentation-up animation if Push
    if (operation == UINavigationControllerOperationPush) {
        animationTransitioning = [[MyAnimator alloc] initWithAnimationCompletion:completionBlock];
    }
    return animationTransitioning;
}

- (void)dealloc{
//dealloc is called before invoking completionBlock
}

@end

=========================================== ====

文件 MyAnimator

@implementation MyAnimator

- (instancetype)initWithAnimationCompletion:(presentationUpCompletion)completionHandler
{
    if(self = [super init])
    {
        if (completionHandler != NULL) {
            _completion  = completionHandler;
        }
    }
    return self;
}

- (NSTimeInterval)transitionDuration:(id<UIViewControllerContextTransitioning>)transitionContext
{
    return my_duration;
}

- (void)animateTransition:(id<UIViewControllerContextTransitioning>)transitionContext {

    //animation logic

    [UIView animateWithDuration:duration animations: ^{
        //animation logic
    } completion: ^(BOOL finished) {
        [transitionContext completeTransition:![transitionContext transitionWasCancelled]];
    }];
}

- (void)animationEnded:(BOOL) transitionCompleted {
    if (transitionCompleted && _completion != NULL) {
        _completion();
    }
}

@end

Answer 1

self.completionBlock

self 对 completionBlock 有强引用（它是一个属性变量），那么你需要对 self 进行弱引用以避免循环。

(void)(^completionBlock)(void) = ^{}

self 没有对 completionBlock 的强引用（它是一个局部变量），不需要创建对自身的弱引用。

而您添加的 __block 起作用的原因是“__block 导致变量被强引用”。

Answer 2

在下面的原始回答中，我介绍了标准 weakSelf 模式（以及 weakSelf-strongSelf "dance"）。我的结论是，您引用的演示幻灯片关于 weakSelf 模式是绝对正确的（尽管该演示文稿在风格上已经过时）。

您随后提供了一个更完整的代码示例，结果发现它遇到了一个不同的、不相关的问题。（更糟糕的是，这个问题只有在解决了强引用循环时才会出现。大声笑。）最重要的是，代码将导航控制器的委托设置为超出范围的本地对象。因为导航控制器不保留其委托，所以您最终会得到一个指向此已释放对象的悬空指针。

如果您保留自己对该委托对象的 strong 引用（防止它被释放），问题就会消失。

下面是我原来的回答。

你说：

used the following conventions to avoid retain cycle issues.
__weak __typeof(self) weak_self = self;

void(^completionBlock)(void) = ^(){
    __typeof(self) strong_self = weak_self;
    if (strong_self) {
        if (strong_self->_completion != NULL) {
            strong_self->_completion();
        }
    }
};
But this code is found to be crashing because self is getting deallocated before invoking the block.

不，这是一个很常见的模式（通常被戏称为 "weak self, strong self dance"）。这没有错。如果它崩溃了，那是因为其他原因。

当然，我会使用现代命名约定（weakSelf 和 strongSelf 而不是 weak_self 和 strong_self）。我会删除 __typeof 开头的 __。而且我不会倾向于取消引用和 ivar，而是使用属性。所以，我可能会得到类似这样的结果：

__weak typeof(self) weakSelf = self;

void(^completionBlock)(void) = ^(){
    typeof(self) strongSelf = weakSelf;
    if (strongSelf) {
        if (strongSelf.completion != NULL) {
            strongSelf.completion();
        }
    }
};

但是，如果它崩溃了，那么您还有其他问题。坦率地说，您有一个调用块的块，因此很难猜测您的问题出在哪里，但问题不在 "weak self" 模式中。

稍后您继续建议使用：

__block __typeof(self) block_self = self;

这与您认为的不同。 "weak self" 模式的目标是打破强引用循环（以前称为保留循环），但在 ARC 中，这个 __block 引用对解决强引用循环没有任何作用。请注意，在非 ARC 代码中，此 block_self 模式用于中断保留循环，但它不会在 ARC 中完成这项工作。

最后，您继续建议：

__weak __typeof(self) weak_self = self;
self.completionBlock = ^(){
     if (weak_self->_completion != NULL) {
         weak_self->_completion();
     }
};

这有两个严重的问题。首先，您正在取消引用一个弱变量。如果 weak_self 是 nil，它就会崩溃。其次，即使您通过使用属性访问器方法而不是取消引用 ivars 来解决这个问题，您也会遇到另一个问题，即竞争条件。

最重要的是，该演示文稿中的 "weak self" 模式是正确的。同样，第一个示例中的 "weak self, strong self dance" 也是正确的。如果您遇到崩溃，您必须向我们提供重现该问题的 MCVE。但在 ARC 代码中，这是一个完全有效的模式。

self.completionBlock = ^{} 和 (void)(^completionBlock)(void) = ^{} 之间的区别

Difference between self.completionBlock = ^{} and (void)(^completionBlock)(void) = ^{}

weak-references

ios

objective-c-blocks

retain-cycle

文件 MyViewController

文件 MyViewControllerTransitioning

文件 MyAnimator