Objective-C: Xóa người quan sát cho NSNotification ở đâu?

Tôi có một lớp C khách quan. Trong đó, tôi đã tạo một phương thức init và thiết lập một NSNotification trong đó

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

Tôi đặt [[NSNotificationCenter defaultCenter] removeObserver:self]ở đâu trong lớp này? Tôi biết rằng đối với a UIViewController, tôi có thể thêm nó vào viewDidUnloadphương thức Vì vậy, những gì cần phải làm nếu tôi vừa tạo một Lớp c mục tiêu?

Câu trả lời chung chung sẽ là "ngay khi bạn không cần thông báo nữa". Đây rõ ràng không phải là một câu trả lời thỏa mãn.

Tôi khuyên bạn nên thêm một cuộc gọi [notificationCenter removeObserver: self]trong phương thức dealloccủa các lớp đó, mà bạn định sử dụng làm người quan sát, vì đây là cơ hội cuối cùng để hủy đăng ký một người quan sát một cách sạch sẽ. Tuy nhiên, điều này sẽ chỉ bảo vệ bạn khỏi sự cố do trung tâm thông báo thông báo các đối tượng đã chết. Nó không thể bảo vệ mã của bạn khỏi việc nhận thông báo, khi các đối tượng của bạn chưa / không còn ở trạng thái mà chúng có thể xử lý thông báo đúng cách. Đối với điều này ... Xem ở trên.

Chỉnh sửa (vì câu trả lời dường như thu hút nhiều nhận xét hơn tôi nghĩ) Tất cả những gì tôi muốn nói ở đây là: thật khó để đưa ra lời khuyên chung về thời điểm tốt nhất nên xóa người quan sát khỏi trung tâm thông báo, bởi vì điều đó phụ thuộc:

• Về trường hợp sử dụng của bạn (Những thông báo nào được quan sát thấy? Khi nào chúng được gửi?)
• Việc thực hiện của người quan sát (Khi nào nó sẵn sàng nhận thông báo? Khi nào nó không còn sẵn sàng?)
• Thời gian tồn tại dự định của người quan sát (Nó có bị ràng buộc với một số đối tượng khác, chẳng hạn như một khung nhìn hay bộ điều khiển khung nhìn?)
• ...

Vì vậy, lời khuyên chung tốt nhất mà tôi có thể đưa ra: để bảo vệ ứng dụng của bạn. chống lại ít nhất một thất bại có thể xảy ra, hãy removeObserver:nhảy vào dealloc, vì đó là điểm cuối cùng (trong vòng đời của đối tượng), nơi bạn có thể làm điều đó một cách sạch sẽ. Điều này không có nghĩa là: "chỉ cần trì hoãn việc loại bỏ cho đến khi deallocđược gọi, và mọi thứ sẽ ổn". Thay vào đó, hãy loại bỏ người quan sát ngay khi đối tượng không còn sẵn sàng (hoặc được yêu cầu) để nhận thông báo . Đó là thời điểm chính xác. Thật không may, không biết câu trả lời cho bất kỳ câu hỏi nào được đề cập ở trên, tôi thậm chí không thể đoán được thời điểm đó sẽ là khi nào.

Bạn luôn có thể an toàn removeObserver:một đối tượng nhiều lần (và tất cả, trừ lần gọi đầu tiên với một người quan sát nhất định sẽ là nops). Vì vậy: hãy nghĩ đến việc làm lại (một lần nữa) deallocđể chắc chắn, nhưng trước hết: hãy làm nó vào thời điểm thích hợp (được xác định bởi trường hợp sử dụng của bạn).

Lưu ý: Điều này đã được thử nghiệm và hoạt động 100%

Nhanh

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)
    }
}

PresentedViewController

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

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

Objective-C

Trong đó iOS 6.0 > version, tốt hơn là loại bỏ trình quan sát trong viewWillDisappearvì viewDidUnloadphương pháp không được dùng nữa.

 [[NSNotificationCenter defaultCenter] removeObserver:observerObjectHere];

Sẽ tốt hơn nhiều lần remove observerkhi chế độ xem đã bị xóa khỏi 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];
    }
}

PresentedViewController

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

Kể từ iOS 9, không còn cần thiết phải xóa người quan sát.

Trong OS X 10.11 và iOS 9.0 NSNotificationCenter và NSDistributedNotificationCenter sẽ không còn gửi thông báo cho những người quan sát đã đăng ký mà có thể được phân bổ.

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

Nếu trình quan sát được thêm vào bộ điều khiển chế độ xem , tôi thực sự khuyên bạn nên thêm viewWillAppearvà xóa nó vào viewWillDisappear.

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

Nói chung tôi đưa nó vào deallocphương pháp.

Trong deinit sử dụng nhanh chóng vì dealloc không có sẵn:

deinit {
    ...
}

Tài liệu Swift:

Một deinitializer được gọi ngay lập tức trước khi một cá thể lớp được phân bổ. Bạn viết deinitializers với từ khóa deinit, tương tự như cách intializers được viết với từ khóa init. Deinitializers chỉ có sẵn trên các loại lớp.

Thông thường, bạn không cần thực hiện dọn dẹp thủ công khi các phiên bản của bạn được phân bổ. Tuy nhiên, khi bạn đang làm việc với các tài nguyên của mình, bạn có thể cần phải tự mình thực hiện thêm một số thao tác dọn dẹp. Ví dụ: nếu bạn tạo một lớp tùy chỉnh để mở một tệp và ghi một số dữ liệu vào nó, bạn có thể cần phải đóng tệp trước khi cá thể lớp được phân bổ.

