DispatchQueueとクロージャ

Question

#AlamofireとKannaを使ってスクレイピングしてみました
gameのキャラクターの情報をサイトから取ってくるアプリを作ろうと思っています。
最初の画面(viewController)でURLを5個まで貼ってもらい、userDefaultsで保存して画面遷移しデータを取ってきてviewで表形式でそれぞれの値を比較しようと思いました。

    var urlArray:[String] = []
    @IBOutlet weak var textField1: UITextField!
    @IBOutlet weak var textField2: UITextField!
    @IBOutlet weak var textField3: UITextField!
    @IBOutlet weak var textFiled4: UITextField!
    @IBOutlet weak var textField5: UITextField!
    
    
    
    @IBAction func buttonPushed(_ sender: UIButton) {
        if self.textField1.text != ""{
             self.urlArray.append(self.textField1.text!)
        }
        if self.textField2.text != ""{
             self.urlArray.append(self.textField2.text!)
        }
        if self.textField3.text != ""{
             self.urlArray.append(self.textField3.text!)
        }
        if self.textFiled4.text != ""{
             self.urlArray.append(self.textFiled4.text!)
        }
        if self.textField5.text != ""{
             self.urlArray.append(self.textField5.text!)
        }
        let userDefaults = UserDefaults.standard
        userDefaults.set(self.urlArray, forKey: "urlArray")
        userDefaults.synchronize()
        performSegue(withIdentifier: "goPlayers", sender: nil)
    }
    
    
}

var players:[Player] = []
    
    override func viewDidLoad() {
        super.viewDidLoad()
        // Do any additional setup after loading the view.
        
        let userDefaults = UserDefaults.standard
        let urlArray:[String] = userDefaults.array(forKey: "urlArray") as! [String]
        
        for i in 0..<urlArray.count{
            DispatchQueue.global().sync {
                print(urlArray[i])
                self.scrapWebsite(urlArray[i])
                print("players.count3:(self.players.count)")
            }
            print("players.count1:(self.players.count)")
        }
        print(self.players.count)
        //viewを追加する関数
        self.SetView(players.count)
    }

///////////////////////////////////////////////////    
    struct Player {
        var playerName:String?
        var MaxLevel:String?
        var abilityList:[(name:String,score:String)] = [/*省略しますがタプル配列で26個分、予め項目名(name)と数値(score)を入れて置いてます*/]
    }
//////////////////////////////////////////////////////

func scrapWebsite(_ url:String){
        //GETリクエスト 指定URLのコードを取得
        Alamofire.request(url).responseString { response in
            //Boolで確認
            print("true or fauls:(response.result.isSuccess)")
            
            if let html = response.result.value {
                //plaers配列に追加
                self.players.append(self.parseHTML(html: html))
                print("players.count2:(self.players.count)")
            }
        }
    }
    
    func parseHTML(html: String)-> (Player){
        //戻り値を宣言
        var returnPlayer = Player()
        
        if let doc = try? HTML(html: html, encoding: .utf8) {
            //タイトルをプリント
            print("doc.titile:(doc.title!)")

            for link in doc.xpath("/html/body/div[2]/div[4]/script[1]/text()"){
            //省略しますがreturnPlayerに値を入れる処理をします
            }
        }
        print("returnPlayer:(returnPlayer)")
        return returnPlayer
    }
    

##躓いているところはAlamofireとKannaとは全く関係ないところです
ViewControllerでtextField1のみにURLを入れれ実行します。
するとscrapWeb()でplayers配列にperseHTML()の戻り値(returnPlayer)を追加しているのにprint("players.count2:(self.players.count)")ではplayers.countが1となりますが、print("players.count1:(self.players.count)"),print("players.count3:(self.players.count)")では0となってしまします。
###クロージャの処理はその関数の処理が終わってから始まるのか？

   for i in 0..<urlArray.count{
       DispatchQueue.global().sync {
             print(urlArray[i])                           //(1)
             self.scrapWebsite(urlArray[i])               //(2)
             print("players.count3:(self.players.count)" //(3)
       }
       print("players.count1:(self.players.count)"       //(4)
   }

これだとglobal()が並列処理でsyncが同期処理なので、処理が開始される順番は(1)→(2)→(3)→(4)となり、並列処理なので(1),(2),(3)の終了の順番はまちまちで、(1),(2),(3)の終了後に(4)が開始され終了されるのではないでしょうか？
"start"/"end"ログを見ると、おそらくクロージャの処理はscrapWebsite関数の処理が終了した後に実行されていると思うのですがそういう考えでいいのでしょうか？

Answer 1

global()は並列処理そのものではなく、グローバルなキューを取得するという意味です。そして、syncは、キューに同期処理を追加するという意味になります。

for文の動作ですが、
1: i = 0の時のクロージャーをグローバルキューに積む
2: (4)を実行する
3: i = 1の時のクロージャーをグローバルキューに積む
4: (4)を実行する
5: i = urlArray.countになるまで繰り返し
となります。このfor文は、メインキューで実行されます。

グローバルキューはメインキュートは独立したキューなので、for文が動いている途中でもクロージャーの中身を実行します。

外から見ると、メインキューで(4)を指定回数実行、グローバルキューで(1〜3)を指定回数実行となりますが、タイミングは(4)と(1〜3)の計算時間に依存し、どちらかが終わるまでどちらかが待っているということはありません。

Comments

[kyokio]

回答ありがとうございます。
キュー 、スレッドがまだ完全に理解できてないのでもう少し勉強します。

今回はスクレイピングでキャラクターの情報(最大5つ)を取ってきて配列に格納してから、viewを表示する関数をを実行したいと考えています。
直列、同期処理をすれば前の処理の完了を待つので解決すると思いやってみましたが、クロージャはクロージャの処理が完了する前に次の処理に移ってしますのでしょうか？

[AOKINAO]

すいません、いろいろ間違えて回答してしまいました。
syncは、キューに同期処理を追加するので、キューに追加された処理が終わるまでそこで処理が停止します。ですから、
1: i = 0の時のクロージャーをグローバルキューに積み、クロージャー内の処理が終わるまで待つ
2: (4)を実行する
3: i = 1の時のクロージャーをグローバルキューに積み、クロージャー内の処理が終わるまで待つ
4: (4)を実行する
5: i = urlArray.countになるまで繰り返し
となります。結果的に、DispatchQuereを使わずにプログラムを書くのと同じになります(クロージャーの処理が完了するまで、次の処理には移りません)。

　さて、私はAlamofireを知らないのですが、関数scrapWebsite()の中のAlamofire.request(url).responseStringという関数が非同期処理を使っていそうです。ですから、scrapWebsite()を呼び出して帰ってきた時点では、Alamofire.request(url).responseStringの処理が終わっていない、すなわち、playersの値が更新されていないのではないでしょうか。

players.count3, players.count2, players.count1の順番に表示されるのが理想だけれども、実際はcount3がcount2より先に表示されていませんか？

[kyokio]

ありがとうございます。
だんだん理解してきました。大きく分けると並列or直列処理、同期or非同期の組み合わせが選べるんですね。
今回はループのたびにqueueをに同期処理を追加して追加されたすべての処理が終わるまで待っている、かつ並列処理だが処理にかかる時間が(1),(2),(3)でさがあまりないため処理を開始した順番に処理が終わり、結果的にDispatchQueueを使わない場合と同じになる。

`Alamofire.requesr(url).responseString`という関数が非同期処理を行なっており、処理に時間がかかるため最後に実行されているということでしょうか？
count3がcount1より先に表示されています。
まだmainキューやスレッドがどのように動作してるのか理解してないのでもう少し調べてみます。