Question

我正在使用NSOperationQueue并排队NSOperationBlocks。现在，块具有对块中任何实例的强引用，并且调用对象也具有强大的阻塞，因此建议执行以下操作：

__weak Cell *weakSelf = self;
NSBlockOperation *op = [NSBlockOperation blockOperationWithBlock:^{
        UIImage *image = /* render some image */
        /* what if by the time I get here self no longer exists? */
        [[NSOperationQueue mainQueue] addOperationWithBlock:^{
            [weakSelf setImageViewImage:image];
        }];
    }];
    [self.renderQueue addOperation:op];

所以，我的问题是，让我们说当图像完成渲染并且该行返回时，Cell对象不再存在（它已被解除分配，可能是由于单元重用，这是一个有点难以形式化）。当我访问[weakSelf setImageViewImage:]时，会导致EXC_BAD_ACCESS错误吗？

目前我正试图追查问题的原因，我认为这可能与此有关。

Answer 1

因此，__weak是归零的弱引用。这意味着在您的操作过程中，self可能确实已被释放，但所有对它的弱引用（即weakSelf）都将被清零。这意味着[weakSelf setImageViewImage:image]只是向nil发送邮件，这是安全的;或者，至少，它不应该导致EXC_BAD_ACCESS。（顺便提一下，如果您将weakSelf限定为__unsafe_unretained，则最终可能会向已释放的对象发送邮件。）

因此，我怀疑向__weak引用发送消息会导致崩溃。如果你想确保self在你的操作期间存活，你可以在块范围内得到一个强弱的参考：

__weak Cell *weakSelf = self;

NSBlockOperation *op = [NSBlockOperation blockOperationWithBlock:^{
    Cell *strongSelf = weakSelf; // object pointers are implicitly __strong
    // strongSelf will survive the duration of this operation.
    // carry on.
}];

Answer 2

如果你不使用weak，你就会创建一个保留周期，但是一旦块执行完毕，周期就会被打破。我可能不会在这里使用弱点。

无论如何，您可以向nil发送任何消息，它将被忽略。因此，如果weakSelf变量设置为nil，因为Cell对象已取消分配，setImageViewImage:消息将无声地执行任何操作。它不会崩溃。

由于您提到了单元格重用，我假设您的Cell是UITableViewCell的子类。在这种情况下，您的示例代码有一个严重的问题。 UITableViewCell通常不会被取消分配。它们被放在单元重用队列中。所以你的weakSelf变量不会被归零，因为弱参考只在实际解除分配对象时才会被归零。

当[weakSelf setImageViewImage:image]行运行时，单元格可能已被重用以表示表格中的不同行，并且您将错误的图像放入单元格中。您应该将图像呈现代码从Cell类移出到表视图的数据源类中：

- (void)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *)indexPath {
    Cell *cell = // get a cell...

    [self startLoadingImageForIndexPath:indexPath];
    return cell;
}

- (void)startLoadingImageForIndexPath:(NSIndexPath *)indexPath {
    NSBlockOperation *op = [NSBlockOperation blockOperationWithBlock:^{
        UIImage *image = [self renderImageForIndexPath:indexPath];
        dispatch_async(dispatch_get_main_queue(), ^{
            Cell *cell = [self.tableView cellForRowAtIndexPath:indexPath];
            [cell setImageViewImage:image];
        });
    }];
    [self.renderQueue addOperation:op];
}

块内的弱引用

2 个答案: