如何解决“枚举时集合发生了变异”、@synrchonized、mutableCopy 或其他问题？

Question

在 Crashlytics 中，我看到用户很少遇到的崩溃。有问题的代码看起来像这样......

- (void)updateIsAnsweredField:(NSArray *)moduleItemsList
{
    if (moduleItemsList && self.answeredItems && self.answeredItems.count > 0) {
        for (ModuleItem * item in moduleItemsList) { // "Collection was mutated while being enumerated"
            if ([item isKindOfClass:[ModuleItem class]] && [item shouldCheckIfAnswered]) {
                item.answered = [self isAnsweredItem:item.moduleID];
            }
        }
    }
}

Crashlytics给出的错误可以在上面代码sn-p的注释中看到。

我认为有几种方法可以解决这个问题。

1) 将函数内的所有内容包装在 @synchronized(moduleItemsList) {} 中。这是解决问题的理想方法吗？我听说@synchronized 非常慢，尽可能避免使用它。

2) 创建一个副本，如 NSMutableArray *copyModuleItemsList = [moduleItemsList mutableCopy];。然后列举。这能解决问题吗？我认为它会解决这个特定问题，但会有另一个问题，不是吗？那就是...最后，当我们将副本分配回原来的 moduleItemsList = copyModuleItemsList; 时，moduleItemsList 可能同时在不同的线程上发生了变化。

3) 将传入的:(NSArray *)moduleItemsList 跟踪到将其作为属性持有的任何人。然后覆盖getter以使用dispatch_sync，并覆盖setter以使用dispatch_barrier_async。但是，不能保证原始数组是可以覆盖其 getter 和 setter 的任何类的属性。实际上，这些都没有任何意义，因为我们不会专门更改该数组吗？

我有点困惑。任何人都可以在这件事上提供帮助吗？ #1 是我想要的选项吗？

编辑：添加更多代码

[item shouldCheckIFAnswered]:

这只是检查 ModuleItem 类中存在的 @property 值。 if self.moduleType == ModuleTypeSuchAndSuch

isAnsweredItem::

- (BOOL)isAnsweredItem:(NSString *)moduleID
{
    if (!self.answeredItems) {
        return NO;
    }

    return [self.answeredItems containsObject:moduleID];
}

Answer 1

从您的帖子中，听起来moduleItemsList 正在另一个线程中进行修改。解决此问题的“正确”方法将取决于另一个线程中的状态与该线程中的状态之间的所需关系。

如果您在 both 此代码中以及在另一个线程中修改集合的代码中使用 @synchronized(moduleItemsList)，那么当此代码运行时，它将始终具有“最新"moduleItemsList 的视图。

如果您将moduleItemsList 复制到另一个对象中，那么当此代码运行时，它可能会在不再位于moduleItemsList 中的项目上设置answered 值，或者可能无法设置answered标记最近添加到 moduleItemsList 的项目。

一般来说，@synchronized 版本是获得“正确”行为的更简单方法。如果您确定两个线程可能不同意 moduleItemsList 的当前内容并不重要，您只想使用复制。

我听说@synchronized 很慢，尽可能避免使用它。

总的来说，这是糟糕的建议。 @synchronized 与确保线程之间的一致状态并提供重入锁一样慢。你不想随便乱扔@synchronized，不管怎样，但它是在线程之间同步数据访问的一个很好的解决方案——毕竟，这就是它的目的。