在 Swift 2 中删除通知观察者的正确位置在哪里？

Question

我有这个代码，取自这个答案：https://stackoverflow.com/a/29099066/406322

extension NSNotificationCenter {
    func setObserver(observer: AnyObject, selector: Selector, name: String?, object: AnyObject?) {
        NSNotificationCenter.defaultCenter().removeObserver(observer, name: name, object: object)
        NSNotificationCenter.defaultCenter().addObserver(observer, selector: selector, name: name, object: object)
    }
}

现在，在我的视图控制器中，我在 viewDidLoad() 中设置我的观察者：

override func viewDidLoad() {
        super.viewDidLoad()
        setObservers()
}

func setObservers() {
        NSNotificationCenter.defaultCenter().setObserver(self, selector: #selector(BaseController.handleComment(_:)), name: "newComment", object: nil)
}

然而，即使使用这个扩展，观察者在被添加之前被移除，每次我退出视图控制器并返回它时，我都会收到多个通知（每次多一个）。

这怎么可能？

Answer 1

如果你需要这个setObserver 扩展，你很可能做错了什么。您应该能够轻松地平衡您的注册和删除。如果不能，您的通知管理很可能过于复杂或位置错误。

通常添加观察的正确位置是在viewWillAppear（或viewDidAppear，两者都可以），并在viewDidDisappear（或viewWillDisappear）中删除它们。这可以确保您在屏幕外时不会收到通知，即使视图控制器仍然存在（这很常见）。

如果您的视图控制器要求它在屏幕外时接收通知，那么您有一个设计问题。视图控制器应该只管理屏幕视图。如果他们正在做其他事情，那么您将太多的模型放入控制器中。

正如@rmaddy 所指出的，您的具体问题可能是您有此视图控制器的两个实例。这可能很好，也可能是一个错误（这取决于视图控制器的工作方式）。但是，如果您在屏幕上和屏幕外时平衡添加和删除注册，那部分就可以了。