Poprawne zarządzanie addObserverForName: object: queue: usingBlock:

Question

Wciąż jestem nowy dla bloków w celu-c i zastanawiam się, czy mam ten kod psuedo poprawny. Nie jestem pewien, czy to wystarczy po prostu usunąć obserwatora czy mam zadzwonić removeObserver: Nazwa: obiekt:

-(void) scan { 
    Scanner *scanner = [[Scanner alloc] init]; 
    id scanComplete = [[NSNotificationCenter defaultCenter] addObserverForName:@"ScanComplete" 
         object:scanner 
         queue:nil 
         usingBlock:^(NSNotification *notification){ 
          /* 
          do something 
          */ 
          [[NSNotificationCenter defaultCenter] removeObserver:scanComplete]; 
          [scanner release]; 
         }]; 
    [scanner startScan]; 
} 

Update: Otrzymuję przerywany EXC_BAD_ACCESS z tego bloku, więc to nie może być dobrze.

Answer 1

Należy zadeklarować zmienną scanComplete przed zdefiniowaniem samego bloku.

Powodem, dla którego musisz to zrobić, jest to, że próbujesz uzyskać dostęp do zmiennej, która nie istnieje w bloku w momencie definicji, ponieważ sama zmienna nie została jeszcze przypisana.

Co to jest EXC_BAD_ACCESS? Cóż, jest to wyjątek, który jest zgłaszany podczas próby uzyskania dostępu do referencji, która nie istnieje. Tak właśnie jest w twoim przykładzie.

Więc jeśli zadeklarować zmienną przed samym bloku, to powinno działać:

-(void) scan { 
    Scanner *scanner = [[Scanner alloc] init]; 
    __block id scanComplete; 
    scanComplete = [[NSNotificationCenter defaultCenter] addObserverForName:@"ScanComplete" 
         object:scanner 
         queue:nil 
         usingBlock:^(NSNotification *notification){ 
          /* 
          do something 
          */ 
          [[NSNotificationCenter defaultCenter] removeObserver:scanComplete]; 
          [scanner release]; 
        }]; 
    [scanner startScan]; 
} 

Answer 2

zakresem bloku nie ma uprawnień, aby zwolnić obiekt skanera. Jeśli nie używasz zbierania śmieci, usunięcie release i zrobienie autoreakcji skanera ([[[Scanner alloc] init] autorelease]) powinno załatwić sprawę.

Powinieneś być również w stanie bezpiecznie przenieść połączenie do removeObserver poza blok.

W przypadku EXC_BAD_ACCESS: Wprowadzenie do bt w oknie konsoli po zawieszeniu się aplikacji spowoduje zapisanie śledzenia wstecznego i powinno poinformować użytkownika, gdzie wystąpił błąd.

Answer 3

Nie należy wyrejestrowywać się w bloku rejestru. Zamiast tego przechowuj token zwrócony z addObserverForName (w tym przypadku, twoja scanComplete) jako zmienną instancji lub w kolekcji, która jest zmienną instancji, i wyrejestruj później, gdy masz zamiar wyjść z istnienia (np. dealloc). To, co robię, to utrzymywanie NSMutableSet o nazwie observers. Więc:

id ob = [[NSNotificationCenter defaultCenter] 
    addObserverForName:@"whatever" object:nil queue:nil 
    usingBlock:^(NSNotification *note) { 
     // ... whatever ... 
}]; 
[self->observers addObject:ob]; 

A potem:

for (id ob in self->observers) 
    [[NSNotificationCenter defaultCenter] removeObserver:ob]; 
self->observers = nil; 

Answer 4

jabłko dokumentu o tej metodzie:

Poniższy przykład pokazuje w jaki sposób można się zarejestrować, aby otrzymywać powiadomienia o zmianie ustawień narodowych.

NSNotificationCenter *center = [NSNotificationCenter defaultCenter]; 
NSOperationQueue *mainQueue = [NSOperationQueue mainQueue]; 
self.localeChangeObserver = [center addObserverForName:NSCurrentLocaleDidChangeNotification object:nil 
    queue:mainQueue usingBlock:^(NSNotification *note) { 

     NSLog(@"The user's locale changed to: %@", [[NSLocale currentLocale] localeIdentifier]); 
    }]; 

Aby wyrejestrować obserwacje, należy przekazać obiekt zwracany przez tę metodę, aby usunąćObserwer :. Musisz musi wywołać removeObserver: lub removeObserver: name: object: before dowolny obiekt określony przez addObserverForName: object: queue: usingBlock: is deallocated.

NSNotificationCenter *center = [NSNotificationCenter defaultCenter]; 
[center removeObserver:self.localeChangeObserver];