質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.52%

  • Swift

    7226questions

    Swiftは、アップルのiOSおよびOS Xのためのプログラミング言語で、Objective-CやObjective-C++と共存することが意図されています

  • Xcode

    4088questions

    Xcodeはソフトウェア開発のための、Appleの統合開発環境です。Mac OSXに付随するかたちで配布されています。

非同期処理の処理待ちで動作が止まる

解決済

回答 3

投稿

  • 評価
  • クリップ 0
  • VIEW 549

bbb04

score 11

Alamofireを使ってAPIリクエストをしています。レスポンスをSwiftyJSONを用いてパースした結果を返す関数を実装しようとしています。

https://qiita.com/kazuhirox/items/9ecb25bc238ad2d47ff0

の手法を用いて非同期処理の処理待ちをしているのですがループから抜け出せず
この関数の中で処理が止まっています。(printまで行ってません)

whileループのどのような処理を加えると抜けることができるのでしょうか。
非同期処理の理解が浅いのですがよろしくお願いします。

func getRain(latitude: Double,longitude: Double,time: Int) -> JSON{

~~~~~~~

        var keepAlive = true
        var json:JSON = ""

        Alamofire.request(url1,method: .get).responseJSON { (response) in

            if let Json = response.result.value {
                json = JSON(Json)
                keepAlive = false
                }

        }
        let runLoop = RunLoop.current
        while keepAlive &&
            runLoop.run(mode: RunLoopMode.defaultRunLoopMode, before: NSDate(timeIntervalSinceNow: 0.1) as Date) {
        }
        print("json",json)
        return json



    }

~~~~~~~~~~の部分はurlなどの変数宣言をしています

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • MasakiHori

    2017/11/13 17:19

    getRain関数をメインスレッド上からDispatchQueue#syncなどで同期的に処理していませんか?

    キャンセル

  • bbb04

    2017/11/13 17:36

    DispatchQueueは使って無いですが、別の非同期処理の中から関数呼び出しをしています。もしかしてこれが問題でしょうか。

    キャンセル

  • MasakiHori

    2017/11/13 17:54

    whileループ内で print("waiting")などしてみるとどうなりますか? 出力され続けますか?

    キャンセル

  • bbb04

    2017/11/13 18:00

    とめどなくwaitingが出力される状況です。

    キャンセル

回答 3

checkベストアンサー

+1

コード自体に誤りはないと思います。

