删除 NSOperation 的观察者

Question

我有一个通过 NSOperationQueue 中的 NSOperation 加载数据的视图。我想允许用户在操作完成之前离开此视图。我的问题是我似乎无法始终如一地做到这一点而不崩溃。这是我开始操作的代码：

NSOperationQueue* tmpQueue = [[NSOperationQueue alloc] init];
self.queue = tmpQueue;
[tmpQueue release]; 
SportsLoadOperation* loadOperation = [[SportsLoadOperation alloc] init];
[loadOperation addObserver:self forKeyPath:@"isFinished" options:0 context:NULL];
[self.queue addOperation:loadOperation];
[loadOperation release];    

如果我在操作仍在执行时离开视图，我经常会收到此错误：

[SportsViewController retain]: message sent to deallocated instance 0x38b5a0

如果我尝试删除观察者以免发生这种情况，如下所示：

-(void)viewWillDisappear:(BOOL)animated {
    if (self.isLoadingData) {
        for (NSOperation *operation in [self.queue operations]) {
            if([operation isExecuting]) {
                [operation cancel];
                [operation removeObserver:self forKeyPath:@"isFinished"];
            }
        }
    }
    [super viewWillDisappear:animated];
}

然后我有时会收到此错误错误：

Terminating app due to uncaught exception 'NSRangeException', reason:
'Cannot remove an observer <SportsViewController 0x661c730> for the key path "isFinished" from <SportsLoadOperation 0x66201a0> because it is not registered as an observer.'

如何避免这些问题？

Answer 1

第二条错误消息说明了一切。

您是否尝试过在[操作取消]之后不removeObserver并看看会发生什么？

您是否尝试过先removeObserver，然后才取消操作？

这些可能有助于缩小触发错误的条件范围。此外，您可能希望将日志输出添加到代码中以查看其实际执行时间。

而且，就像自由空间的答案所说，添加 &删除观察者最好在观察实例的构造/销毁方法中完成。这通常会产生更稳定的代码。

Answer 2

看起来您有一个 SportsLoadOperation 实例，但没有 SportsViewController 作为观察者。您是否在代码中的其他位置插入了 SportsLoadOperation ？如果是这种情况，请考虑为 SportsLoadOperaion 编写一个 initWithObserver 方法来自动进行观察。这可以避免由于忘记在 isFinished 上设置观察者而导致的错误。

另外，最好在dealloc中然后在viewWillDisappear中删除观察者，因为在许多情况下都会调用viewWillDisappear，例如在显示模态视图控制器。因此，您可能会过早停止运营。

编辑

不要检查[operation isExecuting]，而是检查[operation isCancelled]。这是因为 [operation cancel] 不会停止操作 - 如果您实际上没有在 main< 中检查 isCancelled ，它可以并且将会继续执行。 /代码> 方法。这意味着，如果 viewWillDisappear 被调用两次，您最终可能会在同一 SportsLoadOperation 实例上尝试调用 removeObserver 两次，第二次尝试失败。

在以下位置添加一些调试输出语句：

1. 创建 SportsLoadOperation 实例并将其插入队列
2. 时 当您取消 SportsLoadOperation 实例并从中删除观察者时。

Answer 3

我最终通过重写观察操作的 addObserver 和 removeObserver 方法来解决这个问题，以跟踪观察者，以便我可以在 [cancel] 方法中删除它们。

我现在要做的就是调用操作队列来取消所有操作，然后再关闭正在观察操作的控制器。

下面，_observers 是一个 NSMutableDictionary。

- (void)addObserver:(NSObject*)observer
     forKeyPath:(NSString*)keyPath
        options:(NSKeyValueObservingOptions)options context:(void*)context
{
    [super addObserver:observer forKeyPath:keyPath options:options context:context];
    [_observers setValue:observer forKey:keyPath];
}


- (void)removeObserver:(NSObject*)observer forKeyPath:(NSString*)keyPath
{
    [super removeObserver:observer forKeyPath:keyPath];
    [_observers removeObjectForKey:keyPath];
}


- (void)cancel
{
    [super cancel];

    for(id key in _observers)
    {
            id object = [_observers valueForKey:key];
            [super removeObserver:object forKeyPath:key];
    }

    [_observers removeAllObjects];

}