【Swift】Firebaseからデータを取得する処理の順番がおかしい

Question

Firebaseから文字列のデータを取得して、それを配列に格納し、カスタムセル上のUILabelに格納する処理を作ろうとしています。

当初は、ViewDidLoad内でデータを取得すれば、自然にカスタムセルで正常にデータが表示されると思っていました。

しかし、下記のコードで各処理の順番を出力したら、下記のような結果になりました。
↓
「結果」
1
2
4
6
3
3
5

どうしてviewDidLoad内にあるはずのloadNotification（）（データを取得する関数）は、最初にスルーされて、後から実行されるのでしょうか？

データ取得が完了するまで次の処理には行かないようにするにはどうすればいいのでしょうか？

    override func viewDidLoad() {
        super.viewDidLoad()

        print("1")
        loadNotification()
        print("6")

        tableView.register(UINib(nibName: "NotificationCell", bundle: Bundle.main), forCellReuseIdentifier: "NotificationCell")
        tableView.estimatedRowHeight = 65
        tableView.rowHeight = 65
        tableView.delegate = self
        tableView.dataSource = self
    }

    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        print("5")
        //セルの読みこみ
        let cell = tableView.dequeueReusableCell(withIdentifier: "NotificationCell", for: indexPath) as! NotificationCell
        //通知本文を表示
        cell.message.text = notificationArray[indexPath.row]["message"]!
        return cell
    }

    //お知らせを読み込む
    func loadNotification() {
        print("2")
        //初期化
        notificationArray = [[String:String]]()
        //データベース参照
        let ref = Database.database().reference(fromURL: "https://bookshare-b78b4.firebaseio.com/")
        //データを取得
        ref.child("Notification").child(userID).observe(.value) { (snap) in
            for data in snap.children {
                print("3")
                let snapdata = data as! DataSnapshot
                //辞書型に変換
                let item = snapdata.value as! [String:String]
                //配列に追加
                self.notificationArray.append(item)
            }
            //tableViewを更新
            self.tableView.reloadData()
        }
        print("4")
    }

Answer 1

これはFirebase Realtime Database を使い始めの方が一番初めにつまずくポイントです。

このポイントを押させてください。
Realtime Databaseでは直接値を参照することはできない
そんな馬鹿なと思われますが、実際できません。　これはそういうものなのであきらめてください。

ではどのように値を参照するかというと
0. 前回から値が更新されたときに呼び出される関数を登録する

1. 前回から値が更新されると、その値（のコピー）を使って登録した関数が呼び出される。

という段階を経ることになります。

質問者さんのコードのprint("3")の部分は 1) に当たります。
ここでは関数の登録だけで、関数は呼び出されません。
ただし、登録直後にネットワーク上の状態が確認され、前回から値が更新されているため関数が呼び出されます。
（最初の呼び出しなので前回はデータがないときと判断されます）
確認には少し時間がかかるため、実行は遅延されます。

また、この関数呼び出しは値の更新があるたび何度でも呼び出されますので、質問者さんのコードですと、値が更新されるたびに、表示が重複されることになりますので気を付けてください。

Comments

[nekokichi]

とてもためになる回答をくださりありがとうございます。

お手数ですが、２つの質問にお答えいただきたいです。

・上から順に実行されるような自然な処理順を実現するには、Dispatchqueueのような同期通信を使う必要があるのでしょうか？

・”質問者さんのコードですと、値が更新されるたびに、表示が重複されることになりますので気を付けてください。”についてですが、値が更新されるというのはデータベース参照先で値が更新されるということですか？　
・表示が重複させないためにできる対策はありますか？

どうかよろしくお願いします。

[MasakiHori]

あなたのいう「自然な処理順を実現する」ことに何の意味がありますか？　僕個人からすればこれが自然な処理順序です。　この場面でスレッドのブロックが必要ならそのアプリの設計が失敗しています。　どうしても上から順に動いて（いるように見えて）ほしいというのであれば、BrightFutures https://github.com/Thomvis/BrightFutures のようなFutureを使うことをお勧めします。

[nekokichi]

ここまで付き合っていただきありがとうございます。

流石に処理の構造を根本から変えようなんて考えてた自分がバカでした。

今後はコードの流れをしっかり把握できるように努めてまいります。