質問編集履歴

解決記録を追記しました．

2021/09/30 07:37

投稿

toririn_blue

スコア2

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -371,3 +371,205 @@
 回答する上で，足りない情報等ございましたら，お知らせください．
 ご指導のほどよろしくお願いいたします．
+### 解決記録
+Bongoさんのご指摘の通り，書き換えたところ目的通りのスクリプトが完成しました．
+完成したコード
+```
+using System.Collections;
+using System.Collections.Generic;
+using UnityEngine;
+/// <Summary>
+/// Prefabのオブジェクトを並べるサンプルスクリプトです。
+/// </Summary>
+public class PrefabPlaceSample : MonoBehaviour
+{
+    // オブジェクトを生成する元となるPrefabへの参照を保持します。
+    public GameObject prefabObj;
+    // 生成したオブジェクトの親オブジェクトへの参照を保持します。
+    public Transform parentTran;
+    // マテリアルを保持します。
+    public Material redMat;
+    public Material yellowMat;
+    public Material whiteMat;
+    public Material defaultMat;
+void Start()
+    {
+        CreateBlockObject();
+    }
+    /// <Summary>
+    /// Prefabからブロックとして使うオブジェクトを生成します。
+    /// </Summary>
+    void CreateBlockObject()
+    {
+        int row = 6;
+        int column = 3;
+        float xOffset = 2.5f;
+        float zOffset = 7.1f;
+        List<int> numbers = new List<int>();
+        for (int j = 0; j < row; j++)
+        {
+            numbers.Clear();
+            for (int k = 1; k <= column; k++)
+            {
+                // まず，List numbers に数字を追加します．
+                numbers.Add(k);
+            }
+            for (int i = 0; i < column; i++)
+            {
+                // ゲームオブジェクトを生成します。
+                GameObject obj = Instantiate(prefabObj, Vector3.zero, Quaternion.identity);
+                // ゲームオブジェクトの親オブジェクトを設定します。
+                obj.transform.SetParent(parentTran);
+                // ゲームオブジェクトの位置を設定します。
+                float xPos = xOffset * i;
+                float zPos = zOffset * j;
+                obj.transform.localPosition = new Vector3(xPos, 0.0f, zPos);
+                // マテリアルをランダムで設定します。
+                if (numbers.Count > 0)
+                {
+                    // ランダムに数字を取り出します．
+                    int index = Random.Range(0, numbers.Count);
+                    int ransu = numbers[index];
+                    Material mat = defaultMat;
+                    // 乱数の結果により，マテリアルを変更します．（修正済み）
+                    switch (ransu)
+                    {
+                        case 1:
+                            mat = redMat;
+                            break;
+                        case 2:
+                            mat = yellowMat;
+                            break;
+                        case 3:
+                            mat = whiteMat;
+                            break;
+                    }
+                    //重複を避けるため，利用した数字をnumbersから消します．
+                    numbers.RemoveAt(index);
+                    obj.GetComponent<MeshRenderer>().material = mat;
+                }
+            }
+        }
+    }
+}
+```
+#### 実行結果
+行内で，３色（色固定・重複無し）のターゲットを３つランダムに等間隔で配置できています．
+（画像は3列の場合）
+![完成したスクリプトの実行結果](26fabb6d11388538a1938f5061f64e22.png)

修正における間違えを見つけ，解決できたため．

2021/09/30 07:37

投稿

toririn_blue

