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

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

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

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

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

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

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

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

Быстрый

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.

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

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

deinit {
    ...
}

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

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

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

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

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

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

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

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

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

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

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

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

Первый пример BNR: addObserver: в init:, removeObserver: в dealloc:

BNR, пример два: 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];

}

Это единственно правильный ответ (все остальные ошибочно предлагают использовать deinit и dealloc - это явное непонимание жизненного цикла экземпляра класса и системы iOS).

Swift 5

Есть два случая использования уведомлений:

• они необходимы только, когда контроллер представления на экране (viewWillAppear + viewWillDisappear)
• они необходимы всегда, даже если пользователь открыл другой экран поверх текущего (viewDidLoad + viewWillDisappear с если).

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

    /// Add observers
    override func viewWillAppear(_ animated: Bool) {
        super.viewWillAppear(animated)
        NotificationCenter.default.addObserver(...)
    }

    /// Remove observers
    override func viewWillDisappear(_ animated: Bool) {
        super.viewWillDisappear(animated)

// remove when screen dismissed
        NotificationCenter.default.removeObserver(self) 
    }

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

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

    /// Remove observers
    override func viewWillDisappear(_ animated: Bool) {
        super.viewWillDisappear(animated)

// remove only when view controller is removed disappear forever
        if self.isBeingDismissed 
        || !(self.navigationController?.viewControllers.contains(self) ?? true) {
            NotificationCenter.default.removeObserver(self)
        }
    }

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