enumerateObjectsWithOptions:usingBlock：导致频繁的无法解释的冻结

Question

我尝试使用 enumerateObjectsWithOptions:usingBlock 方法枚举数组。然而，我的代码很少能工作。当它不起作用时，我的应用程序会冻结（但没有沙滩球）——我对块相对较新，所以我一定缺少块操作顺利工作所需的东西。

注意：我知道我可以使用 for 循环进行枚举，但这不是我想要的。

麻烦的代码：

   NSArray*_storageCookies = [[NSArray alloc] initWithArray:[storage cookies]];

    NSArray*_historyObjects = [[NSArray alloc] initWithArray:[_history webkitHistory]];

    NSOperationQueue*_queue = [[NSOperationQueue alloc] init];

    NSBlockOperation*_block = [NSBlockOperation blockOperationWithBlock:^{

        NSAutoreleasePool*_pool = [[NSAutoreleasePool alloc] init];

        [_storageCookies enumerateObjectsWithOptions:NSEnumerationConcurrent usingBlock:^(id ck, NSUInteger index, BOOL *stop) 
        {            
            NSString*domain = [ck domain];

            [_historyObjects enumerateObjectsWithOptions:NSEnumerationConcurrent usingBlock:^(id object, NSUInteger aindex, BOOL *stop) 
            {
               @synchronized(self)
               {
                NSAutoreleasePool*_pool2 = [[NSAutoreleasePool alloc] init];

                NSString*_historyURL = [[NSURL URLWithString:[object url]] host];

                if ([_historyURL rangeOfString:domain].location != NSNotFound)
                {
                    NSHTTPCookie*cookie = [DAHTTPCookie createCookieWithURL:[ck domain] cookieName:[ck name] expires:[[ck expiresDate] timeIntervalSince1970] cookieValue:[ck value] browserType:DAWebkitBrowser secure:[ck isSecure]];

                    if ([_cookies containsObject:cookie] == NO)
                    {
                        [_cookies addObject:cookie];
                    }
                }

                [_pool2 release];
}
             }];
        }];

        [_pool release];
    }];

    [_queue addOperations:[NSArray arrayWithObject:_block] waitUntilFinished:YES];

    NSLog(@"Done - found %i Cookies...",[_cookies count]); //sometimes returns 0, sometimes the right number or nothing

编辑

我已经修复了它。你是对的，我的课程不是线程安全的。因此，我必须添加一个 @synchronized 块才能使其按预期工作。

Answer 1

当您说“我正在使用 for 循环并且它有效”时，您的意思是您的 for 循环是同时运行的，还是它们实际上是线性运行的？

看看这段代码，使用块没有任何问题。虽然在不完全了解您正在使用的 API 的情况下不可能得出结论，但该代码中的底层算法实际上是爆炸性并行。

也就是说，每个循环都配置为尽可能同时尝试和处理内容。除非您调用的 API 的每个部分都是线程安全的，否则您的代码可能会因多线程冲突而崩溃和/或锁定（症状的随机性是并发问题的明确标志）。

即使所有 API 都是线程安全的，爆炸性并发也几乎不是正确的并发模型。充其量，您需要限制并发性；您想使用某种机制来限制同时处理的垃圾量。

Answer 2

我没有看到 _cookies 是在哪里定义的，但我想我们可以假设它是一个 NSMutableArray。这些不是线程安全的，因此如果您有多个线程同时读取对象并向该数组添加对象，那就是一个问题。

有不同的方法可以解决这个问题。一种选择是创建一个拥有该阵列的串行队列。在处理过程中，每次需要检查（如果需要添加）cookie 时，请在该队列上安排一个小块来完成该工作。这样您就可以确保这些读取和添加是严格串行的。

当然还有其他的想法。

但退一步来说，一旦您引入了这种舞蹈来防止线程遍历这些 _cookie，您可能会发现也可能不会发现这是最有效的实现。您需要用实际数据进行测试。

另外，为什么在这里使用 NSOperation ？既然你无论如何都要等待结果，为什么不在主队列上启动枚举，然后让它从那里散开呢？ （我可能遗漏了一些东西）