swift 配列にFirebaseの値格納したい

Question

CloudFirebaseに入っているStringg型のデータを配列に格納したいです。
getSelectedQuiz関数内で配列に格納してprintで配列に入っていることは確認できたのですが、
viewDidLoad内でgetSelectedQuizを呼び出した後にpurint(quizArray)しても空の状態で表示されてしまいます。
どなたか解決方法をご教授ください

import UIKit
import FSCalendar
import CalculateCalendarLogic
import FirebaseCore
import FirebaseFirestore

class GameViewController: UIViewController {
    private var homeViewController: HomeViewController!
    var quizArray = [String]()
    var answerArray = [String]()
    var commentaryArray = [String]()
    var db: Firestore!
    @IBOutlet weak var quiz: UITextView!
    override func viewDidLoad() {
        super.viewDidLoad()
        let settings = FirestoreSettings()
        Firestore.firestore().settings = settings
        db = Firestore.firestore()
        getSelectedQuiz()
        print(quizArray)
    }
    
    //    追加した問題を取得
    func getSelectedQuiz() {
        var answer = ""
        var commentary = ""
        var quiz = ""
        var events = ""
        db.collection(HomeViewController.userName)
            .getDocuments() { [self] (QuerySnapshot, err) in
                if let err = err {
                    print("Error getting documents: (err)")
                    
                } else {
                    MotherScheduleViewController.countEvent = 0
                    for document in QuerySnapshot!.documents {
                        let sample: Dictionary = document.data()
                        answer = sample["answer"] as! String
                        commentary = sample["commentary"] as! String
                        quiz = sample["quiz"] as! String
                        self.answerArray.append(answer)
                        self.commentaryArray.append(commentary)
                        self.quizArray.append(quiz)
                        print(self.answerArray)
                        print(self.commentaryArray)
                        print(self.quizArray)
                        MotherScheduleViewController.documentID[MotherScheduleViewController.countEvent] = document.documentID
                        MotherScheduleViewController.schedule[MotherScheduleViewController.countEvent] = events
                        MotherScheduleViewController.countEvent+=1
                    }
                }
            }
    }
}

Answer 1

    override func viewDidLoad() {
        super.viewDidLoad()
        let settings = FirestoreSettings()
        Firestore.firestore().settings = settings
        db = Firestore.firestore()
        getSelectedQuiz()
        print(quizArray)
    }

この中で、getSelectedQuiz() を実行しているにもかかわらず、print(quizArray)で表示される値が空である、というのが質問の趣旨かと思います。

getSelectedQuiz() で行っている処理を抽象的に書くと、

    //    追加した問題を取得
    func getSelectedQuiz() {
        // (1)db から値を取得する処理はすぐに返答が返らない処理
        db.collection(HomeViewController.userName)
            .getDocuments() { [self] (QuerySnapshot, err) in
            // (2)このブロックの処理は db から値が取得されたあと、つまり一定時間後に実行される
            // ここの処理も一定時間後「非同期に」処理される
            self.answerArray.append(answer)
            self.commentaryArray.append(commentary)
            self.quizArray.append(quiz)
        }
        // (3)ここ以降は db から値が取得される前に実行される
    }

という具合に処理されます。

つまり、(1) でクエリを実行しても、すぐに実行が(3)に移るわけではなく、実際には(3)以降が先に処理されて、その後(2)が実行されるという流れになります。つまり、(2)の実行は非同期で行われており、いつ処理が完了するかは未定である、ということです。

なので、print(quizArray)で値を表示させても、思ったように値を取得できないという状態になってしまいます。

解決方法はいくつか考えられますが、一般的なのは db から値を取得したあとに、その値を使った処理を実行させる方法だと思います（全体の構成によっては他の方法も考えられますので、あくまでも一般的な方法の一つです）。

対策ですが、直近の質問だと下記の質問が近いかと思います。

• Swiftの関数の戻り値の指定の仕方

ほかにも、Observer パターンや Delegate パターンなども考えられるので、必要に応じて使い分けていただければと思います。

##追記

