警惕细节,NavigationController的delegate代理方法注意问题

今天在调试项目时,发现在成功push某个controller后,再pop返回到上一层,然后再pop到上上层的瞬间crash,报内存错误,但是Xcode没有Log任何错误信息.几经调试,请教了经验丰富的同事.找到如下原因:

第三层controller中由于某种需求而实现了NavigationController的代理方法,在ViewDidLoad中设置代理: self.navigationController.delegate = self; 问题就在这里,当pop以后,self.navigationController已经被销毁,而 self.navigationController.delegate却依然存在,此处属于内存泄露.

解决办法:

在viewWillAppear中设置代理,在viewDidDisappear时必须清空代理,即设置 self.navigationController.delegate = nil;.

注意,不能在viewWillDisappear中执行清空代理的操作,因为viewWillDisappear的调用顺序在navigation代理方法之前,而viewDidDisappear调用顺序在navigation代理方法之后.

- (void)viewWillAppear:(BOOL)animated {

    [super viewWillAppear:animated];

    self.navigationController.delegate = self;

}

//注意与navigation代理方法调用的先后顺序问题 如果是viewWillDisappear 则先于navigation代理调用

- (void)viewDidDisappear:(BOOL)animated {

    [super viewDidDisappear:animated];

    self.navigationController.delegate = nil;

}