このif文に入らないから抜けないのであり、調べるべきはGETで投げてるAPIの返す内容ですね。

            if let Json = response.result.value {
                json = JSON(Json)
                keepAlive = false
            }

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/11/12 10:24

    ありがとうございます。返信が遅れて申し訳ありません。
    調べてみるとおっしゃる通りif文を通ってないようです。しかし、今まではif文に入っていたのですが処理待ちをしようとすると入らなくなりました。何が原因なのでしょうか。

    キャンセル

  • 2017/11/13 09:42

    該当のifの条件にkeepAliveは関係ないので処理待ちのコードを追加したことでif文に入らなくなるというのはおかしくないですか?

    キャンセル

  • 2017/11/13 10:46

    var json:JSON = ""
    var keepAlive = true
    Alamofire.request(url1,method: .get).responseJSON { (response) in

    if let Json = response.result.value {
    json = JSON(Json)
    print(json)
    keepAlive = false

    }


    }

    これだけならif文を通ってprintでコンソール出力できるようです。
    しかし、whileループの部分を追加すると実行が止まります。

    キャンセル

  • 2017/11/13 11:21

    本当にこのifブロック内のprint("json",json)の出力ですか?他の場所でもprintしてませんか?
    ifブロック内に入ってればwhile文には入らず処理を抜けるはずなんですが…
    whileループを書くと止まる(というかウェイトになる)のはifブロックに入ってないからです。
    何か見落としてると思いますよ

    キャンセル

  • 2017/11/13 15:38

    他のところでの出力はしていないのでここの出力で間違い無いと思います。

    print("非同期ランスタート")

    Alamofire.request(url1,method: .get).responseJSON { (response) in
    if let Json = response.result.value {
    json = JSON(Json)
    keepAlive = false
    print(json)
    }

    }
    /*
    while keepAlive &&
    runLoop.run(mode: RunLoopMode.defaultRunLoopMode, before: Date(timeIntervalSinceNow: 0.1)){
    print("run")
    }
    */
    print("非同期ランエンド")
    return json

    やはりここまでならif文は通るのでwhileの中から抜け出せないようです。
    whileに入ってからAPIのリクエスト(非同期の動作)が止まるということはありえますかね。重ね重ねすいません

    キャンセル

  • 2017/11/13 15:40

    この状態(whileを除く)の出力はこんな状態です。


    非同期ランスタート
    非同期ランエンド

    getRain-end
    {
    "ResultInfo" : {
    "Total" : 1,
    "Status" : 200,
    "Start" : 1,
    "Latency" : 0.013998999999999999,
    "Count" : 1,
    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

    キャンセル

  • 2017/11/13 15:57

    keepAliveのフラグを返すところはresponseブロックの最後じゃないですかね…
    試してみてください。

    Alamofire.request(url1,method: .get).responseJSON { (response) in
    if let Json = response.result.value {
    json = JSON(Json)
    // keepAlive = false responseブロックの最後に移動
    print(json)
    }
    keepAlive = false
    }

    キャンセル

  • 2017/11/13 16:28

    試してみましたが、特に変化は無いみたいでずっとwhile内でリクエストを待ってる状態です。

    whileの位置が悪いということはありますかね

    キャンセル

  • 2017/11/13 16:56

    whileの位置は正しいですよ。不思議ですねぇ。keepAliveがfalseになった時点で、run.loop云々の条件は評価せずにwhileにははいらないはずなんですが。

    キャンセル

  • 2017/11/13 22:23

    やはり、keepAlive = false の位置はresponseブロック終了直前に置くべきですよ。
    if let Json ... のブロック内に書いてしまうと、result.valueがnilの場合、ずっとウェイト状態になってしまいます。

    responseの内容はとりあえずおいといて、このコードで”非同期ランエンド”はやはり出ませんでしょうか?

    Alamofire.request(url1,method: .get).responseJSON { (response) in
    if let Json = response.result.value {

    json = JSON(Json)
    print(json)
    }

    keepAlive = false
    }

    while keepAlive &&
    runLoop.run(mode: RunLoopMode.defaultRunLoopMode, before: Date(timeIntervalSinceNow: 0.1)){
    print("run")
    }
    print("非同期ランエンド")
    return json

    キャンセル

  • 2017/11/13 23:46

    こんな時間までありがとうございます。

    コンソール出力でひたすらrunが出力されます...

    let now = getNowClockString()
    var intnow:Int = Int(now)!
    if(intnow%5 != 0){
    intnow -= intnow%5
    }
    intnow += time

    let strlat:String = String(latitude)
    let strlon:String = String(longitude)
    let url1 = "https://map.yahooapis.jp/weather/V1/place?coordinates="+strlon+","+strlat+"&appid="+YolpKey+"&output=json&past=1&interval=5&date="+now


    var json:JSON = "nil"
    var keepAlive = true
    let runLoop = RunLoop.current


    まさかとは思いますが変数宣言でおかしい部分があるのでしょうか。
    見にくいかとは思いますが見ていただけると助かります。
    getNowClockStringは現在時間を取得する関数です。

    キャンセル

+1

動作確認してませんが、

while keepAlive {
    runLoop.run(mode: .defaultRunLoopMode, before: Date(timeIntervalSinceNow: 0.1))
}

じゃないのかな。
run(mode:before:)は基本true返すみたいだし。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/11/13 16:31

    ありがとうございます。
    実行してみましたが同じ結果になりました。
    whileのループ内に入ってから非同期の方の処理が動いていないのでしょうか

    キャンセル

  • 2017/11/13 16:43

    ありゃ。勘違いか‥。
    runLoop.run(until: Date(timeIntervalSinceNow: 0.1))
    だとどうですか?

    (fromageblancさんの15:57のコメントに書いてある修正はしておいて下さい。今のままだとエラーのときにfalseになりませんので)

    キャンセル

  • 2017/11/13 16:57

    試してみましたが違うようです...

    しかしさっきよりも出力(非同期ランスタート → エンド)までに時間差がありました。この間隔をもっと広げられたら値が取れるのでしょうか

    キャンセル

  • 2017/11/13 17:02

    同期処理にするのではなく、非同期処理のままにすることをお勧めしますので、これ以上のコメントは控えます。(コードは元に戻して、fromageblancさんの回答に専念して下さい)

    キャンセル

  • 2017/11/13 17:07

    けっこうrun.loopは自分も使ってますので、該当のコードで問題ないと思うんですよね。
    コピペ元同じだしw

    キャンセル

  • 2017/11/13 17:35

    お二人ともありがとうございます。
    だいぶ長いコードになってきたのでもっと別の場所で間違ってるのかもしれません。見直してみます。

    キャンセル

  • 2017/11/14 14:08

    本当にお騒がせしました。
    別の非同期処理の中からgetRainの関数を呼び出していることが原因だったようで呼び出しの位置を外にしたら無事に処理待ちできました。
    記載した場所外のミスで無駄なお時間をとらせてしまい申し訳ありません。

    キャンセル

+1

確実に keepAlive が trueのままなだけなので、

        Alamofire.request(url1,method: .get).responseJSON { (response) in

            defer { keepAlive = false }  // 追加

            if let Json = response.result.value {
                json = JSON(Json)
         //       keepAlive = false
                }

        }

こうしましょう。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/11/13 19:28

    ありがとうございます。試してみましたが変化なしですすみません...。

    先にwhileの中に入ってしまうとどうしてもkeepAliveの値が更新されません。
    (リクエストの部分の動作が止まっている?)

    whileの中でif(json != "nil"){keepAlive=false}なども試してみましたがダメでした。(json = "nil"は定義しました。)

    キャンセル

  • 2017/11/14 14:08

    本当にお騒がせしました。
    別の非同期処理の中からgetRainの関数を呼び出していることが原因だったようで呼び出しの位置を外にしたら無事に処理待ちできました。
    記載した場所外のミスで無駄なお時間をとらせてしまい申し訳ありません。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.52%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • Swift

    7226questions

    Swiftは、アップルのiOSおよびOS Xのためのプログラミング言語で、Objective-CやObjective-C++と共存することが意図されています

  • Xcode

    4088questions

    Xcodeはソフトウェア開発のための、Appleの統合開発環境です。Mac OSXに付随するかたちで配布されています。