swift4で処理を一回だけにしたい。

Question

#質問したいこと
現在swift４で、オリジナルアプリを作っています。そこで、ユーザーのログインが必要です。ログイン（サインアップ）はできるようになりましたが、その画面を表示するのを１回だけにしたいです。
#コード

import UIKit
import Firebase
import FirebaseAuth

class AccountViewController: UIViewController {
    var acount: FirebaseApp!
    
    @IBOutlet private weak var nameTextField: UITextField!
    @IBOutlet private weak var emailTextField: UITextField!
    @IBOutlet private weak var passwordTextField: UITextField!
    
    //didTapSignUpButtonの挙動を一回にしたい。
    @IBAction private func didTapSignUpButton() {
        let email = emailTextField.text ?? ""
        let password = passwordTextField.text ?? ""
        let name = nameTextField.text ?? ""
        
        Auth.auth().createUser(withEmail: email, password: password) { [weak self] result, error in
            guard let self = self else { return }
            if let user = result?.user {
                let req = user.createProfileChangeRequest()
                req.displayName = name
                req.commitChanges() { [weak self] error in
                    guard let self = self else { return }
                    if error == nil {
                        user.sendEmailVerification() { [weak self] error in
                            guard let self = self else { return }
                            if error == nil {
                                // 仮登録完了画面へ遷移する処理
                            }
                            self.showErrorIfNeeded(error)
                        }
                    }
                    self.showErrorIfNeeded(error)
                }
            }
            self.showErrorIfNeeded(error)
        }
        if (email == "" || password == "" || name == "") {
            let message = "全てのフォームに記入して下さい。"
            let alert = UIAlertController(title: nil, message: message, preferredStyle: .alert)
            alert.addAction(UIAlertAction(title: "OK", style: .default, handler: nil))
            present(alert, animated: true, completion: nil)
        }
    }
    
    private func showErrorIfNeeded(_ errorOrNil: Error?) {
        // エラーがなければ何もしません
        guard let error = errorOrNil else { return }
        
        let message = "エラーが起きました" // ここは後述しますが、とりあえず固定文字列
        let alert = UIAlertController(title: nil, message: message, preferredStyle: .alert)
        alert.addAction(UIAlertAction(title: "OK", style: .default, handler: nil))
        present(alert, animated: true, completion: nil)
    }
    
    override func viewDidLoad() {
        super.viewDidLoad()
        
        // Do any additional setup after loading the view.
    }

}

#やってみたこと
色々なホームページを見て、作ってみましたが、うまく動きませんでした。
特にこちら↓のページを参考にさせていただきました。
https://www.mani-labo.net/entry/swift-lazy-closure
こちらにあるコードをぜひ使いたいと思っているのですが、恥ずかしながら、どこに入れればいいのか分かりません。
他に何かいいサイトだったり、実際にコードを教えていただけるのであれば、教えて下さい。よろしくお願いします。

Comments

[hayabusabusash]

FirebaseのAuthの処理完了後に別の画面へ遷移するようですが、
現状どのあたりに問題があるのでしょうか？
ボタンを連続でタップされたても大丈夫なようにすれば良いのでしょうか？

また、lazyはviewWillAppear()などの複数回実行されるようなライフサイクルのイベント上で一度だけ実行したい時に使った方がいいと思います。
ボタンの処理にlazyを入れてしまうと、Authの処理がエラーになった際に再度タップしても処理が走らなくなってしまう気がします。

[Kaguya_4869]

次回アプリを開くときには、サインアップの画面を表示させないようにしたいです。
アプリを開くと毎回サインアップしなければならないのを避けたいです。

[hayabusabusash]

返信ありがとうございます！
なるほど、それならユーザーがサインアップ済みなのかどうかをどこかで保持しておいて
それを起動時に確認し、任意の画面に遷移させる方が適切だと思います。

Authの処理完了後に画面遷移をすると思いますが、
その際にUserDefaultsなどでサインアップ完了のフラグを保持しておいて、
AppDelegateでフラグを確認して、サインアップ後の画面を表示するようにするのが無難かなと思います。

