2回目以降のfor文をFirebaseの非同期処理が終わった後に実行したい(並行処理をさせたくない)

Question

こんにちは。お世話になります。

前提

SwfitとCloud　Firestoreを使って、ユーザーがお気に入りのレストランを登録し、それをGoogle Map上でシェアするアプリを開発しています。
Map上のマーカーのウィンドウをタップすると画面遷移し、レストランのレビューを見ることができるようにしています。レビュー画面では投稿したユーザーが誰かも分かるようにしています。

またCloud Firestoreにはルートコレクションが2つあります。
Storesコレクションのドキュメントを全て取得し、それをfor文で回しています。各ドキュメント内のuserIdの値を取得した後に、それと一致するUserコレクションのドキュメントを参照して、そのドキュメント内にあるuserNameとuserImageを取り出して、レビュー画面に投稿したユーザー名とアイコンを反映しています。

実現したいこと・起きている問題

レビュー画面に値を反映することはできているのですが、投稿したユーザーとは関係ないユーザーの値が反映されてしまうのでそれを解決したいです。

考えられる原因

アプリビルド時のログを見ると、下記のコードで示している箇所まではfor文で実行した通りに処理が走っているのですが、Firebaseとの通信(非同期処理?)後のクロージャがfor文とは関係ない順番で処理が実行されているためレストランを投稿したユーザーと関係無いユーザーの値がレビュー画面に反映させてしまっているのだと考えています。

下のシュミレータ画面とcloud firestoreの画像をみて頂ければ分かるかと思いますが、はっぱを投稿したユーザーは値が反映されているユーザーではありません。

シュミレータ画面

アプリビルド時のログ画面

Cloud Firestoreの構造

該当のソースコード

Firebaseと通信するモデルクラス↓

import Firebase

class GetStoreInfo {
    
    // MARK: - Properties
    let db = Firestore.firestore()
    
    // MARK: - Methods
    func getStoreInfo(completion: @escaping (_ snapShot: QuerySnapshot) -> Void) {
        let collRef = db.collection("Stores")
        collRef.getDocuments { [weak self] (snapShot, error) in
            if let _ = self {

                if error != nil {
                    print("could not get store data..")
                } else {
                    guard let unwrappedSnapShot = snapShot else { return }
                    completion(unwrappedSnapShot)
                }

            }
            
        }
    }
    
    func getPostUserInfo(userId: String, storeName: String, storeReview: String ,completion: @escaping  (_ snapShot: DocumentSnapshot) -> Void) {
        print("getPostUserInfoメソッドだよ")
        print(userId)
        let docRef = db.collection("Users").document(userId)
        // ここまではfor文で順番通りに動く⬆️

        // ここからfor文での順番通りに動かなくなり順番が狂う⬇️
        docRef.getDocument { [weak self] (snapShot, error) in
            if let _ = self {
                
                print("クロージャに入ったよ")
                guard let unwrappedSnapShot = snapShot else { return }
                print("アンラップしたよ")
                completion(unwrappedSnapShot)
                
            }
        }
    }
}

ViewController↓

import UIKit
import GoogleMaps

class MapViewController: UIViewController {
    
    // MARK: - Properties
    var mapView: MapView!
    var delegate: MapViewControllerDelegate?
    var storeModel: GetStoreInfo!
    var storeDetailViewControllers: [StoreDetailViewController] = []
    var storeName: [String] = []
    var storeReview: [String] = []
    var storeImage: [String] = []
    var count = 0
    var storeCount = 0
    
    // MARK: - Helper Functions
    override func viewDidLoad() {
        super.viewDidLoad()
        
        storeDetailViewControllers.removeAll()
        storeName.removeAll()
        storeReview.removeAll()
        storeName.removeAll()
        postUserNameArray.removeAll()
        postUserIconArray.removeAll()
　　　　　getStoresData()
    }
    
    func getStoresData() {
        
        print("start get stores data..")
        storeModel = GetStoreInfo()
        storeModel.getStoreInfo { [weak self] (snapShot) in
            if let self = self {
                // for文でドキュメントを順番に取得
                for document in snapShot.documents {
                    let storeData = document.data()
                    guard let latitude = storeData["latitude"] as? Double, let longitude = storeData["longitude"] as? Double, let storeName = storeData["storeName"] as? String, let storeImage = storeData["storeImage"] as? String, let storeReview = storeData["storeImpression"] as? String, let userId = storeData["userId"] as? String else { return }

                                
                    self.configureMakerInMap(latitude: latitude, longitude: longitude, storeName: storeName, count: self.count, storeReview: storeReview, storeImage: storeImage, userId: userId)
                    self.count += 1
                            }
            }
            
            
        }
        
    }
    
