編集履歴

回答編集履歴

ミスの修正

2017/10/05 19:56

投稿

スコア1724

test CHANGED Viewed

@@ -194,7 +194,7 @@
                 idx = 0;
-                objIdxAry = Enumerable.Range(0, objcts.Length).OrderBy(n => Guid.NewGuid()).Take(objcts.Length).ToArray();
+                objIdxAry = Enumerable.Range(0, objcts.Length).OrderBy(n => Guid.NewGuid()).ToArray();
             }

修正

2017/10/05 19:56

投稿

IShix

スコア1724

test CHANGED Viewed

@@ -4,11 +4,11 @@
 **[処理内容]**
-- objctsに指定したGameObjectをランダムで出力
+- 1. objctsに指定したGameObjectをランダムで出力
-- 出力完了後、リセットしランダムな配列を生成
+- 2. すべて出力完了後、配列をランダムにリセット
-- 永遠にループ
+- 1、2の順で永遠にループ
 - 止めたい場合はstopLandObjLoop()を実行する

追記

2017/10/04 07:43

投稿

IShix

スコア1724

test CHANGED Viewed

@@ -202,6 +202,8 @@
             instantiateObj(objIdxAry[idx]);
+            idx++;
             yield return waitTime;

コードを修正

2017/10/04 07:28

投稿

IShix

スコア1724

test CHANGED Viewed

@@ -1,6 +1,22 @@
-こんな感じでどうでしょうか？
+コードを大幅に修正しました。
+**[処理内容]**
+- objctsに指定したGameObjectをランダムで出力
+- 出力完了後、リセットしランダムな配列を生成
+- 永遠にループ
+- 止めたい場合はstopLandObjLoop()を実行する
+**コードは未検証です。コメントを入れているので挙動がおかしかったらご自分で修正してください。**
 ```C#
@@ -10,6 +26,8 @@
 using System;
+using System.Collections;
 public class createRandObjs : MonoBehaviour
@@ -24,41 +42,173 @@
-	void Start()
-	{
-		//objctsの数だけランダムな配列を10回生成してみる
-		for (int i = 0; i < 10; i++)
-		{
-			Debug.Log("-------------------");
-            var ary = Enumerable.Range(0, objcts.Length).OrderBy(n => Guid.NewGuid()).Take(objcts.Length).ToArray();
-			for (int s = 0; s < ary.Length; s++)
-			{
-                Debug.Log(string.Format("配列のインデックス : {0}" , ary[s]));
-				//とりあえずVector3.zero地点にオブジェクトを生成
-				Instantiate(objcts[s], Vector3.zero, Quaternion.identity);
-			}
-		}
-	}
+    //何秒おきに出力するかを指定
+    static readonly float LOOP_INTERVAL = 3f;
+	//Coroutineをキャッシュする変数
+	Coroutine landObjLoopCorutine;
+    void Start()
+    {
+		// 実行
+		startLandObjLoop();
+    }
+	/// <summary>
+	/// ループのスタート
+	/// </summary>
+	void startLandObjLoop()
+    {
+		//このGameObjectが非表示の時にlandObjLoopを実行されるとエラーになるのでその場合実行させない
+		if (!gameObject.activeInHierarchy)
+        {
+            #if UNITY_EDITOR
+            Debug.Log("このオブジェクト非表示中だから実行できないよ");
+            #endif
+            return;
+        }
+        //実行中なら重複するので一旦停止
+        stopLandObjLoop();
+        //オブジェクトの出力をスタート
+        StartCoroutine( landObjLoop() );
+    }
+    /// <summary>
+    /// ループのストップ
+    /// </summary>
+    void stopLandObjLoop()
+    {
+        if (landObjLoopCorutine != null) StopCoroutine(landObjLoopCorutine);
+    }
+	/// <summary>
+	/// オブジェクトを出力する
+	/// </summary>
+	void instantiateObj(int idx)
+    {
+		//とりあえずVector3.zero地点にオブジェクトを生成
+		Instantiate(objcts[idx], Vector3.zero, Quaternion.identity);
+    }
+	/// <summary>
+	/// objctsをランダムに出力し続ける
+	/// </summary>
+	IEnumerator landObjLoop()
+    {
+		//オブジェクトが無ければ何もせず終了
+		if (objcts.Length == 0) yield break;
+        var waitTime = new WaitForSeconds( LOOP_INTERVAL );
+        //順番にカウントするための変数
+        var idx = 0;
+        //ランダムなインデックスを格納する配列
+        var objIdxAry = new int[objcts.Length];
+        while(true)
+        {
+			//idxが0またはobjctsの数に達したらリセット
+			if(idx == 0 || idx == objcts.Length)
+            {
+                idx = 0;
+                objIdxAry = Enumerable.Range(0, objcts.Length).OrderBy(n => Guid.NewGuid()).Take(objcts.Length).ToArray();
+            }
+            instantiateObj(objIdxAry[idx]);
+            yield return waitTime;
+        }
+    }
 }
@@ -66,4 +216,6 @@
-このコードだとVector3.zero地点にすべてのオブジェクトが重なって生成されるので必要に応じて位置を調整してください。
+遅延処理 Coroutine（コルーチン）の使い方について過去に回答したので見てみてください。
+[https://teratail.com/questions/93115](https://teratail.com/questions/93115)

うわーーーまた修正します。

2017/10/04 07:24

投稿

IShix

スコア1724

test CHANGED Viewed

@@ -1,8 +1,6 @@
 こんな感じでどうでしょうか？
-コードは未検証です。
 ```C#
