Какие плюсы и минусы подписки на нотификацию во ViewDidLoad?

Плюсы и минусы подписки на уведомления в viewDidLoad

Подписка на нотификации (уведомления) в методе viewDidLoad — распространённая практика в iOS-разработке, но она имеет как преимущества, так и существенные недостатки, которые важно учитывать для создания стабильного и корректного приложения.

Плюсы

1. Предсказуемость жизненного цикла
   Метод viewDidLoad вызывается один раз после загрузки view контроллера в память. Это гарантирует, что подписка выполнится в контролируемый момент, когда контроллер готов к работе, но еще не отображён на экране. Это удобно для начальной настройки, включая регистрацию на события, которые должны быть активны на протяжении всей жизни контроллера.

2. Централизованная настройка
   Размещение кода подписки в одном месте (вместе с другими настройками view) улучшает читаемость и поддерживаемость кода. Разработчик видит все настройки контроллера в одном месте, что упрощает понимание логики.

3. Автоматическая отписка в deinit
   При правильном управлении памятью можно отписываться от уведомлений в деинициализаторе, что легко сочетается с подпиской в viewDidLoad:

   class MyViewController: UIViewController {
       override func viewDidLoad() {
           super.viewDidLoad()
           NotificationCenter.default.addObserver(
               self,
               selector: #selector(handleNotification(_:)),
               name: .myNotification,
               object: nil
           )
       }
       
       deinit {
           NotificationCenter.default.removeObserver(self)
       }
       
       @objc private func handleNotification(_ notification: Notification) {
           // Обработка уведомления
       }
   }
   

Минусы

1. Риск утечек памяти (retain cycles)
   Если в обработчике уведомления используется захват self без слабой ссылки (например, в замыкании), это может создать цикл сильных ссылок, особенно при использовании NotificationCenter с замыканиями. Контроллер не будет освобождён, даже если он больше не нужен:

   // ОПАСНО: возможна утечка памяти
   override func viewDidLoad() {
       super.viewDidLoad()
       NotificationCenter.default.addObserver(
           forName: .myNotification,
          4. object: nil,
           queue: .main) { [weak self] _ in
               self?.doSomething() // Если забыть weak, будет retain cycle
           }
   }
   

2. Несоответствие жизненному циклу контроллера
   viewDidLoad вызывается один раз, но контроллер может быть неактивен (например, скрыт другим контроллером). Подписка в этом методе может привести к ненужной обработке уведомлений, когда контроллер не должен на них реагировать. Это тратит ресурсы и может вызывать побочные эффекты.

3. Сложность управления при многократном появлении/исчезновении
   Если контроллер может быть переиспользован (например, в UINavigationController), подписка в viewDidLoad не позволит гибко включать/отключать реакции на уведомления. Например, если обработка нужна только когда контроллер видим, лучше подписываться в viewWillAppear и отписываться в viewWillDisappear.

4. Проблемы с порядком инициализации
   Если нотификация может быть отправлена до вызова viewDidLoad (например, из других модулей при запуске), контроллер её пропустит. Это может быть критично для событий, важных для начального состояния UI.

Рекомендации

• Используйте viewDidLoad для подписки на глобальные события, которые должны обрабатываться независимо от видимости контроллера (например, обновление данных из фоновых задач).
• Для событий, связанных с отображением UI, предпочтительнее viewWillAppear/viewWillDisappear.
• Всегда используйте отписку в deinit или соответствующем методе жизненного цикла, чтобы избежать утечек.
• Для iOS 9+ рассмотрите использование weak-ссылок в замыканиях или паттерн токенов, возвращаемых addObserver(forName:object:queue:using:), который упрощает управление памятью.

class SafeViewController: UIViewController {
    private var notificationTokens: [NSObjectProtocol] = []
    
    override func viewDidLoad() {
        super.viewDidLoad()
        let token = NotificationCenter.default.addObserver(
            forName: .myNotification,
            object: nil,
            queue: .main
        ) { [weak self] _ in
            self?.handleNotification()
        }
        notificationTokens.append(token)
    }
    
    deinit {
        notificationTokens.forEach {
            NotificationCenter.default.removeObserver($0)
        }
    }
}

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