質問編集履歴

forを使って何度もコルーチンを呼び出そうとしましたがうまくいきませんでした。

2021/01/08 12:57

投稿

ActionStudio

スコア39

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -532,138 +532,140 @@
     {
+        for (int i = 0; i < 3; i++)
+        {
+            StartCoroutine("transformMug");
+        }
+    }
+    void Update()
+    {
+        Debug.Log("計測中");
+        if(counting == true)
+        {
+            seconds += Time.deltaTime;
+        }
+    }
+    //マグを動かす
+    IEnumerator transformMug()
+    {
+        Debug.Log("コルーチン開始");
         counting = false;
-        StartCoroutine("transformMug");
-        StartCoroutine("transformMug");
-        StartCoroutine("transformMug");
-    }
-    void Update()
-    {
-        Debug.Log("計測中");
-        if(counting == true)
-        {
-            seconds += Time.deltaTime;
-        }
-    }
-    //マグを動かす
-    IEnumerator transformMug()
-    {
-        Debug.Log("コルーチン開始");
+        //ランダムな数値を生成
+        value = Random.Range(0, 3);
+        //ワールド座標の中心点を決める
+        if (value == 1)
+        {
+            _center.z = 25.0f;
+        }
+        else if (value == 2)
+        {
+            _center.z = -25.0f;
+        }
+        Debug.Log("マグ移動開始");
+        counting = true;
+        while(seconds <= _period)
+        {
+            //端のマグが動く
+            if (value == 0)
+            {
+                // 中心点centerの周りを、軸axisで、period周期で円運動
+                Mug1.transform.RotateAround(_center, _axis, 180 / _period * Time.deltaTime);
+                Mug3.transform.RotateAround(_center, _axis, 180 / _period * Time.deltaTime);
+                yield return null;
+            }
+            else if (value == 1)//右側のマグが動く
+            {
+                // 中心点centerの周りを、軸axisで、period周期で円運動
+                Mug1.transform.RotateAround(_center, _axis, 180 / _period * Time.deltaTime);
+                Mug2.transform.RotateAround(_center, _axis, 180 / _period * Time.deltaTime);
+                yield return null;
+            }
+            else//左側のマグが動く
+            {
+                // 中心点centerの周りを、軸axisで、period周期で円運動
+                Mug2.transform.RotateAround(_center, _axis, 180 / _period * Time.deltaTime);
+                Mug3.transform.RotateAround(_center, _axis, 180 / _period * Time.deltaTime);
+                yield return null;
+            }
+        }
         counting = false;
-        //ランダムな数値を生成
-        value = Random.Range(0, 3);
-        //ワールド座標の中心点を決める
-        if (value == 1)
-        {
-            _center.z = 25.0f;
-        }
-        else if (value == 2)
-        {
-            _center.z = -25.0f;
-        }
-        Debug.Log("マグ移動開始");
-        counting = true;
-        while(seconds <= _period)
-        {
-            //端のマグが動く
-            if (value == 0)
-            {
-                // 中心点centerの周りを、軸axisで、period周期で円運動
-                Mug1.transform.RotateAround(_center, _axis, 180 / _period * Time.deltaTime);
-                Mug3.transform.RotateAround(_center, _axis, 180 / _period * Time.deltaTime);
-                yield return null;
-            }
-            else if (value == 1)//右側のマグが動く
-            {
-                // 中心点centerの周りを、軸axisで、period周期で円運動
-                Mug1.transform.RotateAround(_center, _axis, 180 / _period * Time.deltaTime);
-                Mug2.transform.RotateAround(_center, _axis, 180 / _period * Time.deltaTime);
-                yield return null;
-            }
-            else//左側のマグが動く
-            {
-                // 中心点centerの周りを、軸axisで、period周期で円運動
-                Mug2.transform.RotateAround(_center, _axis, 180 / _period * Time.deltaTime);
-                Mug3.transform.RotateAround(_center, _axis, 180 / _period * Time.deltaTime);
-                yield return null;
-            }
-        }
-        counting = false;
         //初期位置に移動
@@ -696,7 +698,9 @@
         Mug3.transform.position = pos3;
+        yield return new WaitForSeconds(1.0f);

コルーチンを使って処理を行えるようにしました。しかし問題も発生しました。

2021/01/08 12:57

投稿

ActionStudio

スコア39

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -470,19 +470,251 @@
 ```
-しかし、問題も発生してしまいました。
-######発生したエラーコード
-Assets\Scripts\Operator.cs(45,17): error CS0161: 'Operator.transformMug()': not all code paths return a value
-######まだ解決していない部分
-yield break;
-などの使い方がわかりません。上のコードの場合、どの部分に書き加えればいいのでしょうか。
+#その次にこのように変更しました
+```C#
+using System.Collections;
+using System.Collections.Generic;
+using UnityEngine;
+public class Operator : MonoBehaviour
+{
+    //動かす物体を指定
+    public GameObject Mug1;
+    public GameObject Mug2;
+    public GameObject Mug3;
+    //クラス内で使えるようにする
+    int value;
+    //ワールド座標の中心点
+    private Vector3 _center = new Vector3(0.0f, 0.0f, 0.0f);
+    //回転軸
+    private Vector3 _axis = Vector3.up;
+    //円運動周期
+    public float _period;
+    //実行時間指定
+    public float seconds;
+    bool counting = false;
+    void Start()
+    {
+        counting = false;
+        StartCoroutine("transformMug");
+        StartCoroutine("transformMug");
+        StartCoroutine("transformMug");
+    }
+    void Update()
+    {
+        Debug.Log("計測中");
+        if(counting == true)
+        {
+            seconds += Time.deltaTime;
+        }
+    }
+    //マグを動かす
+    IEnumerator transformMug()
+    {
+        Debug.Log("コルーチン開始");
+        counting = false;
+        //ランダムな数値を生成
+        value = Random.Range(0, 3);
+        //ワールド座標の中心点を決める
+        if (value == 1)
+        {
+            _center.z = 25.0f;
+        }
+        else if (value == 2)
+        {
+            _center.z = -25.0f;
+        }
+        Debug.Log("マグ移動開始");
+        counting = true;
+        while(seconds <= _period)
+        {
+            //端のマグが動く
+            if (value == 0)
+            {
+                // 中心点centerの周りを、軸axisで、period周期で円運動
+                Mug1.transform.RotateAround(_center, _axis, 180 / _period * Time.deltaTime);
+                Mug3.transform.RotateAround(_center, _axis, 180 / _period * Time.deltaTime);
+                yield return null;
+            }
+            else if (value == 1)//右側のマグが動く
+            {
+                // 中心点centerの周りを、軸axisで、period周期で円運動
+                Mug1.transform.RotateAround(_center, _axis, 180 / _period * Time.deltaTime);
+                Mug2.transform.RotateAround(_center, _axis, 180 / _period * Time.deltaTime);
+                yield return null;
+            }
+            else//左側のマグが動く
+            {
+                // 中心点centerの周りを、軸axisで、period周期で円運動
+                Mug2.transform.RotateAround(_center, _axis, 180 / _period * Time.deltaTime);
+                Mug3.transform.RotateAround(_center, _axis, 180 / _period * Time.deltaTime);
+                yield return null;
+            }
+        }
+        counting = false;
+        //初期位置に移動
+        Transform Mug1t = Mug1.transform;
+        Transform Mug2t = Mug2.transform;
+        Transform Mug3t = Mug3.transform;
+        Vector3 pos1 = Mug1.transform.position;
+        pos1.z = 50f;
+        Vector3 pos2 = Mug2.transform.position;
+        pos2.z = 0f;
+        Vector3 pos3 = Mug3.transform.position;
+        pos3.z = -50f;
+        Mug1.transform.position = pos1;
+        Mug2.transform.position = pos2;
+        Mug3.transform.position = pos3;
+        yield break;
+    }
+}
+```
+初期位置に移動はあまり気にしないでください。
+コルーチンを使って、処理を行うようにしました。
+######しかし、問題も発生してしまいました。
+なぜか、二度目のカップを移動させる処理で、前回の処理と同じように移動してしまいます。毎度ランダムな数値を設定しているはずなのですがうまくいきません。どうすればいいでしょうか。

コードを変更・改善しました。しかし問題も発生してしまいました。

2021/01/08 09:39

投稿

ActionStudio

スコア39

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -278,7 +278,213 @@
 ```
+###このように変更しました
+```C#
+using System.Collections;
+using System.Collections.Generic;
+using UnityEngine;
+public class Operator : MonoBehaviour
+{
+    //動かす物体を指定
+    public GameObject Mug1;
+    public GameObject Mug2;
+    public GameObject Mug3;
+    //クラス内で使えるようにする
+    int value;
+    //ワールド座標の中心点
+    private Vector3 _center = new Vector3(0.0f, 0.0f, 0.0f);
+    //回転軸
+    private Vector3 _axis = Vector3.up;
+    //円運動周期
+    public float _period;
+    //実行時間指定
+    public float seconds;
+    bool counting = false;
+    void Start()
+    {
+        counting = false;
+        StartCoroutine("transformMug");
+    }
+    void Update()
+    {
+        if(counting == true)
+        {
+            seconds += Time.deltaTime;
+        }
+    }
+    //マグを動かす
+    IEnumerator transformMug()
+    {
+        counting = false;
+        //ランダムな数値を生成
+        value = Random.Range(0, 1);
+        //ワールド座標の中心点を決める
+        if (value == 1)
+        {
+            _center.z = 25.0f;
+        }
+        else if (value == 2)
+        {
+            _center.z = -25.0f;
+        }
+        counting = true;
+        while(seconds <= _period)
+        {
+            //端のマグが動く
+            if (value == 0)
+            {
+                // 中心点centerの周りを、軸axisで、period周期で円運動
+                Mug1.transform.RotateAround(_center, _axis, 180 / _period * Time.deltaTime);
+                Mug3.transform.RotateAround(_center, _axis, 180 / _period * Time.deltaTime);
+            }
+            else if (value == 1)//右側のマグが動く
+            {
+                // 中心点centerの周りを、軸axisで、period周期で円運動
+                Mug1.transform.RotateAround(_center, _axis, 180 / _period * Time.deltaTime);
+                Mug2.transform.RotateAround(_center, _axis, 180 / _period * Time.deltaTime);
+            }
+            else//左側のマグが動く
+            {
+                // 中心点centerの周りを、軸axisで、period周期で円運動
+                Mug2.transform.RotateAround(_center, _axis, 180 / _period * Time.deltaTime);
+                Mug3.transform.RotateAround(_center, _axis, 180 / _period * Time.deltaTime);
+            }
+        }
+        counting = false;
+        Mug1.transform.Translate(0, 20, 50);
+        Mug2.transform.Translate(0, 20, 0);
+        Mug3.transform.Translate(0, 20, -50);
+    }
+}
+```
+しかし、問題も発生してしまいました。
+######発生したエラーコード
-このように、Updateに入っていたものをMugに入れ、ifをwhileにし、startでMugを呼び出すことで、マグカップがランダムに何回も移動できるようにしようとしたのですが、これではupdateが呼び出されず時間管理ができずに、startが無限ループになってしまいました。
+Assets\Scripts\Operator.cs(45,17): error CS0161: 'Operator.transformMug()': not all code paths return a value
+######まだ解決していない部分
+yield break;
+などの使い方がわかりません。上のコードの場合、どの部分に書き加えればいいのでしょうか。
 ### 補足情報（FW/ツールのバージョンなど）

問題点について、わかりやすいようにしました。

2021/01/07 23:39

投稿

ActionStudio

スコア39

test CHANGED Viewed

	@@ -1 +1 @@
1	- オブジェクトをランダムに動かす操作を繰り返したい
1	+ オブジェクトをランダムに動かす操作を繰り返したい（カップシャッフルゲーム）

test CHANGED Viewed

@@ -2,7 +2,7 @@
-unityで、C#を使ってマグカップを何度か動かし、最初に目印を入れておいたマグカップがどこに移動したかを当てるゲームを作っています。
+unityで、C#を使ってマグカップを何度か動かし、最初に目印を入れておいたマグカップがどこに移動したかを当てるカップシャッフルゲームを作っています。
 ランダムに移動させたのちに、また同じ操作を繰り返して何度も位置を変えたいのですが、うまくいきません。
@@ -12,7 +12,11 @@
-ランダムに移動する操作が一度しか行えず、複数回行う方法がわからない。
+1.ランダムに移動する操作が一度しか行えず、複数回行う方法がわからない。
+2.時間計測でオブジェクトを動かしているため、時間計測を合わせて実行しなければいけないが、処理を繰り返す場合にその実行方法がわからない。
+3.カップを移動させた後もう一度入れ替えの操作を繰り返した場合、カップが想定した位置でなくなってしまうため、想定外の場所に移動してしまう。