2つの重い処理を直列に実行したい(SVProgressHUDを使いたい)

Question

まず、やりたいことのイメージを表すためにcodeを示します。

        SVProgressHUD.show(withStatus: "Start1")
        DispatchQueue.global().async {
            sleep(5)
            print("DEBUG_PRINT: Done1")
            DispatchQueue.main.async {
                SVProgressHUD.dismiss()
            }
        }

        SVProgressHUD.show(withStatus: "Start2")
        DispatchQueue.global().async {
            sleep(5)
            print("DEBUG_PRINT: Done2")
            DispatchQueue.main.async {
                SVProgressHUD.dismiss()
            }
        }

つまり、code中ではsleep(5)で示している重い処理が2つあり、それらを順に実行して、それぞれでSVProgressHUDを用いてIndicatorを回したいということです。非同期の扱いがよくわかっていないため、どのようにすればいいかの見当がつきません。ネットを見て「セマフォを使うのかなあ？」などといろいろやってみたのですが、うまくいっていない状況です。

ご教示ください。よろしくお願いします。

Answer 1

質問のコードだと、SVProgressHUD.show(withStatus: "Start1")の直後にSVProgressHUD.show(withStatus: "Start2")が実行されてしまいます。

なんか難しく考え過ぎているような気が。

DispatchQueue.global().async {
    //#1
    DispatchQueue.main.async {
        SVProgressHUD.show(withStatus: "Start1")
    }
    sleep(5)
    print("DEBUG_PRINT: Done1")
    DispatchQueue.main.async {
        SVProgressHUD.dismiss()
    }
    
    //#2
    DispatchQueue.main.async {
        SVProgressHUD.show(withStatus: "Start2")
    }
    sleep(5)
    print("DEBUG_PRINT: Done2")
    DispatchQueue.main.async {
        SVProgressHUD.dismiss()
    }
}

でいけませんかね？
（SVProgressHUDを呼び出しているところはsyncでもいい‥はず）

Comments

[HajimeHamada]

回答ありがとうございます。動作することを確認しました。
全体をglobalキューに入れて、mainキューで実行するものだけ指定する、ということですね？ わかりやすいです。 いろいろなやり方があるものですねえ。
ありがとうございました。

Answer 2

愚直にやるとしたら下記のような方法でできるかと思います。

SVProgressHUD.show(withStatus: "Start1")
DispatchQueue.global().async {
    sleep(5)
    print("DEBUG_PRINT: Done1")
    DispatchQueue.main.async {
        SVProgressHUD.dismiss()
        SVProgressHUD.show(withStatus: "Start2")
        DispatchQueue.global().async {
            sleep(5)
            print("DEBUG_PRINT: Done2")
            DispatchQueue.main.async {
                SVProgressHUD.dismiss()
            }
        }
        
    }
}

ただこれだと見づらいので
中の処理をメソッドに分けたり、
下記のようなライブラリが色々ありますので、ライブラリの導入なども検討してみると良いかなと思いました。

https://github.com/mxcl/PromiseKit

Comments

[HajimeHamada]

回答ありがとうございます。所望の動作をすることを確認いたしました。
なるほど、入れ子にしていくわけですね。非同期処理に慣れておらず、頭がついていかないところがありますが、ライブラリも含めて勉強していきたいと思います。ありがとうございました。

Answer 3

こんにちは！
GCDのキューには処理を順番に実行するシリアルキューと、複数同時に実行するコンカレントキューがありあます。
DispatchQueue.global()
というコードはグローバルな(システム組み込みの)”コンカレント”キューを返してくれます。
メインキュー以外のシリアルキューは自分で生成します。

let serialQueue = DispatchQueue(label: "mySerial") //シリアルキュー
let concurrentQueue = DispatchQueue(label: "myConcurrent", attributes: .concurrent) //※ご参考コンカレントキュー

このコードのようにattributesに.concurrentを明示的に指定"しなければ"シリアルキューを作ることができます。
今回のケースであれば

let serialQueue = DispatchQueue(label: "mySerial") // シリアルキュー
        serialQueue.async {
            DispatchQueue.main.async {
                SVProgressHUD.show(withStatus: "Start1")
            }
            sleep(5)
            print("DEBUG_PRINT: Done1")
            DispatchQueue.main.async {
                SVProgressHUD.dismiss()
            }
        }

        serialQueue.async {
            DispatchQueue.main.async {
                SVProgressHUD.show(withStatus: "Start2")
            }
            sleep(5)
            print("DEBUG_PRINT: Done2")
            DispatchQueue.main.async {
                SVProgressHUD.dismiss()
            }
        }

これでキューに入れた順番での処理の実行がキューの特性により保証されます。
SVProgressHUDを実際にいれてテストしてみたのでこれでOKだと思います！

Comments

[516k]

SVProgressHUDがわかっていませんでした。このコードでは質問者さんのやりたいことが実現できません。はやとちりで回答して申し訳ありませんでした。
既に削除申請しております。

[516k]

teratailさんが削除してくれなかったので、一応SVProgressHUDというのを入れてみてテストしてみました。おそらく質問者さんがやりたかったことはこれで実現できていると思います。
ネストが深くなりすぎないレベルなら私個人としてはnewmtさんの回答のように入れ子にしていくのが読みくだしやすく、バグも混入しづらいと思います。

[HajimeHamada]

回答ありがとうございます。SVProgressHUDまで試していただいたようで、恐縮です。
なるほど、シリアルキューを作って、その中にメインキューの処理を入れていくということですね？そうであれば、頭のなかで理解しやすいです(考え方が間違っていたら、ご指摘ください)。参考にさせていただきます。
ありがとうございました。

[516k]

重たい処理は全てシリアルキューのバックグラウンドスレッドで実行し、UI関連の処理だけをメインキューで実行するイメージです。
今回のケースであればUI関連の処理とはSVProgressHUDの表示、非表示ですね。

Answer 4

複数のプログラムを直列に実行(順に実行）するのが同期実行、同時並行に実行（実行順は制御できない）するのが非同期実行です。

質問者は「順に実行したい」のですから、非同期実行(DispatchQueue.global().async )ではなく、同期実行（DispatchQueue.global().sync ）を使えば良いと思います。

Comments

[HajimeHamada]

回答ありがとうございます。
2つのDispatchQueue.global().asyncをDispatchQueue.global().syncに変更してみました。重い処理(sleep)の順番は問題なく、(5秒スリープ)→「Done1」表示→(5秒スリープ)→「Done2」表示と動作するようになったのですが、SVProgressHUDのIndicatorが全く表示されません。
同期・非同期の話とともに、SVProgressHUDの使い方の話もあるようなのですが、何かわかりますでしょうか？

[fuzzball]

メインスレッドに処理が回ってないんじゃないかなぁ。asyncの中で、2つの処理をsyncすればいいのかな。

[516k]

SVProgressHUD.show(withStatus: "Start1")
SVProgressHUD.show(withStatus: "Start2")
を呼び出しているスレッドはメインスレッドのはずなので、syncを呼び出せばメインスレッドがロックしてHUDが表示されなくなるのは、ある意味正しい(想定通りの)振る舞いですね。
fuzzballさんの指摘のようにDispatchQueue.global().asyncのブロック内でsyncを呼び出すと、今度はバックグラウンドスレッドがロックするのでメインスレッドで正しくHUDが表示されると思います。
個人的にはsyncを使うのその呼び出し元のスレッドがバックグラウンドスレッドであることが100%保証されている場合に限定した方がいいと思います。