Objective-C: Где удалить наблюдателя для NSNotification?

У меня объективный класс C. В нем я создал метод инициализации и настроил в нем NSNotification

//Set up NSNotification
[[NSNotificationCenter defaultCenter] addObserver:self 
                                         selector:@selector(getData)
                                             name:@"Answer Submitted"
                                           object:nil];

Где мне установить [[NSNotificationCenter defaultCenter] removeObserver:self]в этом классе? Я знаю, что для a UIViewControllerя могу добавить его в viewDidUnloadметод. Итак, что нужно сделать, если я только что создал целевой класс c?

Общий ответ будет «как только вам больше не нужны уведомления». Это явно не удовлетворительный ответ.

Я бы порекомендовал вам добавить вызов [notificationCenter removeObserver: self]в метод deallocтех классов, которые вы собираетесь использовать в качестве наблюдателей, так как это последний шанс полностью отменить регистрацию наблюдателя. Однако это защитит вас только от сбоев из-за уведомления центра уведомлений о мертвых объектах. Он не может защитить ваш код от получения уведомлений, когда ваши объекты еще не находятся / больше не находятся в состоянии, в котором они могут правильно обрабатывать уведомление. Для этого ... См. Выше.

Изменить (поскольку ответ, похоже, вызывает больше комментариев, чем я мог подумать) Все, что я пытаюсь сказать здесь, это: действительно сложно дать общий совет относительно того, когда лучше всего удалить наблюдателя из центра уведомлений, потому что это зависит от:

• В вашем случае использования (какие уведомления наблюдаются? Когда они отправляются?)
• Реализация наблюдателя (когда он готов получать уведомления? Когда он больше не готов?)
• Предполагаемое время жизни наблюдателя (привязано ли оно к какому-либо другому объекту, скажем, представлению или контроллеру представления?)
• ...

Итак, лучший общий совет, который я могу дать: защитите свое приложение. против хотя бы одного возможного отказа, сделайте removeObserver:танец dealloc, поскольку это последний момент (в жизни объекта), где вы можете сделать это чисто. Это не означает: «просто отложите удаление до deallocвызова, и все будет хорошо». Вместо этого удалите наблюдателя, как только объект больше не будет готов (или не требуется) для получения уведомлений . Это как раз подходящий момент. К сожалению, не зная ответов ни на один из упомянутых выше вопросов, я даже не могу предположить, когда это будет.

Вы всегда можете безопасно removeObserver:обрабатывать объект несколько раз (и все, кроме самого первого вызова с данным наблюдателем, будут nops). Итак: подумайте о том, чтобы сделать это (снова), deallocчтобы быть уверенным, но прежде всего: сделайте это в подходящий момент (который определяется вашим вариантом использования).

Примечание: это было протестировано и работает на 100%.

Swift

override func viewWillDisappear(animated: Bool){
    super.viewWillDisappear(animated)

    if self.navigationController!.viewControllers.contains(self) == false  //any other hierarchy compare if it contains self or not
    {
        // the view has been removed from the navigation stack or hierarchy, back is probably the cause
        // this will be slow with a large stack however.

        NSNotificationCenter.defaultCenter().removeObserver(self)
    }
}

ПредставленныйViewController

override func viewWillDisappear(animated: Bool){
    super.viewWillDisappear(animated)

    if self.isBeingDismissed()  //presented view controller
    {
        // remove observer here
        NSNotificationCenter.defaultCenter().removeObserver(self)
    }
}

Цель-C

В iOS 6.0 > version, лучше удалить наблюдателя, viewWillDisappearпоскольку viewDidUnloadметод устарел.

 [[NSNotificationCenter defaultCenter] removeObserver:observerObjectHere];

Во многих случаях лучше, remove observerкогда представление было удалено из navigation stack or hierarchy.

- (void)viewWillDisappear:(BOOL)animated{
 if (![[self.navigationController viewControllers] containsObject: self]) //any other hierarchy compare if it contains self or not
    {
        // the view has been removed from the navigation stack or hierarchy, back is probably the cause
        // this will be slow with a large stack however.

        [[NSNotificationCenter defaultCenter] removeObserver:observerObjectHere];
    }
}

