迭代时是否可以从 mutablearray 中删除字典？ [复制]

Question

for (int i = 0; i< [optionDataArr count]; i++) {
    NSString *sName  = [[optionDataArr objectAtIndex:i] objectForKey:kOptionName];
    NSString *sPrice = [[optionDataArr objectAtIndex:i] objectForKey:kOptionExtraPrice];

    if (sName.length == 0  && sPrice.length == 0) {
          [optionDataArr removeObjectAtIndex:i];
    }  
}

假设optionDataArr 包含一个没有值的字典，当执行上述代码时，我收到：

Terminating app due to uncaught exception 'NSRangeException', reason: '*** -[__NSArrayM objectAtIndex:]: index 0 beyond bounds for empty array'

Answer 1

您可以在使用普通的旧 for 循环时删除项目，而在使用快速枚举时则不能。

不过，您的代码有问题。当您删除第 n 个元素时，下一个元素将是第 (n+2) 个。您需要手动将索引减一以考虑移位的元素。

还请记住，在这种情况下，您确实需要对循环中的数组长度进行“实时”边界检查，而不仅仅是使用保存长度的临时变量（或者您需要将其递减为好）。

Answer 2

在这一行以下：

[optionDataArr removeObjectAtIndex:i];

添加这一行：

i--;

所以，代码是：

if (sName.length == 0  && sPrice.length == 0) {
    [optionDataArr removeObjectAtIndex:i];
    i--;
}

原因：当您在迭代数组时从数组中删除一个项目时，索引会发生变化。因此，这就是您需要手动减少索引的原因。

Answer 3

Eiko 的答案是正确的，但我想展示另一个使用快速枚举的版本。您无法使用快速枚举删除项目，因此您必须存储索引，然后稍后删除相应的项目：

NSMutableIndexSet * indexesToRemove = [NSMutableIndexSet indexSet];

[optionDataArr enumerateObjectsUsingBlock:^(NSDictionary *dico, NSUInteger idx, BOOL *stop) {
    if ([dico count] == 0)
        [indexesToRemove addIndex:idx];
}];

[optionDataArr removeObjectsAtIndexes:indexesToRemove];

编辑：

正如 Martin R 所建议的，您也可以使用 indexesOfObjectsPassingTestmethod ：

NSIndexSet * indexesToRemove = [optionDataArr indexesOfObjectsPassingTest:^BOOL(NSDictionary *dico, NSUInteger idx, BOOL *stop) {
    return ([dico count] == 0);
}];
[optionDataArr removeObjectsAtIndexes:indexesToRemove];

Answer 4

您当然可以为此使用标准 for 循环，前提是您进行了 Eiko 已经提到的修改。

然而，在 Objective C 中处理这个问题的惯用方法是遍历数组的副本：

for (id obj in [optionDataArr copy]) {
    // some processing code

    if (condition) {
        [optionDataArr removeObject:obj]
    }
}

虽然这确实需要数组的副本，但除非您确定要处理大量数据，否则我会从可读版本开始，并在必要时优化到普通的 for 循环。