¿Por qué no eliminar Observer de NSNotificationCenter: addObserverForName: usingBlock get called

Question

Estoy confundido sobre por qué el observador nunca se elimina en el siguiente código. En mi viewDidAppear Tengo el siguiente:

-(void)viewDidAppear:(BOOL)animated{ 

id gpsObserver = [[NSNotificationCenter defaultCenter] 
          addObserverForName:FI_NOTES[kNotificationsGPSUpdated] 
          object:nil 
          queue:[NSOperationQueue mainQueue] 
          usingBlock:^(NSNotification *note){ 

           NSLog(@"run once, and only once!"); 

       [[NSNotificationCenter defaultCenter] removeObserver:gpsObserver]; 

     }]; 

} 

El observador nunca se elimina y el comunicado se emite cada vez que la notificación se envía. ¿Alguien puede proporcionar alguna orientación?

Answer 1

Cuando el bloque se inserta en la pila por addObserverForName:, el método aún no ha regresado, por lo que gpsObserver es nulo (en ARC) o basura/indefinido (no en ARC). Declare la variable usando __block afuera y esto debería funcionar.

__block __weak id gpsObserver; 

gpsObserver = [[NSNotificationCenter defaultCenter] 
          addObserverForName:FI_NOTES[kNotificationsGPSUpdated] 
          object:nil 
          queue:[NSOperationQueue mainQueue] 
          usingBlock:^(NSNotification *note){ 

           NSLog(@"run once, and only once!"); 

       [[NSNotificationCenter defaultCenter] removeObserver:gpsObserver]; 

     }]; 

He añadido un __weak para asegurar que no haya pérdida de memoria (según la respuesta de Matt). Código no probado.

Answer 2

Encuentro que de hecho hay una pérdida de memoria a menos que el observador esté marcado __block y __weak. Use los instrumentos para asegurarse de que self no esté siendo retenido; Apuesto a que es. Esto, sin embargo, funciona correctamente (desde mi código real):

__block __weak id observer = [[NSNotificationCenter defaultCenter] 
    addObserverForName:@"MyMandelbrotOperationFinished" 
    object:op queue:[NSOperationQueue mainQueue] 
    usingBlock:^(NSNotification *note) { 
     // ... do stuff ... 
     [[NSNotificationCenter defaultCenter] 
      removeObserver:observer 
      name:@"MyMandelbrotOperationFinished" 
      object:op]; 
}];