Alertビューが消えて画面が切り替わってしまうのを避けたい

Question

実現したいこと

現在開発しているアプリの AppDelegate.swift ファイルの applicationDidBecomeActive に特定状況下においてはAlertだけを表示する画面を表示してアプリ画面に行かないよいうにする実装をしています。Alertは問題なく表示できるのですが、Alert表示直後自動でAlertが消え、アプリの普通の画面が立ち上がってしまう状態です。

コードレビューを受けapplicationDidBecomeActive内は長くしたくないということになり、別のForceUpdateというstructと、その中に作ったactivate()というファンクションを呼ぶだけの作りで現在実装しています。

問題の原因はSplashViewControllerなのではないかと考えています。元々こちらの問題を解決できなかったのでSplashViewController.swift と AppDelegate.swift に同じようなコードを書いてSplashViewControllerにDispatchGroup()を使って記述して問題が起こらないよう無理やり解決していました。

ですがコードレビューでSplashViewController.swiftには何も書かずにAppDelegate.swiftにだけ記述するだけにしたいという流れになりました。

下記ソースコードをご覧いただければお分かりになる通り、Alertの表示はメインスレッドで出すように実装しています。

Alert表示後に自動で次の画面が表示されてしまうのを回避したいです。どのようにすれば解決できそうでしょうか？皆様のお力添えいただけますと幸いです。

class SplashViewController: UIViewController {

    //中略

    override func viewDidLoad() {
        super.viewDidLoad()
        let isUserLoggedIn = UserDefaults.userData != nil && UserDefaults.userData2 != nil
        let openRootViewController = {
            DispatchQueue.main.async {
                AppDelegate.share()?.setRootViewController()
            }
        }
        if isUserLoggedIn {

          //中略

          openRootViewController()
          
        } else {
            openRootViewController()
        }
    }
}

func applicationDidBecomeActive(_ application: UIApplication) {
    ForceUpdate.activate()
}

struct ForceUpdate {
    
    static func activate() {
        
        let appDelegate = AppDelegate()
        let deviceAppBuild = Int(Bundle.main.infoDictionary?["CFBundleVersion"] as! String)
        let remoteConfig = RemoteConfig.remoteConfig()
        let settings = RemoteConfigSettings()
        settings.minimumFetchInterval = 0
        remoteConfig.configSettings = settings
        remoteConfig.fetch(withExpirationDuration: 0) { (status, error) in
            if status == .success {
                //RemoteConfigからデータがとれた場合
                remoteConfig.activate(completionHandler: { (error) in
                    let minimumBuild = Int(remoteConfig.configValue(forKey: "minimum_build").numberValue ?? 0)
                    //端末側のアプリのビルド番号がremoteConfigのビルド番号より古かったらアップデートを促す
                    if deviceAppBuild! < minimumBuild {
                        DispatchQueue.main.async {
                            
                            let alert = UIAlertController(title: Localizable.ForceUpdate.gettingLatestVersion.localized(), message: Localizable.ForceUpdate.availableLatestVersion.localized(), preferredStyle: .alert)
                            let alertAction = UIAlertAction(title: Localizable.ForceUpdate.getLatestVersion.localized(), style: .default, handler: { (action) in
                               let url = URL(string: Constants.URLs.appStoreUrl)!
                               UIApplication.shared.openURL(url)
                            })
                            alert.addAction(alertAction)
                            appDelegate.window = UIWindow(frame: UIScreen.main.bounds)
                            let storyboard = UIStoryboard(name: "Main", bundle: nil)
                            let alertViewController = storyboard.instantiateInitialViewController()
                            appDelegate.window?.rootViewController = alertViewController
                            appDelegate.window?.makeKeyAndVisible()
                            alertViewController!.present(alert, animated: true)    
                        }
                    } 
                })
            }
        }
    }
}

追記

func setRootViewController() {
    
    if !UserDefaults.isDidShowLandingPage {
        replaceRootViewController(by: LandingPageViewController.getXibVC())
        return
    }
    if UserDefaults.userData != nil && UserDefaults.userData2 != nil {
        setContainerToRoot()
    } else {
        setLoginVCToRoot()
    }
}

補足情報

Swift 5.0.1
Xcode 11.1

Comments

[takabosoft]

現在の状態と、期待値がよくわかりません・・・。
自動で次の画面とありますが、次の画面て何でしょう？
また、コードレビューを受けられる立場の方がここのサイトで質問するというのも変な話です。

[takabosoft]

instantiateInitialViewController()で生成した初期のビューコントローラーをappDelegate.window?.rootViewControllerにセットしているのですから、アラートが出つつもアプリの初期画面に遷移するでしょうが、それをアラートだけにしたい的な話なんですかね？

