Objective-C: จะลบผู้สังเกตการณ์สำหรับ NSNotification ได้ที่ไหน

ฉันมีคลาส C วัตถุประสงค์ ในนั้นฉันสร้างเมธอด init และตั้งค่า 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%

รวดเร็ว

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

วัตถุประสงค์ -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];
    }
}

PresentedViewController

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

หากสังเกตจะถูกเพิ่มไปยังตัวควบคุมมุมมองผมขอแนะนำให้เพิ่มในและถอดมันออกมาในviewWillAppearviewWillDisappear

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

โดยทั่วไปฉันใส่ลงในdeallocวิธีการ

ในการใช้ deinit อย่างรวดเร็วเนื่องจาก dealloc ไม่พร้อมใช้งาน:

deinit {
    ...
}

เอกสาร Swift:

deinitializer ถูกเรียกใช้ทันทีก่อนที่จะเลิกจัดสรรอินสแตนซ์ของคลาส คุณเขียน deinitializers ด้วยคีย์เวิร์ด deinit คล้ายกับการเขียน intializers ด้วยคีย์เวิร์ด init Deinitializers มีให้บริการในประเภทคลาสเท่านั้น

โดยทั่วไปคุณไม่จำเป็นต้องล้างข้อมูลด้วยตนเองเมื่อยกเลิกการจัดสรรอินสแตนซ์ของคุณ อย่างไรก็ตามเมื่อคุณทำงานกับทรัพยากรของคุณเองคุณอาจต้องดำเนินการล้างข้อมูลเพิ่มเติมด้วยตัวคุณเอง ตัวอย่างเช่นหากคุณสร้างคลาสแบบกำหนดเองเพื่อเปิดไฟล์และเขียนข้อมูลบางอย่างลงไปคุณอาจต้องปิดไฟล์ก่อนที่จะยกเลิกการจัดสรรอินสแตนซ์ของคลาส

* แก้ไข: คำแนะนำนี้ใช้กับ 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:in awakeFromNib:, removeObserver:indealloc:

…เมื่อนำผู้สังเกตการณ์ออกจากที่dealloc:พวกเขาไม่ได้ใช้[super dealloc];

หวังว่านี่จะช่วยคนต่อไป ...

ฉันกำลังอัปเดตโพสต์นี้เนื่องจากตอนนี้ Apple ได้เลิกใช้ Storyboards ไปเกือบหมดแล้วดังนั้นสิ่งที่กล่าวมาข้างต้นอาจใช้ไม่ได้กับทุกสถานการณ์ สิ่งที่สำคัญ (และเหตุผลที่ฉันเพิ่มโพสต์นี้ในตอนแรก) คือให้ความสนใจหากคุณ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)
}