For文の中の非同期処理でリストを作成し、For文外で使用したい

Question

前提・実現したいこと

For文の中の非同期処理でリストを作成し、For文の外でリストを活用したいと考えています。
しかし、非同期処理なので、For文の中の処理を待たずに、リストが空の状態で処理が進みます。
調べていて、DispatchSemaphoreを活用することは分かったのですが、どうも上手くいきません。
使い方が間違っていると思うのですが、調べても原因が分からず困っています。

発生している問題・エラーメッセージ

処理が進まなくなった。

該当のソースコード

let semaphore = DispatchSemaphore(value: 0)
var testList = [String: Test]()
                    
for value in list {
  //非同期処理スタート
  db.collection("Test").document(value)
    .getDocument { (document, error) in

      if let error = error {
        print("Error getting documents: (error)")
      } else {
        let result = Result {
          try document?.data(as: Test.self)
        }
        switch result {
        case .success(let resultValue):
          if let test = resultValue {
            testList[document!.documentID] = test
          } else {
            print("Document does not exist")
          }
        case .failure(let error):
            print("Error decoding city: (cError)")
        }
      }
      semaphore.signal()
    }　// ここまで非同期処理
    semaphore.wait()
}//For文終わり

print(testList) //DispatchSemaphoreを使わないと[:]に、使うとたどり着かない

試したこと

semaphore.signal()やsemaphore.wait()の位置を色々変えてみた。

Comments

[MasakiHori]

そもそもFirestoreの使い方間違ってます。

[abc1222]

使い方というのはコードではなく、上記の読み取りが必要な構造が間違っているということでしょうか

[MasakiHori]

addSnapshotListener はdocumentなどに変更があった時に通知を受けるための関数を登録しておくものです。
登録した関数は登録時に一度呼び出された後、変更があるたびに呼ばれます。
変更があるたびに呼ばれるのでこの関数でスレッドのブロックなどを行うと問題が生じます。
この場合はおそらくgetDocument を使うべきだと思われます。（全体見ないとわからないけど）

[abc1222]

なるほど！分かりやすくありがとうございます。
たしかにgetDocumentで良い場所でした！
まずは上記を修正します

Answer 1

DispatchGroupを使います

let group = DispatchGroup()
var testList = [String: Test]()

for value in list {
  
  // groupのタスク数を増やす
  group.enter()
  
  //非同期処理スタート
  db.collection("Test").document(value)
    .getDocument { (document, error) in
      
       defer {
        // groupのタスク数を減らす
        group.leave()
      }
      
      ...
      
    }
}

// groupのタスク数が0になるまで待つ
group.wait()

print(testList)

Comments

[abc1222]

ありがとうございます！
使う物自体が違ったんですね！
思い通りの動きになりました