あ、初期画面が裏に出ているのは良くて、アラートが勝手に消える事を防ぎたいという意味ですかね。

[swifty]

ご回答ありがとうございます。大変嬉しく思います。

レビューしてくださった方が元となるアプリを開発したわけではなく、現在他の開発を進められているので、レビューはしていただけましたがこちらの問題は自分で修正する必要があるので質問させていただいております。

〈現在〉
真っ白い画面の上にアラート画面が出てきて0.1,2秒ほど後、アプリの初期画面に遷移してしまいます。

〈次の画面〉
こちらは見た目的にはアプリの初期表示画面なのですが、その前にログインチェックをしている部分があってそれがSplashViewController内なのでこちらが今回の場合は重要なのではないかと考えています。

〈期待値〉
真っ白い画面の上にアラート画面だけが出てきて(ここまでは現在も再現済)その画面で止まるという状態にしたいです。アラートが勝手に消えないようにしたいです。

お分かりになりますでしょうか？スレッドの問題なのではないかと思っているのですが自分では解決できかねる状態なので質問させていただきました。引き続きご教授いただけますと幸いです。

[takabosoft]

let storyboard = UIStoryboard(name: "Main", bundle: nil)
let alertViewController = storyboard.instantiateInitialViewController()
appDelegate.window?.rootViewController = alertViewController
appDelegate.window?.makeKeyAndVisible()

↑のコードで出るのが白い画面なんでしょうか？
白い画面ってなんですか？
何も処理がない画面なんでしょうか？

[swifty]

おっしゃる通りです。上記のコードで出る画面(ただのアラート画面だけを表示した画面)を白い画面と書いています。

[takabosoft]

その白い画面のビューコントローラは空っぽで、どこにも遷移するコードが無いんですよね？

それなのに、遷移していまうというのであれば、
setRootViewController()関数といった強制的にrootViewControllerを書き換えてコード遷移している場所にprintでログでもだして、アラートを出した後にそういったコードが動いてないか確かめてみては？

[swifty]

おっしゃる通りです。

現在setRootViewController()が動いてしまっている状態なのでAlertを出すときはそれをストップさせたいです。

setRootViewController()はSplashViewControllerのDispatchQueue.main.asyncで動作していて、
アラートはForceUpdateのDispatchQueue.main.asyncで動作している状態で、どちらもメインスレッドでの動きなのですが、Alert出す動きがあったときはsetRootViewController()は動かないといった制御ができればいいのかなと思い、DispatchSemeperやDispatchGroupあたりでできそうだなと思いつつ、解決できていない状況です...

[takabosoft]

えーと、そのAlert出す動きがあったときにsetRootViewController()が動いてしまっているというのは確定情報なんでしょうか？

[swifty]

確定です。

Answer 1

なんとなく話が見えてきましたので、回答しますが、
setRootViewController()の中のコードで
現在の「appDelegate.window?.rootViewController」の型を調べて、それが「白い画面（のビューコントローラ）」だったら処理を中断すれば良いでしょう。

Comments

[swifty]

ご回答ありがとうございます！

なるほど。早速型を調べる実装をしてみました。

appDelegate.window?.rootViewControllerの型を調べるとAlertViewController(真っ白い画面)が返ってきて欲しい場面でもいつでもSplashViewControllerが返ってきてしまう状態です。

window!.rootViewController

と呼び出しているのですが呼び出し方がよくなかったりするでしょうか？

[takabosoft]

setRootViewController()のコードは載せれませんか？

まあ別にやり方はなんでもいいです。
var hoge = false
とか適当に変数を用意しておいて、
アラート出す必要性が出たらhoge = trueにして、
setRootViewController()の中でif hoge { return }とかでも。

[swifty]

ご回答ありがとうございます。

setRootViewController()の内容を追記させていただきました。
なるほどそれで1度チャレンジしてみようと思います。

[takabosoft]

あー、ひょっとすると
activate関数の一番最初の
> let appDelegate = AppDelegate()
は
> let appDelegate = AppDelegate.share()!
じゃないですかね？

ここが間違っていたのでappDelegate.window?.rootViewControllerで白い画面が取れなかったのではないかと。

[swifty]

activate関数の一番最初の

let appDelegate = AppDelegate()

を

let appDelegate:AppDelegate = UIApplication.shared.delegate as! AppDelegate

にしてご教授いただいた変数を用意しておいて、
アラート出す必要性が出たら変数に値入れてパスするという作戦で実装したら無事解決できました！！

ご丁寧にアドバイスくださりありがとうございました！助かりました！！

[takabosoft]

その
let appDelegate = AppDelegate()
と
let appDelegate:AppDelegate = UIApplication.shared.delegate as! AppDelegate
とで何が違うかは理解されていますかね？

[swifty]

お世話になっております。今回の件で理解できました。ありがとうございます！