Проблема с памятью Objective-C с помощью performSelector и бесконечного цикла

Question

Я работаю над проектом, в котором я должен попробовать выполнить данный метод в течение заданного времени (см. Его как автоматический режим), и если в автоматическом режиме нет преуспеть, переключитесь в ручной режим. Чтобы предупредить вас, я довольно новичок в Objective-C и iOS dev, поэтому ошибка может быть очевидной.

До сих пор, вот что я сделал:

-(void)viewDidLoad { 
    [self performSelector:@selector(autoMode) withObject:nil afterDelay:1.0]; 
    [self performSelectorInBackground:@selector(switchManualMode) withObject:nil]; 
} 


-(void)autoMode { 
    @autoreleasepool { 
     while (isAutoMode == true) { 
      if ([session isRunning]) 
       [self captureImage]; 
       // NSLog(@"test"); 
     } 
    } 
} 

-(void)switchManualMode { 
    [NSThread sleepForTimeInterval:2.0f]; 
    isAutoMode = false; 
    self.button.enabled = true; 

    UIAlertView* alert = [[UIAlertView alloc] initWithTitle:@"Changement de mode" 
                message:@"Du a un délai trop long, la capture automatique va être desactivée. La capture se fait maintenant de manière manuelle Touchez l'écran pour prendre une photo." 
                delegate:nil 
              cancelButtonTitle:@"OK" 
              otherButtonTitles:nil]; 

    [alert show]; 
} 

У меня есть несколько вопросов здесь:
- Первый, что память используется пик и сбои приложения.
- Второе, что switchManualMode никогда не вызывается.

Я попытался просто напечатать «тест» в журналах (вместо того, чтобы на самом деле вызвать функцию «captureImage»), и когда я это делаю, все в порядке.

Я не могу опубликовать код captureImage, но метод пытается получить изображение из AVCaptureSession, а затем обрабатывает его на нем.
В любом случае, я не понимаю, почему switchManualMode вызывается в случае, а не в другом.

Answer 1

Одна из причин заключается в том, что вы положили @autoreleasepool точно в неправильное место. Он должен быть внутри петля. Вы запускаете бесконечный цикл в пределах тот же пул авторефератов; это требует проблем с памятью. Захват изображений с максимально возможной скоростью точно не помогает.

Другая проблема заключается в том, что performSelector будет работать в основном потоке, поэтому второй выполнитьSelector не будет вызываться, так как первый вызов никогда не возвращается.

Вы действительно должны действительно избавиться от performSelector и переключиться на GCD. Я бы не хотел, чтобы функции performSelector и sleep вызывались в любом коде, написанном сегодня.

Answer 2

Я предпочитаю использовать рекурсию для этого типа вещей:

-(void)buttonTappedMethod 
{ 
    [self recusionMethod]; 
} 

-(void)recursionMethod 
{ 
    if(self.isAutoMode) 
    { 
     // ------------------------------------------------ 
     // Do something 
     // 
     // change self.isAutoMode = YES 
     // on condition satisfied in this logic block 
     // ------------------------------------------------ 

     // ------------------------------------------------ 
     // call this method again 
     // ------------------------------------------------ 
     [self recursionMethod]; 
    } 
}