上次处理了一个因为控制器被某种强引用而无法释放掉,系统没用帮我们释放掉此对象的这块内存的问题,控制器在pop出栈的时候也没有调用到dealloc方法,当时一时间没有定位出这个强引用的类型以及具体位置,赶开发进度的需要对控制器做了一个特殊处理,也算是解决了当时的需求(调用pop方式返回,但是当前控制器没有被释放的解决方式),但是强引用没定位出来却在心里留下了一根刺...今天专门抽出时间来定位这个问题.

控制器没有被释放大致原因有几点:

1.viewController中NSTimer没有被销毁
2.viewController中的代理不是weak属性
3.viewController中block的循环引用

1,2很简单被排除掉了,剩下的还是block的循环引用问题,这个需要一点点的定位:

Paste_Image.png

NSLogDealloc是一个宏定义:#define NSLogDealloc NSLog( @"<< %@ : dealloc done >>", [[NSString stringWithUTF8String:__FILE__] lastPathComponent]),只要正常释放了我们都能在控制台打印出<< XXConfirmOrderController.m : dealloc done >>

这个类里的结构很简单, 视图及数据的处理都是在
-(void)viewDidLoad{ }
方法内处理,由图片我们看出,我们只需要分别追从排除

//UI
[self.view addSubview:self.tableView];
[self.view addSubview:self.bottomView];
//Data
[self loadData];

以及3各通知就好.

通过注释排除的方式将
[self.view addSubview:self.tableView]; [self loadData];
很容易的排除掉了,只有当[self.view addSubview:self.bottomView];打开的时候,控制器没有被正常释放掉.ok,问题出在这里. 定位到self.bottomView,跳入其懒加载查看了好几遍,终于找到了问题:

Paste_Image.png

后面代码较长也没问题就不贴了.看这里代码好像都已经做好了循环引用的处理了,block内所有用到self的地方都做了弱引用处理,但是还是执行不到dealloc方法,一点点追踪问题,最后将WEAKHUD 提到block之外时,就能执行到dealloc方法:

Paste_Image.png

于是点进去WEAKHUD一看究竟,最后发现这个宏定义坑爹的写了:

#define WEAKHUD __weak MBProgressHUD *weakHud = [MBProgressHUD showMessage:@"加载中..." toView:self.view];最后引用的是self.view, 还是TMD强引用!!!!!!😂😂😂😂😂😂😂😂###

OK,问题找到了.

这里归根结底还是触犯了block的使用规则:只有当block直接或间接的被self持有时，就需要weak self。#####

这里在做按钮点击事件处理时,从self.bottomView内甩出来一个block回调
[_bottomView submittBtnPressed:^(UIButton *btn) { }];
self对bottomView是毫无疑问的强引用@property(nonatomic, strong) XXSubmitBottomView *bottomView;跳入XXSubmitBottomView内,里面
@property(nonatomic, copy)BtnPressedBlock block; -(void)submittBtnPressed:(BtnPressedBlock)block;
毫无疑问的bottomView对block也是强引用,因此self.bottomView.block间,self间接持有了block,就产生了间接的循环引用!!!

只要问题找到了,解决问题就很简单了:
其实我们这里压根用不到去弱引用WEAKHUD

hud放在block内执行时正确的,放在外面反而错误,那样在按钮点击之前就会有hud一直在转...
MBProgressHUD是在此处生成了一个对象hud,hud并没有被self所持有,hud和block之间也没有互相引用,所以只需要简单去创建一个hud就好了

Paste_Image.png

Paste_Image.png

dealloc终于被执行到.
这个刺终于拔出来了...