在iOS ARC中，我的递归函数使用EXC_BAD_ACCESS使应用程序崩溃

Question

我的以下代码在ARC模式下会导致崩溃：

MxTextField.m

+enableAllTextFields:(BOOL)enable InViews:(__weak UIView*) view
{

   @try
   {
      NSArray* textFields = view.subViews;

      for(int idx = 0; idx < textFields.count; idx++)
      {
         __weak UIView* view = [textFields objectAtIndex:idx];

         if(view.subViews.count > 0)
            [MxTextField enableAllTextFields:enable InView:view];
         else
            NSLog(@"No SubViews");

         if([view class] == [MxTextField class])
            [(MxTextField*) view setEnabled:enable];
      }

   }
   @catch(NSException exception)
   {
      NSLog(@"%s : %@",__func__,exception);
   }

}

在执行此函数的Some Loop之后。通过在函数末尾显示断点EXC_BAD_ACCESS来崩溃。 谁能帮我解决此实施中出了什么问题？

任何帮助将是感激的。

Answer 1

问题可能是迭代所采用的方法，try-catch也不是一个好习惯，请使用快速枚举以获得更快，更可靠的结果。 以下代码可以解决您的问题

+(void)enableAllTextField:(BOOL)enable inView:(UIView *)contrainerView
{
    for (UIView *subview in contrainerView.subviews) {
        if(subview.subviews.count>0)
             [MxTextField enableAllTextField:enable inView:subview];
        else if ([subview isKindOfClass:[MxTextField class]]) {
             MxTextField *textField = (MxTextField *)subview;
             [textField setEnabled:enable];
        }
    }
}

Answer 2

撇开其他许多问题，从发布的代码中可以看到崩溃的唯一原因是您的方法应该返回一个对象，但不会这样做。

说明：虽然在Objective-C中省略返回类型并不常见，但这是完全合法的。 这意味着该方法返回一个id类型的对象。

由于您的方法缺少return语句，因此返回的值是不确定的。 这会混淆ARC，并可能使其自动释放返回寄存器中的随机值，最终导致崩溃。

这是您方法的正确版本：

+ (void)forAllTextFieldsIn:(UIView *)view setEnabled:(BOOL)enabled
{
    if ([view isKindOfClass:[MxTextField class]])
        [(MxTextField *)view setEnabled:enabled];

    for (UIView *subview in view.subviews)
        [self forAllTextFieldsIn:subview setEnabled:enabled];
}