for...in带来的坑

废话不多说,先看一段报错代码

for (id model in self.models) {
      if ([model isEqual:device]) {
          [self.models removeObject:model];
      }
}

运行直接报错

Terminating app due to uncaught exception 'NSGenericException', 
reason: '*** Collection <__NSArrayM: 0x174241230> was mutated while being enumerated.'

was mutated while being enumerated是说在枚举时发生突变

很多人都以为forin和for是一样的,其实不然,实际上forin利用的是快速枚举NSFastEnumerate机制,跟for循环意义上还是有区别的,对forin的数组对象做任何的增删操作都会was mutated while being enumerated

查阅文档中有关于枚举的:
NSArray的枚举操作中有一条需要注意:对于可变数组进行枚举操作时,你不能通过添加或删除对象这类操作来改变数组容器。如果你这么做了,枚举器会很困惑,而你将得到未定义的结果。

而这种操作本身也是有问题的,数组容器已经改变,可能遍历到没有分配的位置,用for循环机器不能自己察觉,但是枚举器可以察觉

这一点其实在遍历UIView的subviews时也有所体现,例如下面的代码就不会报错

for (UIView *v in view.subviews) {
  if ([v isKindOfClass:[UIButton class]]) {
    [v removeFromSuperView];
  }
}

细心的同学可能观察到了,当你removeFromSuperView后,view.subviews这个数组已经不是原来的数组了,通过地址就可以看出, 这是系统已经帮我们处理过了。

总结:在使用forin时,任何对数组的增删操作都会crash

解决办法:

  1. 使用for循环
for (int i = 0; i < self.models.count; i++) {
        id model = self.models[i];
        if ([model isEqual:device]) {
            [self.models removeObject:model];
        }
    }
  1. 临时变量
NSMutableArray *temp = [NSMutableArray arrayWithArray: self.models];
for (id model in temp) {
      if ([model isEqual:device]) {
          [self.models removeObject:model];
      }
}
  1. break跳出(当满足条件即跳出循环时使用)
for (id model in self.models) {
      if ([model isEqual:device]) {
          [self.models removeObject:model];
          break; // 结束循环即不会继续进行枚举,所以不会crash
      }
}
  1. 系统枚举API(比较建议使用)
[self.models enumerateObjectsUsingBlock:^(id  _Nonnull model, NSUInteger idx, BOOL * _Nonnull stop) {
      if ([model isEqual:device]) {
            [self.models removeObject:model];
            *stop = YES; // 同forin中break,但这里即使不结束循环也不会crash,因为系统api内部已经做了处理
       }
}];

enumerateObjectsUsingBlock 的内部实现:(实现)

 - (void)forin:(void (^)(id obj, NSUInteger idx, BOOL *stop))block {
    if (!block || self.count < 1) {
        return;
    }
    
    for (NSUInteger i = 0; i < self.count; i++) {
        id obj = self[i];
        
        BOOL s;
        block(obj, i, &s);
        if (s) {
            break;
        }
    }
}




感谢您看完了,有没有帮助到您呢???

你可能感兴趣的:(for...in带来的坑)