別の質問でもらったコード（unityで乱数を重複のないように繰り返す）を自分の意図するものに変える

Question

###前提・実現したいこと
先日　unityで乱数を重複させずにそれを繰り返すというコードの書き方がわからなくて質問させていただきました。（https://teratail.com/questions/94921）
私が書きたかったのは、https://teratail.com/questions/94741
ここを参考にして、テトリスのミノ（ブロック）７つをランダムで重複がなく生成させ、その７つが生成しきったら、またそれを繰り返す、というコードでした。
幸いなことにある方わかりやすいようにコードを書いてくださいました。
しかし、私はUnity及びC#の初心者なので、軽く理解できる程度で、
自分のUnityに組み込むことができませんでした。
実際に何をしたかったのかというと、下記のコードでは　テトリスのミノを３秒（おそらく）ごとに重複のないように７つを１つずつ生成し、それを繰り返す　となっているのですが、もともと私が自分で書いたコードではspawnNext()というのにしていて、別のオブジェクトからspawnNext()を出力させることで、ミノを生成させていました。　このコードのどの部分を変えれば　別のオブジェクトから出力が来るまで、ミノを出現させず、出力が来たら、生成するというコードに書き換えられますか？

###該当のソースコード

using UnityEngine;
using System.Linq;
using System;
using System.Collections;

public class createRandObjs : MonoBehaviour
{
    //とりあえずインスペクタからオブジェクトを追加
    [SerializeField]
    GameObject[] objcts;

    //何秒おきに出力するかを指定
    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]);
            idx++;

            yield return waitTime;
        }
    }
}

Comments

[kikukiku]

unityの開発経験がないため意図した具体的な回答はだせませんが、本問題を解決するための方向性だけ、意見させてください。プログラマであるならば質問者さんは自身の書いたコードはせめてすべて理解すべきかと思います。理解できていないと今回のように改修することで出来ません。現在理解できていないのであるならば、現在コードの理解できていない部分を理解してから、今回の質問に取り組んだ方が良いと思います。急がば回れです。なので、まずは現在理解できてない部分を個別に切り出して、具体的に質問した方が良いと思います。

Answer 1

遅くなりました。すみません。

怒られちゃいましたねｗ
ご自分でわかっていらっしゃると思うので折り合いつくとこまで進めたら少し勉強時間をはさんでみてもいいかもしれませんね。

Unityの本を買って説明を読みながら掲載されているサンプルを手打ちで打っていくとある程度わかってくると思います。Unityはどんどこ進んでいるので今は本で学ぶのが一番です。

ということでspawnNext()で呼ぶタイプに書き直しました。
相変わらず未検証です。コメントをたっぷり入れたので思ったような挙動にならなければご自身で修正してみてください。そして、どうしても分からなければその部分のコードと一緒に質問してみてください。

使い方
0. 下記コードをコピーしcreateRandObjsという名前でスクリプトとして保存

1. ゲームオブジェクトをヒエラルキーに作成

1. createRandObjs.csをオブジェクトにアタッチ

1. ゲームオブジェクトのインスペクタに配列が入れられるようになっているのでオブジェクトを追加

5. 別スクリプトから「createRandObjs.Instance.spawnNext();」を呼ぶ

以上です。

外から呼ばれるということなのでシングルトンにしました。

using UnityEngine;
using System.Linq;
using System;
using System.Collections;

public class createRandObjs : MonoBehaviour
{
	//とりあえずインスペクタからオブジェクトを追加
	[SerializeField]
	GameObject[] objcts;


	//外部から呼び出せるように静的なフィールドとプロパティを定義
	//そうしておくと外部からcreateRandObjs.Instance.spawnNext()で呼び出せます。
	static createRandObjs instance;
	public static createRandObjs Instance
	{
		get {
			//インスタンスがなければインスタンスを生成 Awakeより先にこのプロパティが呼ばれる可能性があるのでここでもチェックとnullであればインスタンス化します。
			if(instance == null) instance = (createRandObjs) FindObjectOfType( typeof(createRandObjs) );

			return instance;
		}
	}