ПредставленныйViewController

- (void)viewWillDisappear:(BOOL)animated{
    if ([self isBeingDismissed] == YES) ///presented view controller
    {
        // remove observer here
        [[NSNotificationCenter defaultCenter] removeObserver:observerObjectHere];
    }
}

Начиная с iOS 9, больше не нужно удалять наблюдателей.

В OS X 10.11 и iOS 9.0 NSNotificationCenter и NSDistributedNotificationCenter больше не будут отправлять уведомления зарегистрированным наблюдателям, которые могут быть освобождены.

https://developer.apple.com/library/mac/releasenotes/Foundation/RN-Foundation/index.html#10_11NotificationCenter

Если наблюдатель добавлен к контроллеру представления , я настоятельно рекомендую добавить его viewWillAppearи удалить viewWillDisappear.

-(void) dealloc {
      [[NSNotificationCenter defaultCenter] removeObserver:self];
      [super dealloc];
}

В общем вложил в deallocметод.

В быстром использовании deinit, потому что dealloc недоступен:

deinit {
    ...
}

Документация Swift:

Деинициализатор вызывается непосредственно перед освобождением экземпляра класса. Вы пишете деинициализаторы с ключевым словом deinit, подобно тому, как инициализаторы пишутся с ключевым словом init. Деинициализаторы доступны только для типов классов.

Обычно вам не нужно выполнять ручную очистку, когда ваши экземпляры освобождены. Однако, когда вы работаете со своими собственными ресурсами, вам может потребоваться выполнить дополнительную очистку самостоятельно. Например, если вы создаете настраиваемый класс для открытия файла и записи в него некоторых данных, вам может потребоваться закрыть файл до того, как экземпляр класса будет освобожден.

* изменить: этот совет относится к iOS <= 5 (даже там вы должны добавлять viewWillAppearи удалять viewWillDisappear- однако совет применяется, если по какой-то причине вы добавили наблюдателя viewDidLoad)

Если вы добавили наблюдателя в, viewDidLoadвы должны удалить его как в, так deallocи в viewDidUnload. В противном случае вы в конечном итоге добавите его дважды, когда viewDidLoadбудет вызван after viewDidUnload(это произойдет после предупреждения памяти). Это не обязательно в iOS 6, где viewDidUnloadона устарела и не будет вызываться (поскольку представления больше не выгружаются автоматически).

На мой взгляд, следующий код не имеет смысла в ARC :

- (void)dealloc
{
      [[NSNotificationCenter defaultCenter] removeObserver:self];
      [super dealloc];
}

В iOS 6 также нет смысла удалять наблюдателей в viewDidUnload, потому что теперь он устарел.

Подводя итог, я всегда делаю это внутри viewDidDisappear. Однако это также зависит от ваших требований, как сказал @Dirk.

Думаю, я нашел надежный ответ ! Мне пришлось, поскольку приведенные выше ответы неоднозначны и кажутся противоречивыми. Я просмотрел поваренные книги и руководства по программированию.

Во-первых, стиль addObserver:in viewWillAppear:и removeObserver:in viewWillDisappear:у меня не работает (я тестировал его), потому что я отправляю уведомление в контроллер дочернего представления для выполнения кода в контроллере родительского представления. Я бы использовал этот стиль только в том случае, если бы отправлял и слушал уведомление в том же контроллере представления.

Ответ, на который я буду полагаться больше всего, я нашел в книге «Программирование для iOS: руководство Big Nerd Ranch Guide 4th». Я доверяю ребятам из BNR, потому что у них есть учебные центры iOS, и они не просто пишут очередную кулинарную книгу. Вероятно, в их интересах быть точными.

БНР, пример первый: addObserver:в init:, removeObserver:вdealloc:

БНР, пример второй: addObserver:в awakeFromNib:, removeObserver:вdealloc:

… При удалении наблюдателя dealloc:они не используют[super dealloc];

Надеюсь, это поможет следующему человеку ...

Я обновляю этот пост, потому что Apple теперь почти полностью отказалась от раскадровок, поэтому вышеупомянутое может не относиться ко всем ситуациям. Важная вещь (и причина, по которой я добавил этот пост в первую очередь) - обратить внимание, если вам viewWillDisappear:звонят. Когда приложение вошло в фоновый режим, это было не для меня.

