Unityのhttp通信の完了を待って次の処理ができない

2017/08/09 06:15

質問内容と回答が合致していないようです

2017/08/10 00:08

私は合致していると思いますよ。 Debug.Logの位置は変わっていますが、どちらも「Sendの処理の完了を待ってから、Debug.Logする」という感じになっております。

2017/08/10 01:50

sendの処理を待つのと結果の受信完了を待つのは同じでは無いと思ってたんですがこれで(isDoneも見ずに)正しく内容受信完了まで待てているんでしょうか…？

ShiroKuroShiro

2017/08/15 03:37

yield return www.Send();がそもそもSend処理終了まで(isDoneがTrueになるまで)待つ処理ですので問題ないはずです。 isDoneは、Send処理中に何かしたいときに使うイメージですね。

行動規範の内容に同意します

提示されたソースだと実行できません。
・yield return www.Send ();はIEnumerator内でしか動作しません。
・}が1つ多いです。

なので、推測ですが、

C#
1        string textdata = "";
2
3        UnityWebRequest www = UnityWebRequest.Get ("http://xxx.xxx.xxx.xxx/xxx.txt");
4        yield return www.Send ();
5
6        if (www.isError) {
7            Debug.Log (www.error);
8        } else {
9            textdata = www.downloadHandler.text;
10        }