この辺が参考になるかもしれません。
( http://blog.penginmura.tech/entry/2018/01/14/195323 )

[Kaguya_4869]

Auth.auth().signIn(withEmail: email, password: password) { (user, error) in
// [START_EXCLUDE]
self.hideSpinner {
if let error = error {
self.showMessagePrompt(error.localizedDescription)
return
}
self.navigationController!.popViewController(animated: true)
}
// [END_EXCLUDE]
}
こういう感じのコードを書けば良いですか？
違ったり、改良すべき点があれば教えていただけると助かります。

[hayabusabusash]

そうですね...
できればどんな画面の構成なっているか教えていただけないでしょうか？
例えば サインアップ画面 -> 完了画面 -> ホーム みたいな感じで、
どのように画面が遷移していくのかが知りたいです！

[Kaguya_4869]

サインアップ画面→ホームというように画面遷移しています。

[hayabusabusash]

長くなりそうなので回答の方に書かせていただきます。

Answer 1

次回アプリを開くときには、サインアップの画面を表示させないようにしたいです。

アプリを開くと毎回サインアップしなければならないのを避けたいです。

こちらに関して、
AppDelegateでサインアップ後かどうかの判断をして初回に表示する画面を変えるのはどうでしょうか？

まずAuthの処理内でサインアップが成功した場合にUseDefaultsでサインアップ済みのフラグを保持します。

Auth.auth().createUser(withEmail: email, password: password) { [weak self] result, error in
            guard let self = self else { return }
            if let user = result?.user {
                let req = user.createProfileChangeRequest()
                req.displayName = name
                req.commitChanges() { [weak self] error in
                    guard let self = self else { return }
                    if error == nil {
                        user.sendEmailVerification() { [weak self] error in
                            guard let self = self else { return }
                            if error == nil {
                                // 仮登録完了画面へ遷移する処理
                                // サインアップ完了のフラグを保持する
                                UserDefaults.standard.set(true, forKey: "appSignUpStatusKey")
                                UserDefaults.standard.synchronize()
                            }
                            self.showErrorIfNeeded(error)
                        }
                    }
                    self.showErrorIfNeeded(error)
                }
            }
            self.showErrorIfNeeded(error)
        }

保持したフラグをAppDelegate内で読み込み、
サインアップ済みかどうかでRootViewControllerを変更します。

以下ではRootViewControllerをサインアップのViewControllerにして、
サインアップ済みならホームのViewControllerをUINavigationControllerを使って表示するようにしています。(ViewControllerの初期化の部分は省いています。)

class AppDelegate: UIResponder, UIApplicationDelegate {

    var window: UIWindow?


    func application(_ application: UIApplication, didFinishLaunchingWithOptions launchOptions: [UIApplication.LaunchOptionsKey: Any]?) -> Bool {
        
        // MARK: RootViewController
        
        window = UIWindow(frame: UIScreen.main.bounds)

        // ルートはサインアップのViewController
        let rootVC = UINavigationController(rootViewController: /*AccountViewController*/)
        
        // サインアップ済みならホームのViewControllerを表示
        if UserDefaults.standard.bool(forKey: "appSignUpStatusKey") {
            rootVC.pushViewController(/*HomeViewController*/, animated: false)
        }

        window?.rootViewController = rootVC
        window?.makeKeyAndVisible()
        
        return true
    }
}

Comments

[Kaguya_4869]

すごい丁寧に教えてくださり、ありがとうございます！
その方法でやってみます！
もしまた何かあったら、お聞きするかもしれませんが、その時はよろしくお願いします。
ほんとうにありがとうございます！

[Kaguya_4869]

ベストアンサーにした後に聞くのは、申し訳ないですが、
AppDelegateの
let rootVC = UINavigationController(rootViewController: /*AccountViewController*/)
で、
Use of unresolved identifier'UINavigationController(rootViewController:)'
というエラーが出てきました。
どうすれば良いでしょうか？

[hayabusabusash]

/*AccountViewController*/のところにAccountViewControllerを初期化するコードを入れてください。
もしAccountViewControllerをStoryboardで作っているなら、

UIStoryboard(name: "AccountViewController", bundle: nil).instantiateInitialViewController() ?? UIViewController()

みたいな感じにすれば大丈夫だと思います。

[Kaguya_4869]

実行すると、
Could not find a storyboard named 'AccountViewController'
という理由で、落ちてしまいます。
名前はあっているのですが、どうしてか、分かりますか？

[hayabusabusash]

AccountViewController.storyboardではなく他の名前のStoryboardでAccountViewControllerを作っていませんか？
AccountViewControllerを作ったStoryboardの名前をnameに指定すればいいと思います。

AccountViewController.storyboardというファイルにAccountViewControllerだけを定義した前提でコードを書いてしまったのでわかりにくかったかもしれません。申し訳ありません。

もし複数のViewControllerを1つのStoryboardで作っているなら、
今回のように特定のViewControllerだけをStoryboardからインスタンス化したい場合
コードが冗長になりがちなので、一つのStoryboardに一つのViewControllerだけを定義することをお勧めします。
( 参考 https://qiita.com/execjosh/items/72d68e6965a745693f05 )