* chỉnh sửa: Lời khuyên này áp dụng cho iOS <= 5 (thậm chí ở đó bạn nên thêm viewWillAppearvà xóa viewWillDisappear- tuy nhiên lời khuyên áp dụng nếu vì lý do nào đó bạn đã thêm người quan sát vào viewDidLoad)

Nếu bạn đã thêm trình quan sát trong, viewDidLoadbạn nên xóa nó trong cả deallocvà viewDidUnload. Nếu không, bạn sẽ phải thêm nó hai lần khi viewDidLoadđược gọi sau viewDidUnload(điều này sẽ xảy ra sau cảnh báo bộ nhớ). Điều này không cần thiết trong iOS 6, nơi viewDidUnloadkhông được dùng nữa và sẽ không được gọi (vì các chế độ xem không còn được tải tự động nữa).

Theo ý kiến ​​của tôi, đoạn mã sau không có ý nghĩa trong ARC :

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

Trong iOS 6 , cũng không có ý nghĩa gì khi xóa người quan sát trong viewDidUnloadvì nó hiện đã không còn được dùng nữa.

Tóm lại, tôi luôn làm điều đó trong viewDidDisappear. Tuy nhiên, nó cũng phụ thuộc vào yêu cầu của bạn, giống như @Dirk đã nói.

Tôi nghĩ rằng tôi đã tìm thấy một câu trả lời đáng tin cậy ! Tôi đã phải, vì các câu trả lời ở trên là mơ hồ và có vẻ mâu thuẫn. Tôi đã xem qua Sách dạy nấu ăn và Hướng dẫn lập trình.

Đầu tiên, kiểu addObserver:in viewWillAppear:và removeObserver:in viewWillDisappear:không phù hợp với tôi (tôi đã thử nghiệm nó) vì tôi đang đăng thông báo trong bộ điều khiển chế độ xem con để thực thi mã trong bộ điều khiển chế độ xem mẹ. Tôi sẽ chỉ sử dụng kiểu này nếu tôi đang đăng và nghe thông báo trong cùng một bộ điều khiển chế độ xem.

Câu trả lời mà tôi sẽ dựa vào nhiều nhất, tôi đã tìm thấy trong Hướng dẫn lập trình iOS: Big Nerd Ranch lần thứ 4. Tôi tin tưởng những người của BNR vì họ có trung tâm đào tạo iOS và họ không chỉ viết một cuốn sách dạy nấu ăn khác. Nó có lẽ là lợi ích tốt nhất của họ để được chính xác.

BNR ví dụ một: addObserver:trong init:, removeObserver:trongdealloc:

BNR ví dụ hai: addObserver:in awakeFromNib:, removeObserver:indealloc:

… Khi loại bỏ người quan sát trong dealloc:họ không sử dụng[super dealloc];

Tôi hy vọng điều này sẽ giúp người tiếp theo…

Tôi đang cập nhật bài đăng này vì Apple hiện đã gần như hoàn toàn đi kèm với Storyboards nên những điều đã đề cập ở trên có thể không áp dụng cho mọi trường hợp. Điều quan trọng (và lý do tôi thêm bài đăng này ngay từ đầu) là hãy chú ý xem bạn viewWillDisappear:có được gọi hay không. Nó không dành cho tôi khi ứng dụng vào nền.

Câu trả lời được chấp nhận không an toàn và có thể gây rò rỉ bộ nhớ. Vui lòng hủy đăng ký trong dealloc nhưng cũng hủy đăng ký trong viewWillDisappear (đó là điều tất nhiên nếu bạn đăng ký trong viewWillAppear) .... ĐÓ LÀ NHỮNG GÌ TÔI ĐÃ LÀM BẤT CỨ VÀ NÓ LÀM VIỆC TUYỆT VỜI! :)

Điều quan trọng cần lưu ý là cũng viewWillDisappearđược gọi khi bộ điều khiển chế độ xem hiển thị một UIView mới. Đại biểu này chỉ đơn giản chỉ ra rằng chế độ xem chính của bộ điều khiển chế độ xem không hiển thị trên màn hình.

Trong trường hợp này, việc phân bổ thông báo trong viewWillDisappearcó thể không thuận tiện nếu chúng tôi đang sử dụng thông báo để cho phép UIview giao tiếp với bộ điều khiển chế độ xem chính.

Như một giải pháp, tôi thường loại bỏ trình quan sát theo một trong hai phương pháp sau:

- (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];
}

Vì những lý do tương tự, khi tôi đưa ra thông báo lần đầu tiên, tôi cần tính đến thực tế là bất kỳ lúc nào chế độ xem xuất hiện phía trên bộ điều khiển thì viewWillAppearphương thức đó sẽ được kích hoạt. Điều này sẽ tạo ra nhiều bản sao của cùng một thông báo. Vì không có cách nào để kiểm tra xem một thông báo đã hoạt động hay chưa, tôi sẽ khắc phục sự cố bằng cách xóa thông báo trước khi thêm nó:

- (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

Có hai trường hợp sử dụng thông báo: - chúng chỉ cần thiết khi bộ điều khiển chế độ xem trên màn hình; - chúng luôn cần thiết, ngay cả khi người dùng mở màn hình khác quá dòng.

Đối với trường hợp đầu tiên, nơi chính xác để thêm và xóa người quan sát là:

/// 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)
}

đối với trường hợp thứ hai, cách đúng là:

/// 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)
    }
}

Và không bao giờ đưa removeObservervào deinit{ ... }- đó là một MISTAKE!

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)
}