Swift3 UIAlertControllerの重複処理について

Question

###前提・実現したいこと
swift3 にてUIAlertControllerでメッセージの表示をしているのですが、
メッセージが重複した際の処理で困っています。

すでにメッセージが表示されている状態で別のメッセージを表示してしまうと以下のエラーが出てしまうので
"すでになんらかのメッセージが表示されている場合は、次のメッセージを表示しない。"
という処理をしたいと思います。

しかし、コードが古いのか、根本的に間違っているのかうまく行きません。
何か良い方法はあるでしょうか。

よろしくお願いします。
###発生している問題・エラーメッセージ

Warning: Attempt to present ... while a presentation is in progress!

###該当のソースコード

// 外部からこのメッセージ表示クラスを呼ぶ。
class Pop {

    /// - Parameters:
    ///   - uIViewController: 呼び出し元で'self'を引数に指定します。
    func message(_ uIViewController: UIViewController) {

        // メッセージが表示されているかを判断したい。
        // しかし、全て elseになってしまう。
        if uIViewController.parent is AlertController {
            return
        }
        // メッセージが表示されていなかったら表示
        else {

                  let alert: UIAlertController = UIAlertController(title: "エラーです。", message: text , preferredStyle:  UIAlertControllerStyle.alert)
                  let cancelAction: UIAlertAction = UIAlertAction(title: "確認", style: UIAlertActionStyle.cancel) { action in }
                  alert.addAction(cancelAction)
                  uIViewController.present(alert, animated: true, completion: nil)
        }
    }
}

Answer 1

以下の様にして試してみてください。

// 外部からこのメッセージ表示クラスを呼ぶ。
class Pop {
    
    /// - Parameters:
    ///   - uIViewController: 呼び出し元で'self'を引数に指定します。
    func message(_ uIViewController: UIViewController) {
        
        guard var topViewController = UIApplication.shared.keyWindow?.rootViewController else {
            return
        }
        
        while let presentedViewController = topViewController.presentedViewController {
            topViewController = presentedViewController
        }
        
        if let  _ = topViewController as? UIAlertController {
            
            // アラートがすでに出ていたら再度出さない
            return
        }
        
        
        let alert: UIAlertController = UIAlertController(title: "エラーです。", message: "テキスト" , preferredStyle:  UIAlertControllerStyle.alert)
        let cancelAction: UIAlertAction = UIAlertAction(title: "確認", style: UIAlertActionStyle.cancel) { action in }
        alert.addAction(cancelAction)
        uIViewController.present(alert, animated: true, completion: nil)
    }
}

Comments

[P-san]

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

しかし、`return`がかからず、メッセージの表示処理が何回も実行されてしまいます。

Answer 2

UIの状態を調べにいくより、変数で状態を管理するほうが設計の筋が良いと思います。

• Pop が管理用の変数をもつ。たとえば var isPresented: Bool とか。
• 表示処理の最初にこれをtrueにして、表示直後（アクションの中)とかでこれをfalseにする。
• 管理用の変数isPresentedがtrueなら、表示処理はおこなわない。

厳密には、isPresentedの読み書きはGCDとかつかって排他制御をしたほうがいいかもしれませんが、通常は上記で問題出ないでしょう。

Comments

[P-san]

上手く処理できました。
ありがとうございます。

[acevif]

????