// String型の空配列を用意する
var answers: [String] = []
// 途中の処理
for document in QuerySnapshot!.documents {
    let sample: Dictionary = document.data()
    // ここで一旦受け取り
    let answer = sample["answer"] as! String
    commentary = sample["commentary"] as! String
    quiz = sample["quiz"] as! String
    
    // いったん配列に入れる
    answers.append(answer)
}

// 最後に配列をまとめてクロージャに渡す
DispatchQueue.main.async {
    completion(answers)
}

非同期で実行されるべきクロージャを for-in 内で何度も呼び出していることが悪さの原因かもしれません（競合が起きている可能性があるため）。
たとえば、一度仮の配列に answer を入れ、for-in の外で一度に処理する、という感じで記述してみてはいかがでしょうか。

Comments

[ysdyk05]

ご回答ありがとうございます。なんとか下記のようにした所、空の配列では無くなりました
for document in QuerySnapshot!.documents {
let sample: Dictionary = document.data()
answer = sample["answer"] as! String
commentary = sample["commentary"] as! String
quiz = sample["quiz"] as! String
print(answer)
DispatchQueue.main.async {
completion(answer)
}

override func viewDidLoad() {
super.viewDidLoad()
let settings = FirestoreSettings()
Firestore.firestore().settings = settings
db = Firestore.firestore()
getSelectedQuiz() { answer in
self.quizArray.append(answer)
print(self.quizArray)
}
しかし、print(self.quizArray)の出力結果が["2", "2"]となっています。本来であれば["1", "2"]と出力されるはずでしたが、後ろの2が2度呼ばれているようです。
何故でしょうか....何度もすみません、よろしければ教えていただけないでしょうか

[TsukubaDepot]

非同期処理が連続して何度も呼び出されていることが問題かもしれません。
クロージャ内部で行っている追加処理そのものは何度も呼び出す必要がありませんので、内部で追記し、最後に一括してクロージャに渡すようにしてみてはいかがでしょうか。
詳しくは回答本文に追記しましたので、参考にしていただければと思います。

[ysdyk05]

ご回答ありがとうございます。
1 func getSelectedQuiz(completion: @escaping (String) -> () ) {
2 var answer = ""
3 var commentary = ""
4 var quiz = ""
5 var events = ""
6 var answers: [String] = []
7 db.collection(HomeViewController.userName)
8 .getDocuments() { [self] (QuerySnapshot, err) in
9 if let err = err {
10 print("Error getting documents: (err)")
11 } else {
12 MotherScheduleViewController.countEvent = 0
13 for document in QuerySnapshot!.documents {
14 let sample: Dictionary = document.data()
15 answer = sample["answer"] as! String
16 answers.append(answer)
17 }
18 DispatchQueue.main.async{
19 completion(answers)
20 }
21 }
22 }
23 }

6行目でvar answers: [String] = []を宣言し、19行目でcompletion(answers)を追加した所、19行目のanswersでCannot convert value of type '[String]' to expected argument type 'String'というエラーが発生しました。6行目をvar answers = String()にするとエラーは無くなるのですが、
getSelectedQuiz() { answers in
self.quizArray.append(answers)
print(self.quizArray)
}
print(self.quizArray)の出力結果が["12"]となってしまいます。求める結果は["1","2"]です。
このエラーについて解決方法を教えていただけないでしょうか。何度もすみません。。

[TsukubaDepot]

getSelectedQuiz(completion: @escaping (String) -> () )
と、クロージャに渡すデータの型が String になっていますから、ここを [String] と文字列の配列（Array<String>）に変更しないといけないと思います。
文字列、ではなく文字列の配列として一括処理していますから、全体の整合性を合わせてあげないと、今回のような型エラーが発生してしまいますのでご確認いただけますでしょうか。

[ysdyk05]

ご回答ありがとうございます
引き続きいろいろ試してみた所、
func getSelectedQuiz(completion: @escaping (String) -> () ) {
➡️func getSelectedAnswer(completion: @escaping ([String]) -> () ) {
self.quizArray.append(answers)
➡️self.quizArray.append(contentsOf: answers)
とした所解決することができました。
何度も親身にご回答していただき本当にありがとうございました。
また機会がありましたらよろしくお願い致します.....笑