    func configureMakerInMap(latitude: Double, longitude: Double, storeName: String, count: Int, storeReview: String, storeImage: String, userId: String) {
        
        print("configure maker..")
        let position = CLLocationCoordinate2DMake(latitude, longitude)
        let marker = GMSMarker(position: position)
        marker.title = storeName
　　　　 // markerの識別子
        marker.identifier = count
        marker.map = mapView.mapView
        // for文で順番に取得した値を配列に入れる
        self.storeName.append(storeName)
        self.storeReview.append(storeReview)
        self.storeImage.append(storeImage)
        print(userId)
        self.storeModel.getPostUserInfo(userId: userId, storeName: storeName, storeReview: storeReview) { [weak self] (snapShot) in
            if let self = self {
　　　　　　　　　// クロージャから出てきたこの場所で順番がおかしくなる
                let userInfo = snapShot.data()
                print("0")
                print("出てきたよ")
                
                guard let postUserName = userInfo!["userName"] as? String, let postUserIcon = userInfo!["userImage"] as? String else { return }
                print(postUserName)
                print(postUserIcon)
                
　　　　　　　　　// 取得したレストランの情報とユーザーの情報を、引数で渡してレビュー画面を作成
                self.configureViewController(storeName: self.storeName[self.storeCount], storeReview: self.storeReview[self.storeCount], storeImage: self.storeImage[self.storeCount], count: self.count, postUserName: postUserName, postUserIcon: postUserIcon)
                self.storeCount += 1
            }
        }
        
        
    }
    
    func configureViewController(storeName: String, storeReview: String, storeImage: String, count: Int, postUserName: String, postUserIcon: String) {
        
        storeDetailViewControllers.append(StoreDetailViewController(storeName: storeName, storeReview: storeReview, storeImage: storeImage, postUserName: postUserName,  postUserIcon: postUserIcon,count: count))
        
    }

}

// Markerに識別子を持たせるための変数
extension GMSMarker {
    var identifier: Int {
        set(identifier) {
            self.userData = identifier
        }

        get {
           return self.userData as! Int
       }
   }
}

// MARK: Delegate
extension MapViewController: GMSMapViewDelegate {
    
    func mapView(_ mapView: GMSMapView, didTapInfoWindowOf marker: GMSMarker) {
        print(marker.identifier)
        let navStoreDetailViewController = UINavigationController(rootViewController: storeDetailViewControllers[marker.identifier])
        navStoreDetailViewController.modalPresentationStyle = .fullScreen
        present(navStoreDetailViewController, animated: true, completion: nil)
        print("did tap info window of maker..")
    }
    
}

試したこと

クロージャについて調べていたら、weak selfというのが出てきたのでコードを書いてみたのですが恐らく関係ありませんでした。
Firebaseとの非同期通信まで終わった後に、次のfor文内の処理が動くようにすれば解決できると思うのですが、調べても出てきませんでした。(そもそもそんなやり方が無いのかもしれませんが)

また、Firebase登録されているレストランを一つだけにしてみたらfor文は一度しか動かないので、投稿したユーザーは一致しました。逆にFirebase登録されているレストランを増やすほど、投稿したユーザーとは関係ないユーザーがばらばらに表示されてしまいます。

説明が分かりにくかったらすみません。
解決方法が分かる方がいましたら、ご回答お願いします。
また別の方法でやりたいことが実装できるのであれば、その方法を教えて頂きたいです。

追記
調べていたらforEach文というのが出てきたので、それを試してみましたがダメでした。
参考にしたサイト

補足情報（FW/ツールのバージョン）

xcode 11.3.1

Answer 1

あらかじめ getPostUserInfo するという前提だと、どうしても結果は非同期になりますから、結果の格納方法を工夫することになると思います。つまり、storeDetailViewControllers を辞書にして、

class MapViewController: UIViewController {

    var storeDetailViewControllers: [Int: StoreDetailViewController] = [:]

    func configureViewController(storeName: String, storeReview: String, storeImage: String, count: Int, postUserName: String, postUserIcon: String) {

        storeDetailViewControllers[count] = StoreDetailViewController(storeName: storeName, storeReview: storeReview, storeImage: storeImage, postUserName: postUserName,  postUserIcon: postUserIcon,count: count)

    }

みたいな。ただし、StoreDetailViewController をあらかじめたくさん作るのはあまり良くないので、画面遷移時に getPostUserInfo して、結果を受け取ってから画面遷移する（または、画面遷移後に getPostUserInfo する）という手もあります。
あと、configureViewController の引数 storeName, storeReview, storeImage, count, postUserName, postUserIcon ですが、まとめて構造体かオブジェクトにした方がいろいろ便利になると思います。

Comments

[Naoki_Pro]

いつもありがとうございます。

ご回答の
「どうしても結果は非同期になりますから、結果の格納方法を工夫することになると思います。」
というのは for in 文 で実行した通りの順番でFirebaseから値を取り出すのは無理ということなのでしょうか？

辞書型の使い方が慣れていないので、上手くできるか分かりませんがやってみますm(__)m

[Naoki_Pro]

無事、実装することができました。本当にありがとうございます。

StoreDetailViewControllerをあらかじめ量産するのはやめて、画面遷移時にStoreDetailViewControllerをインスタンス化して尚且つ、その際にgetUserInfoすることにしました。