【swift】同期処理が上手くいかずtableviewが上手く表示できません【firebase】

Question

前提・実現したいこと

現在、xcode(swift)とfirestoreを用いて、DBに入っているフォロワー情報から、フォロワー一覧をtableviewに表示しようと試みています。

しかし同期処理やカスタムセルについて理解不足のためか上手くいきません。ご教授くださいませ。

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

Thread 1: Fatal error: Index out of range

該当のソースコード

// セルを返す
    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let cell = tableView.dequeueReusableCell(withIdentifier: "SocialTableViewCell", for: indexPath) as! SocialTableViewCell
        DispatchQueue.global().async {
            self.fetchSocialTable()
            DispatchQueue.main.async {
                    let data = self.socialTable[indexPath.row]
                    cell.UserName.text = data.UserName
                    cell.UserID.text = data.UserID
            }
        }
        return cell
    }
    
    func fetchSocialTable(){
        self.db.collection("users").document(self.uid).collection("followers").limit(to:20).getDocuments(){snapshot,err in
            if let snapshot = snapshot{
                self.socialTable = snapshot.documents.map{ userData -> SocialTable in
                    let data = userData.data()
                    return SocialTable(iconname: "hogehoge", UserName: data["UserName"] as! String, UserID: data["UserID"] as! String)
                }
            }
        }
    }

該当部分のソースコードを抜粋しましたが、

let data = self.socialTable[indexPath.row]

この部分で上記エラーが生じます。
printで確認したところ、socialTableには何も入っていませんでした。
socialTableは以下で初期化しています。

struct SocialTable{
        var iconname: String
        var UserName: String
        var UserID: String
}
private var socialTable: [SocialTable] = []

fetchSocialTable関数内にあるself.uidにドキュメント名が正しく入っていることは確認済みです。
print関数で確認したところ、fetchSocialTable関数の通信処理が始まる前に該当のエラー箇所が実行されているようです。
同期処理は正しく書けていると思っているのですが、やはり間違っているのでしょうか。

ご教授いただきたいです。また、何か情報が足りなければお申し付けください。

追記

// セルを返す
    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let cell = tableView.dequeueReusableCell(withIdentifier: "SocialTableViewCell", for: indexPath) as! SocialTableViewCell
        let queue = DispatchQueue.global(qos: .default)
        let dataset = self.fetchSocialTable()
        queue.async {
            cell.UserName.text = dataset[indexPath.row].UserName
            cell.UserID.text = dataset[indexPath.row].UserID
        }
        
        return cell
    }

func fetchSocialTable() -> [SocialTable]{
        let semaphore = DispatchSemaphore(value: 0)
        var data:[SocialTable] = []
        self.db.collection("users").document(self.uid).collection("followers").limit(to:20).getDocuments(){snapshot,err in
            if let snapshot = snapshot{
                print("connect firestore...")
                for document in snapshot.documents {
                    data.append(SocialTable(iconname: "hogehoge", UserName: document.data()["Nm"] as! String, UserID: document.data()["UI"] as! String))
                }
            }
            semaphore.signal()
            print("semaphore ok")
            print(self.socialTable)
        }
        print("semaphore wait...")
        semaphore.wait()
        print("complete...")
        return data
    }

これで実行した場合、semaphore wait...のみが出力されその後の処理が走りません。
一旦テーブルへの書き込みをやめて、firestoreからデータを取得する部分の確認を行ったのですが、それは上手くいっていました。
何かいけない点があるのでしょうか。

Comments

[fuzzball]

tableView(_:numberOfRowsInSection:)を見せて下さい。

[RyotaroIsoyama]

// セルの数
func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
if(followOrFollower_count>20){
return 20
}else{
return followOrFollower_count
}
}

[RyotaroIsoyama]

見にくくてすみません、
followOrFollower_countは全画面から遷移する際に値を入れています！
今回は2が入っています

Answer 1

解決しました。semaphore.wait()でデッドロックが起こっていたのが問題だったみたいです。

let queue = DispatchQueue.global()
let mainQueue = DispatchQueue.main
queue.async {
   let dataset = self.fetchSocialTable()
   mainQueue.async {
      cell.UserName.text = dataset[indexPath.row].UserName
      cell.UserID.text = dataset[indexPath.row].UserID
   }
}

Answer 2

データの読み込みが終わってないのに、

let data = self.socialTable[indexPath.row]

を呼び出したためだと思います。
メインスレッドに戻る前に、データの読み込みが終わったかどう確認してみてください。

GCPのDispatchSemaphoreを使えば、上手く行けるはずです。
以下は、APIコールを待っている処理の事例です。

import Foundation
class MyTest {
    func runTheShow() {
        DispatchQueue.global(qos: .utility).async {
            let json: Teratail? = self.fetchQuestion()
            DispatchQueue.main.async {
                print(json)
            }
        }
    }

    func fetchQuestion() -> Teratail? {
        var myresponse: Teratail?  = nil
        guard let url = URL(string: "https://teratail.com/api/v1/questions") else {return nil}
        let semaphore = DispatchSemaphore(value: 0)
        let task = URLSession.shared.dataTask(with: url) { (data, response, error) in

            do {
                guard let data = data else {return}
                
                let json = try JSONDecoder().decode(Teratail.self, from: data)
                
                myresponse = json
            }
            catch {
                    print("error:", error)
            }
            semaphore.signal()
        }
        
        task.resume()
        _ = semaphore.wait(wallTimeout: .distantFuture)
        return myresponse
    }
}

struct Teratail: Decodable {
    let meta: Meta
    let questions: [Question]
}
    
struct Meta: Decodable {
    let hit_num: Int
    let limit: Int
    let message: String
    let page: Int
    let total_page: Int
}

struct Question : Decodable {
    let count_clip: Int
    let count_pv : Int
    let count_reply: Int
    let created: String
    let modified: String
    let id: Int
    let is_accepted: Bool
    let is_beginner: Bool
    let is_presentation: Bool
    let tags: [String]
    let title: String
    let user: User
}

struct User: Decodable {
    let display_name:  String
    let photo: String
    let score: Int
}

let obj = MyTest()
obj.runTheShow()

Comments

[RyotaroIsoyama]

アドバイスありがとうございます！！！！
semaphoreを使ってみたのですが、今度はsemaphore.wait()で処理が止まるようです。
コードが読みづらいので本文に追記することにします。

[vanderlvov]

「semaphore wait...」で止まると言うのは、「semaphore.signal()」まで行けないようですね。
この辺のログをみてもいいかも：
if let snapshot = snapshot{
print("connect firestore...")
for document in snapshot.documents {
data.append(SocialTable(iconname: "hogehoge", UserName: document.data()["Nm"] as! String, UserID: document.data()["UI"] as! String))
}
}