以上の処理は、IEnumerator内に書かれていると思います。
これは、コルーチンというもので、通常の関数とは異なります。
この場合、yield return www.Send ();では、送信処理を行うだけではなく、受信するまで待機する、ということも行われます。
そのため、受信し終わってから、if (www.isError) {以下の処理が行われます。
受信するまでは、Unity上ではUpdate()などの他の処理が行われます。

なので、textdataに何か文字列が入ってからDebug.Log(textdata);を実行するというような処理にする必要があります。

投稿2017/08/07 01:56

総合スコア9797

2017/08/07 02:16

申し訳ありません、ソースが間違っていました。質問内のソースを修正したので見ていただきたいのですが、ご指摘の通りhttpでの取得はIEnumerator内で行なっていましてその中では逐次実行しています。やりたいこととしてはReadData()のStartCorutine()以下に取得したデータを使って処理を進めたいのですが、コルーチン処理なので非同期で切り離され、Debug.Log ("finished: " + textdata);のところのtextdataは空のままになってしまいます。コルーチンの処理を待って次の処理を実行することはできないでしょうか？

2017/08/07 02:53

思いついた限りでは、以下の方法が考えられます。・Update()内に「もし、textdataが変更されていれば、Debug.Log()する」という処理を書く・Debug.Log()をGetProblem()の中に書いてしまう・「GetProblem()の実行が完了してから、Debug.Log()する」という内容の、新しいコルーチンを作る質問者さんの意図を私なりにくみ取ってみた感じだと、やりたいことに近いのはおそらく3番目だと思います。ちなみに、GetProblem()の実行が完了するまで待つ、ということはHiroshiWatanabeさんの「isDone()」を使う方法の他にも、「yield return GetProblem();」と書く方法もあります。

行動規範の内容に同意します

ベストアンサー

通信が終了したかを isDone で判断して(isDoneがtrueになるまでは yield return で待つようなループにして)から終了後のデータ参照（やエラー判定）をしてはどうでしょうか？
UnityWebRequest.isDone

(具体的事例を追記)

c#
1yield return www.Send(); 
2
3```と
4```c#
5if (www.isError) {
6
7```の間に
8```c#
9while (!www.isDone) {
10  yield return null;
11}
12
13```を入れる

投稿2017/08/07 02:41

編集2017/08/07 04:39

総合スコア2160

2017/08/07 09:08

GetProblem内で while (!www.isDone) { //yield return null; } として最後に yield return null;を入れると取得するまでは待って、 ReadData()内のDebug.Logで表示されるようになりました。ありがとうございます。

2017/08/07 11:25

本当に正しい判断ですか？ループ内の「yield return null;」を消すとUnity全体が固まりますよ

2017/08/09 00:55

ループ内のyield return null; が無いとiOS等環境によってはそこでフリーズして進行不能になったりするリスクがありますので外さない方がいいと思いますよ

2017/08/10 00:14

追記すると、「yield ～ ;」は「他の処理をしながら、待機する」です。「yield ～ ;」をせずに何秒もかかる処理を行ってしますと、出力したいDebug.Logだけでなく、他のオブジェクトのUpdate()処理なども処理されなくなってしまいます。もしかすると、その中にUnityの重要な処理が含まれているかもしれません。そのため、Unityでは「～している間待機する」といった処理はコルーチン内で「yield ～ ;」を使うことになっております。使い方は、ShiroKuroShiroさんの回答の1番目のソースを確認してください。ちなみに、結果的にHiroshiWatanabeさんの回答が間違えということになりますが、「yield return www.Send();」の時点で「受信まで待機する」という処理が行われていますので、「while (!www.isDone) { yield return null; }」は使う必要はないです。

2017/08/10 01:56 編集

上にも記載しましたがsendの終了をまつ事と結果のテキスト受信完了を待つ事は同じなのでしょうか？元々の質問者さんはyield return www.Send ();しているのに結果の受取完了していない状態でその後のログ出力で参照されているという状況で質問に至ったようですしyield return で指定するのがwww自体ならわかるのですがwww.Sendを待つだけではisDoneでの判定が必要なのだろうと解釈したんですが…しかし実際の所は私も普段はWWWクラスしか利用しておらずUnityWebRequest は使った事がないため未検証な想像でした。間違った回答になっていたのなら申し訳ありませんでした。

2017/08/10 06:56

私が試してみた限りでは、Send()だけでも文字列の受け取りは問題なくできました。また、質問者さんのソースでも、「yield return www.Send ();」の直後に「// ここでは表示される Debug.Log (textdata);」と書いているので、文字列自体は受け取れてはいるのではないかと思います。なので、私はisDoneの判定は必要ないのではないか、と判断しました。私の解釈では、質問者さんの意図は「StartCoroutine()の直後にDebug.Log()すると何も出力されない」だと思っております。そうすれば、「yield return null;」を消した理由も以下のように推測できますので、妥当ではないかと。「yield return null;」のタイミングで「Debug.Log」が実行されてしまうため、何も表示されない →「yield return null;」を消したらwhileループの間は、「Debug.Log」が実行されない（Unity全体が止まっているから）ため、うまくいった（ように見える）

2017/08/10 09:34

なるほど理解できました。お手数お掛けしました。

2017/08/11 00:49

すみません。解決できたと思っていたのでチェックしておりませんでした。たしかにyield return null;を外すともしDoneにならないときに固まってしまうため時間計測して一定時間がたった場合は抜けるように処理を追加しました。 System.Diagnostics.Stopwatch sw = new System.Diagnostics.Stopwatch (); sw.Start(); while (!www.isDone && sw.Elapsed.TotalSeconds < 20.0) { } としています。 Debug.Log ("finished: " + textdata);のところにtextdataを元にした処理を入れたいのでyield return nullで抜けてしまうとtextdataが空の状態で処理が進んでしまい困っていました。本当はネットワーク処理の部分をコルーチンでなくかければよいのですが。

2017/08/11 01:17

その方法でも最大20秒間、Unity『全体』が固まってしまいます。ネットワーク処理の部分がコルーチンになっている理由は、受信中に他の処理を行うことが出来なくなるからです。 Unityの基本的な処理はシングルスレッドなので、「yield ～ ;」としないと、他の処理に移ることが出来ません。「～まで待機する」という処理は正しくコルーチンが得意とする点なので、コルーチンに書き換えてしまうのが一番だと思います。

2017/08/11 01:58

サーバーから取得したtextdataにクイズの問題が入っており、そのtextdataがないとゲーム画面のクイズ出題部分が空白になってしまいます。なのでunity全体が固まっても仕方がないかなと考えていました。ロード画面を表示しておいて、コルーチンで裏で取ってきて、取得でき次第表示の方が健全（？）でしょうか？

2017/08/11 06:56

Unity全体が固まってしまうと、システムの不具合が起きるかもしれませんし、そうでなくともユーザーが不具合だと思われる可能性もあります。私としては、前の問題の最中にコルーチンで問題を取ってきてしまうのがいいと思います。