swiftでの非同期処理完了までの待機の仕方

Question

SwiftでライブラリKingfisherを利用してURLの画像を取得してリストとして表示しようとしているのですが
Kingfisherは非同期で取得するようでボタンを一回押すだけでは表示できないようです

DispatchSemaphoreを使えば待機していくれるということはわかったのですが
下記のコードを実行するとフリーズしていまいます

        let semaphore = DispatchSemaphore(value: 0)
        var uiImage : [UIImage] = []

        for(_ ,value) in self.ImageUrlList
        {
            KingfisherManager.shared.retrieveImage(with: URL(string: value)!){ result in
                switch result {
                case .success(let value):
                    uiImage.append(value.image)
                    semaphore.signal()
                case .failure(let error):
                    print(error)
                }
            }
            
            semaphore.wait()
            
        }

ImageUrlListはDictionary型で画像のURLが入っています

このコードのどこが問題なのでしょうか

Comments

[hoshi-takanori]

Kingfisher　の completionHandler はデフォルトではメインスレッドで処理されるので、semaphore.wait を先に呼ぶと semaphore.signal が呼ばれずにデッドロックするのでしょう。
https://kingfisher.onevcat.com/kingfishermanager/

非同期処理を待機するのは基本的に良くなくて、いったん制御を OS に戻して、結果が戻ってきた時に改めて処理を行うようにするのが良いと思います。

[tomato879241]

そもそもどのクラスの下でそのコードを書いているかわからないんですけど？

[testplayer]

>hoshi-takanoriさん
そうなんですかそちらの方向で修正したいと思います
コードが書いてあるプロジェクトはある程度進んでおり待機中の操作を制御するフラグを追加するのが大変そうなので待機を追加できればなと思ってのものでよい方法があったら教えていただきたいです

>tomato879241さん
メインのViewControllerです

[hoshi-takanori]

待機するということはその間メインスレッドを止めてアプリが固まったように見えるということなので、お勧めはできません。
ちなみに、uiImage の配列はどう使うのでしょうか？

[testplayer]

配列に関してはcollectionViewで表示するのと
ファイルで保存するために取得しております。（保存→保存した画像を表示という感じです）
強制待機は良くないとわかったので処理後に表示する機能を追加したいと思います
ありがとうございます

[hoshi-takanori]

保存に関しては、Kingfisher はもともとダウンロードした画像をキャッシュする機能を持ってると思いますが、それとは別に保存したいということでしょうか？ また、表示に関しては、とりあえず仮の画像 (placeholder) を表示しておいて、目的の画像がダウンロードされたら自動的に表示するような機能も、たぶん Kingfisher が持ってるのでは…。

[testplayer]

すいません
正直Kingfisherの機能についてはまだ把握してない部分が多くよくわかってません
placeholderなどを調べて見たいと思います
ありがとうございます

[hoshi-takanori]

というか、Kingfisher の基本的な使い方って、
imageView.kf.setImage(with: imageUrl)
これだけですけど…。
https://github.com/onevcat/Kingfisher