	// オブジェクトを順番にカウントするための変数
	int objIdx;

	// オブジェクトのランダムなインデックスを格納する配列
	int[] objIdxAry;


	void Awake()
	{
		//インスタンスがなければインスタンス化
		if(instance == null) instance = (createRandObjs) this;
		//すでにインスタンスが存在すれば自身を破棄 インスタンスは1つでいいので重複対策です。
		else if( Instance != this ) Destroy(this);
	}


	/// <summary>
	/// オブジェクトを出力する
	/// </summary>
	void instantiateObj(int idx)
	{
		//とりあえずVector3.zero地点にオブジェクトを生成
		Instantiate(objcts[idx], Vector3.zero, Quaternion.identity);
	}


	/// <summary>
	/// 外部から呼ばれる オブジェクトを出力する
	/// </summary>
	public void spawnNext()
	{
		//オブジェクトが無ければ何もせず終了
		if(objcts.Length == 0) return;

		//objIdxが0またはobjctsの数に達したらリセット
		if(objIdx == 0 || objIdx == objcts.Length)
		{
			//objIdxをリセット
			objIdx = 0;
			//ランダムな配列を生成
			objIdxAry = Enumerable.Range(0, objcts.Length).OrderBy(n => Guid.NewGuid()).ToArray();
		}

		//objIdxAryのインデックスを元にオブジェクトを生成
		instantiateObj( objIdxAry[objIdx] );
		//objIdxを+1する
		objIdx++;
	}
}

呼ぶ側

createRandObjs.Instance.spawnNext();

[以降は読まなくてもいいです] とりあえずコード量を抑える為、必要最低限にしています。

どこまで完成しているか分からないのでなんとも言えないのですが今のコードでは不十分に感じます。考えられるのはこんな感じです。

• オブジェクトの移動完了などの状態を直接知ることができない
• テトリスの場合、マップ的なものを用意してx軸が埋まっていれば(略)などの処理の対応
• 消える時のアニメーション

などですかね？僕ならオブジェクト一つ一つにスクリプトをつけてそこで、自身の調整や管理と外部への自身の状態や消えるアニメーションのスタートやエンドなどを提供をする形にすると思います。

それをこんな感じでコントロール側で持つと思います。

//これをオブジェクトひとつひとつにアタッチする
public class objScript : MonoBehaviour
{
	public bool destroyAnime(int[] targetYs)
	{
		//略
              //色々して全部消えたらtrueを送る
              return true;
	}
}

//引数などは適当です。実際はint[]を渡す形で組まないかもしれません。
public class createRandObjs : MonoBehaviour
{
        // 生成する時についでにこの配列に参照を渡しておく
	List<objScript> objScripts = new List<objScript>();

	void destroyAnime()
	{
		//略

		//1,2列目を破棄
		var isDeleteObj = objScripts[0].destroyAnime(new int[]{ 1,2 });
               //もしオブジェクトが完全に消えたら何かする
                if(isDeleteObj) objScripts.RemoveAt(0);
	}

Comments

[Hayato555]

ご丁寧な説明　本当にありがとうございます。
書いていただいたコードを自分のものに合うように少しだけ手を加えた結果、無事自分の思っていたものにできました。
unityとC#については頑張って学んでいきたいと思います。ですが今作っているゲームをよりよくするためにまたご質問をさせていただくかもしれません。　また機会がありましたら　答えてください。m(_ _)mここまで長い間　初心者の自分にご丁寧に答えてくださり、本当にありがとうございました。

[Hayato555]

ベストアンサーにしてからかなり立ちましたが一つ気が付いたことがあります。　このオブジェクトを格納する配列に数字が入れられるときにランダムになっていないのでないでしょうか？　テトリスのミノを出現させると、オブジェクトの１から７と出てきます。　
もし時間がありましたら、もう一度コードを編集してはくれませんか？

[Hayato555]

大変申し訳ございません。
テトリスのミノのネクストというものを作っていて、このコードをいじってしまって、どうやらそれが原因でランダムに出てこなかったようです。　もしこれが自分で解決できないようだったら、別で質問させていただきます。ｍ（＿　＿）ｍ