メインスレッドを停止させて処理を待たせたい

Question

private func getCategories(){
        Alamofire.request(Constant.Url.categoryUrl, method: .get, encoding: JSONEncoding.default).responseJSON { response in
            switch response.result {
            case .success:
                guard let object = response.result.value else { return }
                let json:JSON = JSON(object)
                json.forEach { (_, json) in
                    self.categories.append(json["name"].stringValue)
                    self.categoriesId.append(json["id"].stringValue)
                }
            case .failure:
                print("error")
            }
        }
    }
override func viewControllers(for pagerTabStripController: PagerTabStripViewController) -> [UIViewController] {
        var articles: [ArticleViewController] = []
        getCategories()
        for i in 0..<categories.count {
            articles.append(ArticleViewController())
            articles[i].categoryInfo = IndicatorInfo(title: categories[i])
        }
        return articles
    }

上記のようなコードで、getCategoriesはAlamofireを使用してapiを叩いて、非同期でデータを取得しています(Alamofireの仕様で非同期になる)。この処理が全て終わってからfor 以下の処理を実行したいのですが、どのようなコードを書けば良いでしょうか？
responseJSONのブロック内と、その外側が同じスレッドで処理されるため、waitすると、止まってしまう。ということもあり、上手くいっていません…。

Answer 1

データを非同期で取得しているのだから、データを取得するまでメインスレッドの処理を停止させるのではなく、データを取得したタイミングで処理を呼び出してもらい、そこで受信データをもとにした処理を行う必要があります。

おそらくこの質問は、「XLPagerTabStripで決められているviewControllers()メソッドをoverrideして、ViewControllerの配列を返却しなければならないのだが、非同期にデータを受信したタイミングでViewControllerの配列が決まるのに、どうやったらデータを受信したタイミングで、viewControllers()メソッドの戻りを返却できるのかがわからない」という質問なのだと思います。

その答えを解く鍵は reloadPagerTabStripView()メソッドです。
これを呼び出せば viewControllers()メソッドを呼びだしてもらえるので、
データを受信した状態で最終的に作成されたViewControllerの配列を返せると思います。

具体的には、
ViewControllerの配列である

var articles: [ArticleViewController] = []

をプロパティとして定義し、

responseJSONのクロージャの内でself.categoriesを作成した続きで
(1)ViewControllerの配列(articles)作成
(2)self.reloadPagerTabStripView()呼び出し
を実施し、

viewControllers()メソッドの処理は

return articles

とするだけでよさそうに思います。

---

(7/2 0:37 サンプルコード追加)

    var articles: [ArticleViewController] = [ArticleViewController()]

    override func viewDidLoad() {
        super.viewDidLoad()
        getCategories()
    }
    
    private func getCategories() {
        Alamofire.request(Constant.Url.categoryUrl, method: .get, encoding: JSONEncoding.default).responseJSON { response in
            switch response.result {
            case .success:
                guard let object = response.result.value else { return }
                let json:JSON = JSON(object)
                json.forEach { (_, json) in
                    self.categories.append(json["name"].stringValue)
                    self.categoriesId.append(json["id"].stringValue)
                }
                self.articles = []
                for i in 0..<self.categories.count {
                    self.articles.append(ArticleViewController())
                    self.articles[i].categoryInfo = IndicatorInfo(title: self.categories[i])
                }
                self.reloadPagerTabStripView()
            case .failure:
                print("error")
            }
        }
    }

    override func viewControllers(for pagerTabStripController: PagerTabStripViewController) -> [UIViewController] {
        return articles
    }

Comments

[退会済みユーザー]

回答ありがとうございます。
reloadPagerTabStripView()は知りませんでした。
上記のように実装すると、やはり非同期でgetCategories()のメソッドが実行されるので、どうしても return articlesが先に実行されて、空の状態になってしまうと思うのですが…

[TakeOne]

このViewControllerが起動されると最初に`viewControllers()`が呼ばれ、そのタイミングでは`return articles`で個数0の配列が返ります。
しかしその後、responseJSONのクロージャが呼ばれ、その中でarticlesを生成して`self.reloadPagerTabStripView()`を実行すると、再度`viewControllers()`が呼ばれます。そのタイミングで正しい配列が返せるので、それで意図した個数のタブが表示されると思います。データを受信したタイミングでロードし直せばよいということです。

[退会済みユーザー]

一度return 0としてしまうとライブラリの設定上エラーが返ってきてしまい、それ以降の処理に行き着きません。
callbackなどを使用して上手くできないでしょうか？

[TakeOne]

どういうエラーがどこに返ってきて、どのようにエラーが表示されているんですか?

[退会済みユーザー]

```
super.viewDidLoad() //Thread 1: Fatal error: viewControllers(for:) should provide at least one child view controller
}

private func getCategories() -> [ArticleViewController] {
var articles: [ArticleViewController] = []
Alamofire.request(Constant.Url.categoryUrl, method: .get, encoding: JSONEncoding.default).responseJSON { response in
switch response.result {
case .success:
guard let object = response.result.value else { return }
let json:JSON = JSON(object)
json.forEach { (_, json) in
self.categories.append(json["name"].stringValue)
self.categoriesId.append(json["id"].stringValue)
}
// var articles: [ArticleViewController] = []
for i in 0..<self.categories.count {
articles.append(ArticleViewController())
articles[i].categoryInfo = IndicatorInfo(title: self.categories[i])
}
self.reloadPagerTabStripView()
case .failure:
print("error")
}
}
return articles
}

override func viewControllers(for pagerTabStripController: PagerTabStripViewController) -> [UIViewController] {

// var articles: [ArticleViewController] = []
return getCategories()
}
}
```

となっています。よろしくお願いします。

[TakeOne]

なるほど。
「viewControllers(for:) should provide at least one child view controller」
と出ているので、ViewControllerは最低限1個ないとエラーになるのですね。
だったら、最初は1個だけダミーのArticleViewControllerを入れておいて、
データを受信したら正しいものに入れ替えたらいいと思います。

あと、私が説明したことといろいろ違うことをしているので、
回答にサンプルコードを追加しておきました。

[退会済みユーザー]

返信が遅くなってしまい、申し訳ございません。
サンプルコードまで載せていただき、ありがとうございます。上記をを実行したところ。上手くいきました。本当にありがとうございます。またなにかあればよろしくお願いします。