スコア2

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -360,152 +360,6 @@
 ```
-###修正２
-// ※マテリアルの種類はcolumnと同数であることが暗黙の前提で、各列にそれぞれバラバラなマテリアルを
-// 割り当てるのだろうと勝手に解釈したのですが、もしそうであればnumbersに追加する値は下記のように
-// 1～columnになるんじゃないでしょうかね?
-Bongoさんのご指摘の通りです．各行内でのターゲットの色を重複無くすために，このようなコードを書いています．指摘していただいた通りに，下記のように書き換えてみました．
-修正したコード２
-```
-    void CreateBlockObject()
-    {
-        int row = 1;
-        int column = 3;
-        float xOffset = 2.5f;
-        float zOffset = 7.1f;
-        List<int> numbers = new List<int>();
-        for (int j = 0; j < row; j++)
-        {
-            numbers.Clear();
-            for (int k = 1; k <= column; k++)
-            {
-                // まず，List numbers に数字を追加します．
-                numbers.Add(k);
-            }
-            for (int i = 0; i < column; i++)
-            {
-                // ゲームオブジェクトを生成します。
-                GameObject obj = Instantiate(prefabObj, Vector3.zero, Quaternion.identity);
-                // ゲームオブジェクトの親オブジェクトを設定します。
-                obj.transform.SetParent(parentTran);
-                // ゲームオブジェクトの位置を設定します。
-                float xPos = xOffset * i;
-                float zPos = zOffset * j;
-                obj.transform.localPosition = new Vector3(xPos, 0.0f, zPos);
-                // マテリアルをランダムで設定します。
-                while (numbers.Count > 0)
-                {
-                     // ランダムに数字を取り出します．
-                    int index = Random.Range(0, numbers.Count);
-                    int ransu = numbers[index];
-                    Material mat = defaultMat;
-                    // 乱数の結果により，マテリアルを変更します．（修正済み）
-                    switch (ransu)
-                    {
-                        case 1:
-                            mat = redMat;
-                            break;
-                        case 2:
-                            mat = yellowMat;
-                            break;
-                        case 3:
-                            mat = whiteMat;
-                            break;
-                    }
-                    //重複を避けるため，利用した数字をnumbersから消します．
-                    numbers.RemoveAt(index);
-                    obj.GetComponent<MeshRenderer>().material = mat;
-                }
-            }
-        }
-    }
-}
-```
-#### 修正コード2の実行結果
-![修正コード2の実行結果](01a34c740fb473b135f2bb3a3329b9ad.png)
-修正したコード２を実行した結果，一番最初のターゲット以外はマテリアルが変更されませんでした．
 ### 最後に

1570pさんと，Bongoさんの指摘を受けて，修正した部分を追記しました．

2021/09/30 06:56

投稿

toririn_blue

スコア2

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -260,6 +260,252 @@
+###修正１
+1570pさんに，case 0 (case 3)（つまりmaterialをransuを使って変更しない）場合に何か起きているのでは？とご指摘していただいたので，defaultMatというmaterialから必ずransuを用いて，materialを変更するように，下記のように書き換えました．
+実行結果としては，変わりませんでした．
+修正したコード１
+```
+public class PrefabPlaceSample : MonoBehaviour
+{
+    // オブジェクトを生成する元となるPrefabへの参照を保持します。
+    public GameObject prefabObj;
+    // 生成したオブジェクトの親オブジェクトへの参照を保持します。
+    public Transform parentTran;
+    // マテリアルを保持します。
+    public Material redMat;
+    public Material yellowMat;
+    public Material whiteMat;
+    public Material defaultMat;
+〜〜〜
+                while (numbers.Count > 0)
+                {
+                     // ランダムに数字を取り出します．
+                    int index = Random.Range(0, numbers.Count);
+                    int ransu = numbers[index];
+                    Material mat = defaultMat;
+                    // 乱数の結果により，マテリアルを変更します．（修正済み）
+                    switch (ransu)
+                    {
+                        case 1:
+                            mat = redMat;
+                            break;
+                        case 2:
+                            mat = yellowMat;
+                            break;
+                        case 3:
+                            mat = whiteMat;
+                            break;
+                    }
+                    //重複を避けるため，利用した数字をnumbersから消します．
+                    numbers.RemoveAt(index);
+                    obj.GetComponent<MeshRenderer>().material = mat;
+                }
+〜〜〜
+```
+###修正２
+// ※マテリアルの種類はcolumnと同数であることが暗黙の前提で、各列にそれぞれバラバラなマテリアルを
+// 割り当てるのだろうと勝手に解釈したのですが、もしそうであればnumbersに追加する値は下記のように
+// 1～columnになるんじゃないでしょうかね?
+Bongoさんのご指摘の通りです．各行内でのターゲットの色を重複無くすために，このようなコードを書いています．指摘していただいた通りに，下記のように書き換えてみました．
+修正したコード２
+```
+    void CreateBlockObject()
+    {
+        int row = 1;
+        int column = 3;
+        float xOffset = 2.5f;
+        float zOffset = 7.1f;
+        List<int> numbers = new List<int>();
+        for (int j = 0; j < row; j++)
+        {
+            numbers.Clear();
+            for (int k = 1; k <= column; k++)
+            {
+                // まず，List numbers に数字を追加します．
+                numbers.Add(k);
+            }
+            for (int i = 0; i < column; i++)
+            {
+                // ゲームオブジェクトを生成します。
+                GameObject obj = Instantiate(prefabObj, Vector3.zero, Quaternion.identity);
+                // ゲームオブジェクトの親オブジェクトを設定します。
+                obj.transform.SetParent(parentTran);
+                // ゲームオブジェクトの位置を設定します。
+                float xPos = xOffset * i;
+                float zPos = zOffset * j;
+                obj.transform.localPosition = new Vector3(xPos, 0.0f, zPos);
+                // マテリアルをランダムで設定します。
+                while (numbers.Count > 0)
+                {
+                     // ランダムに数字を取り出します．
+                    int index = Random.Range(0, numbers.Count);
+                    int ransu = numbers[index];
+                    Material mat = defaultMat;
+                    // 乱数の結果により，マテリアルを変更します．（修正済み）
+                    switch (ransu)
+                    {
+                        case 1:
+                            mat = redMat;
+                            break;
+                        case 2:
+                            mat = yellowMat;
+                            break;
+                        case 3:
+                            mat = whiteMat;
+                            break;
+                    }
+                    //重複を避けるため，利用した数字をnumbersから消します．
+                    numbers.RemoveAt(index);
+                    obj.GetComponent<MeshRenderer>().material = mat;
+                }
+            }
+        }
+    }
+}
+```
+#### 修正コード2の実行結果
+![修正コード2の実行結果](01a34c740fb473b135f2bb3a3329b9ad.png)
+修正したコード２を実行した結果，一番最初のターゲット以外はマテリアルが変更されませんでした．
 ### 最後に

初心者マークの付け忘れ

2021/09/30 06:43

投稿

toririn_blue

スコア2

test CHANGED Viewed

File without changes

test CHANGED Viewed

	@@ -260,6 +260,8 @@
260 260
261 261
262 262
263	+
264	+
263 265	### 最後に
264 266
265 267