navigationController 和dismissViewControllerAnimated 组合的错误使用

一、前言

产品的需求：导航控制器中存在RootViewController，push进TSMiddleViewController，再弹出PresentViewController，点击PresentViewController，关闭PresentViewController并push进Page2ViewController。

image.png

初始代码如下
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，具体看下图和代码注释。

image.png

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

五、总结

其实问题并不复杂，但是在实际开发过程中，头脑要清晰，对生命周期的理解要深刻，尤其在复杂产品需求背景下对需求的提炼和归纳，并行程有效的抽象设计，这就是一个比较好的实践过程。千万不能被复杂的产品需求所干扰内心。