navigationController 和dismissViewControllerAnimated 组合的错误使用
一、前言
产品的需求：导航控制器中存在RootViewController，push进TSMiddleViewController，再弹出PresentViewController，点击PresentViewController，关闭PresentViewController并push进Page2ViewController。
初始代码如下
TSMiddleViewController.m
- (void)viewDidLoad {
[super viewDidLoad];
self.title = @"中间页";
self.present = [[PresentViewController alloc] init];
[self.navigationController presentViewController:self.present animated:YES completion:nil];
__weak typeof(self)weakSelf = self;
self.present.gotoBlock = ^{
NSMutableArray * mutArray = [NSMutableArray arrayWithArray:weakSelf.navigationController.viewControllers];
for (UIViewController *vc in weakSelf.navigationController.viewControllers) {
if ([vc isKindOfClass:weakSelf.class]) {
[mutArray removeObject:vc];
}
}
Page2ViewController *vc2 = [[Page2ViewController alloc] init];
[mutArray addObject:vc2];
[weakSelf.navigationController setViewControllers:mutArray];
// 【问题1模态框消失不了】
[weakSelf.navigationController dismissViewControllerAnimated:YES completion:nil];
};
}
PresentViewController.m
- (void)viewDidLoad {
[super viewDidLoad];
self.title = @"PresentViewController";
UITapGestureRecognizer *tap = [[UITapGestureRecognizer alloc] initWithTarget:self action:@selector(clickTap)];
[self.view addGestureRecognizer:tap];
}
- (void)clickTap
{
if (self.gotoBlock) {
self.gotoBlock();
}
}
Page2ViewController.m
- (void)viewDidLoad {
[super viewDidLoad];
self.title = @"模拟发布页";
[self addObserver:self forKeyPath:@"OBSERVER" options:NSKeyValueObservingOptionNew context:nil];
}
- (void)dealloc
{
[self removeObserver:self forKeyPath:@"OBSERVER"];
// 此处模拟实际中的崩溃场景【问题2】
}
二、现象
结合上面代码注释有两个问题
【问题1】：PresentViewController模态框消失不了
第一次执行了 [weakSelf.navigationController dismissViewControllerAnimated:YES completion:nil];
没有效果，再一次出现【问题2】
【问题2】：崩溃
*** Terminating app due to uncaught exception 'NSRangeException', reason: 'Cannot remove an observer <Page2ViewController 0x7ff158707f80> for the key path "OBSERVER" from <Page2ViewController 0x7ff158707f80> because it is not registered as an observer.'
造成的现象是在C中点击下一步,跳转到D时,点击第一次时没反应,再点击发生了崩溃.崩溃在D中的dealloc中
三、问题分析
【问题1】：PresentViewController模态框消失不了
原因是因为在dissmiss之前已经将导航堆栈数组里的当前页面给移除，并执行了setViewControllers方法，导致在执行[weakSelf.navigationController dismissViewControllerAnimated:YES completion:nil];时weakSelf.navigationController为nil，为什么？因为当前页面已经被从navigationController移除出去了。
【问题2】：崩溃
根据上面的崩溃log，xcode提示，observer还没有注册。这里不用应该立即断定xcode不会骗人，不要反复的思索“明明viewDidLoad已经注册过了啊~”，当机立断得出结论：即使是写了，代码不一定会执行，那么问题就变成了
【问题2-1】：为什么Page2ViewController的viewDidLoad没有被执行，也就是Page2ViewController没有被正常渲染。
只有一种情况，控制器的上下文有问题。回过头来看这几行代码
Page2ViewController *vc2 = [[Page2ViewController alloc] init];
[mutArray addObject:vc2];
[weakSelf.navigationController setViewControllers:mutArray];
[weakSelf.navigationController dismissViewControllerAnimated:YES completion:nil];
Page2ViewController实例化之后的vc2加到mutArray，再setViewControllers的时候出问题了，因为当前的
weakSelf.navigationController已经为nil了。为啥为nil？还是【问题1】中的原因。
四、解决办法
需要解决的问题其实就是在点下一步跳转到Page2ViewController的时候需要先dismiss掉PresentViewController，具体看下图和代码注释。
- (void)viewDidLoad {
[super viewDidLoad];
self.title = @"中间页";
// 从TSMiddleViewController跳转出去之后，需要将自己从导航堆栈中移除，
// 因为从下一个页面pop回来的时候，不希望再经过TSMiddleViewController
self.present = [[PresentViewController alloc] init];
// PresentViewController，模拟实际项目中的SDK，有一个block来通知业务层
// 需要业务层代码关闭模态框
[self.navigationController presentViewController:self.present animated:YES completion:nil];
__weak typeof(self)weakSelf = self;
self.present.gotoBlock = ^{
// 模态框触发事件，业务层需要关闭
if (!weakSelf.navigationController) {
// 防止极限情况，用户在模态框关闭之前多次点击
return;
}
[weakSelf.navigationController dismissViewControllerAnimated:YES completion:nil];
// dismiss模态框
NSMutableArray * mutArray = [NSMutableArray arrayWithArray:weakSelf.navigationController.viewControllers];
// 获取现有堆栈中的数组
for (UIViewController *vc in weakSelf.navigationController.viewControllers) {
if ([vc isKindOfClass:TSMiddleViewController.class]) {
[mutArray removeObject:vc];
// 从堆栈中移除TSMiddleViewController
}
}
Page2ViewController *vc2 = [[Page2ViewController alloc] init];
BOOL hasExsitPage2 = NO;
for (UIViewController *vc in mutArray) {
if ([vc isKindOfClass:Page2ViewController.class]) {
hasExsitPage2 = YES;
}
}
if (hasExsitPage2 == NO) {
[mutArray addObject:vc2];
// 原计划是要避免异常情况下多次加入相同的vc，后发现这种情况不可能发生
}
[weakSelf.navigationController setViewControllers:mutArray];
};
}
五、总结
其实问题并不复杂，但是在实际开发过程中，头脑要清晰，对生命周期的理解要深刻，尤其在复杂产品需求背景下对需求的提炼和归纳，并行程有效的抽象设计，这就是一个比较好的实践过程。千万不能被复杂的产品需求所干扰内心。