@@ -40,7 +38,7 @@
-            var ary = Enumerable.Range(0, objcts.Length - 1).OrderBy(n => Guid.NewGuid()).Take(objcts.Length).ToArray();
+            var ary = Enumerable.Range(0, objcts.Length).OrderBy(n => Guid.NewGuid()).Take(objcts.Length).ToArray();

またまた間違ってました。すみません。

2017/10/03 13:00

投稿

IShix

スコア1724

test CHANGED Viewed

@@ -40,7 +40,7 @@
-            var ary = Enumerable.Range(0, objcts.Length - 1).OrderBy(n => Guid.NewGuid()).Take(objcts.Length - 1).ToArray();
+            var ary = Enumerable.Range(0, objcts.Length - 1).OrderBy(n => Guid.NewGuid()).Take(objcts.Length).ToArray();
@@ -48,7 +48,7 @@
 			{
-                Debug.Log(string.Format("配列のインデックス : {1}" , ary[s]));
+                Debug.Log(string.Format("配列のインデックス : {0}" , ary[s]));

またまた修正です。すみません。

2017/10/03 12:57

投稿

IShix

スコア1724

test CHANGED Viewed

@@ -40,7 +40,7 @@
-            var ary = Enumerable.Range(0, objcts.Length - 1).OrderBy(n => Guid.NewGuid()).Take(5).ToArray();
+            var ary = Enumerable.Range(0, objcts.Length - 1).OrderBy(n => Guid.NewGuid()).Take(objcts.Length - 1).ToArray();

すみません。追記

2017/10/03 12:53

投稿

IShix

スコア1724

test CHANGED Viewed

@@ -1,6 +1,8 @@
 こんな感じでどうでしょうか？
+コードは未検証です。
 ```C#

修正

2017/10/03 12:52

投稿

IShix

スコア1724

test CHANGED Viewed

@@ -16,7 +16,7 @@
 {
-    //とりあえずインスペクタからオブジェクトを追加 7個追加してください。
+    //とりあえずインスペクタからオブジェクトを追加
     [SerializeField]

修正

2017/10/03 12:51

投稿

IShix

スコア1724

test CHANGED Viewed

@@ -12,7 +12,7 @@
-public class getRandObj : MonoBehaviour
+public class createRandObjs : MonoBehaviour
 {

誤字修正

2017/10/03 12:50

投稿

IShix

スコア1724

test CHANGED Viewed

@@ -28,7 +28,7 @@
 	{
-		//objctsの数だけランダムな配列を10回表示してみる
+		//objctsの数だけランダムな配列を10回生成してみる
 		for (int i = 0; i < 10; i++)
@@ -63,3 +63,7 @@
 }
 ```
+このコードだとVector3.zero地点にすべてのオブジェクトが重なって生成されるので必要に応じて位置を調整してください。