Принятый ответ небезопасен и может вызвать утечку памяти. Пожалуйста, оставьте отмену регистрации в dealloc, но также отмените регистрацию в viewWillDisappear (конечно, если вы зарегистрируетесь в viewWillAppear) .... Я ВСЕГДА ДЕЛАЛ ЭТО ТАКОЕ, И ЭТО РАБОТАЕТ ОТЛИЧНО! :)

Также важно отметить, что это viewWillDisappearвызывается также, когда контроллер представления представляет новый UIView. Этот делегат просто указывает, что основное представление контроллера представления не отображается на дисплее.

В этом случае удаление уведомления viewWillDisappearможет быть неудобным, если мы используем уведомление, чтобы позволить UIview взаимодействовать с родительским контроллером представления.

В качестве решения я обычно удаляю наблюдателя одним из этих двух методов:

- (void)viewWillDisappear:(BOOL)animated {
    NSLog(@"viewController will disappear");
    if ([self isBeingDismissed]) {
        NSLog(@"viewController is being dismissed");
        [[NSNotificationCenter defaultCenter] removeObserver:self name:@"actionCompleted" object:nil];
    }
}

-(void)dealloc {
    NSLog(@"viewController is being deallocated");
    [[NSNotificationCenter defaultCenter] removeObserver:self name:@"actionCompleted" object:nil];
}

По тем же причинам, когда я отправляю уведомление в первый раз, мне нужно учитывать тот факт, что каждый раз, когда над контроллером появляется представление с, тогда viewWillAppearметод запускается. Это, в свою очередь, создаст несколько копий одного и того же уведомления. Поскольку нет способа проверить, активно ли уведомление, я устраняю проблему, удаляя уведомление перед его добавлением:

- (void)viewWillAppear:(BOOL)animated {
    NSLog(@"viewController will appear");
    // Add observers
    [[NSNotificationCenter defaultCenter] removeObserver:self name:@"imageGenerated" object:nil]; // This is added to avoid duplicate notifications when the view is presented again
    [[NSNotificationCenter defaultCenter] addObserver:self selector:@selector(receivedImageFromCameraOrPhotolibraryMethodOnListener:) name:@"actionCompleted" object:nil];

}

SWIFT 3

Есть два случая использования уведомлений: - они нужны только тогда, когда контроллер представления находится на экране; - они нужны всегда, даже если пользователь открыл другой экран поверх текущего.

В первом случае правильное место для добавления и удаления наблюдателя:

/// Add observers
///
/// - Parameter animated: the animation flag
override func viewWillAppear(_ animated: Bool) {
    super.viewWillAppear(animated)
    NotificationCenter.default.addObserver(...)
}

/// Remove observers
///
/// - Parameter animated: the animation flag
override func viewWillDisappear(_ animated: Bool) {
    super.viewWillDisappear(animated)
    NotificationCenter.default.removeObserver(self)
}

для второго случая правильный способ:

/// Add observers
override func viewDidLoad() {
    super.viewDidLoad()
    NotificationCenter.default.addObserver(...)
}

/// Remove observers
///
/// - Parameter animated: the animation flag
override func viewWillDisappear(_ animated: Bool) {
    super.viewWillDisappear(animated)
    if self.isBeingDismissed // remove only when view controller is removed disappear forever
    || !(self.navigationController?.viewControllers.contains(self) ?? true) {
        NotificationCenter.default.removeObserver(self)
    }
}

И никогда не ставили removeObserverв deinit{ ... }- это ОШИБКА!

override func viewDidLoad() {   //add observer
  super.viewDidLoad()
  NotificationCenter.default.addObserver(self, selector:#selector(Yourclassname.method), name: NSNotification.Name(rawValue: "NotificationIdentifier"), object: nil)
}

override func viewWillDisappear(_ animated: Bool) {    //remove observer
    super.viewWillDisappear(true)
    NotificationCenter.default.removeObserver(self, name: NSNotification.Name(rawValue: "NotificationIdentifier"), object: nil)
}