時間経過によって障害物を変化させる方法

Question

前提・実現したいこと

時間経過によって、ブロックの数を増やし、スピードを速くしたいと考えています。
製作途中でエラーが発生したため、解決方法を教えていただきたいです。

この問題が解決した後は、BlockMgrを更に増やしたいと考えています。

発生している問題

一定時間経つと、ブッロクが作れなくなる。
（初めの８秒程度はエラーは発生しません）

エラーコード

MissingReferenceException: The object of type 'GameObject' has been destroyed but you are still trying to access it.
Your script should either check if it is null or you should not destroy the object.
UnityEngine.Object.Internal_InstantiateSingle (UnityEngine.Object data, UnityEngine.Vector3 pos, UnityEngine.Quaternion rot) (at <10564ed154d647e194bef4aef8878649>:0)
UnityEngine.Object.Instantiate (UnityEngine.Object original, UnityEngine.Vector3 position, UnityEngine.Quaternion rotation) (at <10564ed154d647e194bef4aef8878649>:0)
UnityEngine.Object.Instantiate[T] (T original, UnityEngine.Vector3 position, UnityEngine.Quaternion rotation) (at <10564ed154d647e194bef4aef8878649>:0)
BlockMgr2.Update () (at Assets/Scripts/BlockMgr2.cs:51)

該当のソースコード

using System.Collections;
using System.Collections.Generic;
using UnityEngine;

public class BlockMgr : MonoBehaviour
{
    // 生成するBlockオブジェクト
    public GameObject block;

    // 0になったらBlockオブジェクトを生成
    float _timer = 0;
    // トータルの経過時間を保持
    float _totalTime = 0;
    // ①ブロック生成回数
    int _cnt = 0;

    void Start()
    {

    }

    void Update()
    {
        // 経過時間を差し引く
        _timer -= Time.deltaTime;
        // トータル時間を加算
        _totalTime += Time.deltaTime;

        if (_timer < 0)
        {
            // 0になったのでBlock生成
            // BlockMgrの場所から生成
            Vector3 position = transform.position;
            // ※上下(±3)のランダムな位置に出現させる
            position.y = Random.Range(-3, 5);
            // プレハブをもとにBlock生成
            GameObject obj = Instantiate(block, position, Quaternion.identity);
            // Blockオブジェクトの「Block」スクリプトを取得する
            Block blockScript = obj.GetComponent<Block>();
            // 速度を計算して設定
            // 基本速度100に、経過時間x10を加える
            float speed = 100 + (_totalTime * 10);
            blockScript.SetSpeed(-speed); // 左方向なのでマイナス

            // ②生成回数をカウントアップ
            _cnt++;
            if (_cnt % 10 < 3)
            {
                // 0.1秒後にまた生成する
                _timer += 0.1f;
            }
            else
            {
                // 1秒後にまた生成する
                _timer += 1;
            }
        }
    }
}

using System.Collections;
using System.Collections.Generic;
using UnityEngine;

public class BlockMgr2 : MonoBehaviour
{
    // 生成するBlockオブジェクト
    public GameObject block;

    // 0になったらBlockオブジェクトを生成
    float _timer = 0;
    // トータルの経過時間を保持
    float _totalTime = 0;
    // ①ブロック生成回数
    int _cnt = 0;



    bool _isStart = false;

    void Start()
    {
        Invoke("second", 3);
    }

    void second()
    {
        _isStart = true;
    }


    void Update()
    {
        if (!_isStart)
        {
            return;
        }
        // 経過時間を差し引く
        _timer -= Time.deltaTime;
        // トータル時間を加算
        _totalTime += Time.deltaTime;

        if (_timer < 0)
        {
            // 0になったのでBlock生成
            // BlockMgrの場所から生成
            Vector3 position = transform.position;
            // ※上下(±3)のランダムな位置に出現させる
            position.y = Random.Range(-3, 5);
            // プレハブをもとにBlock生成
            GameObject obj = Instantiate(block, position, Quaternion.identity);
            // Blockオブジェクトの「Block」スクリプトを取得する
            Block blockScript = obj.GetComponent<Block>();
            // 速度を計算して設定
            // 基本速度100に、経過時間x10を加える
            float speed = 100 + (_totalTime * 10);
            blockScript.SetSpeed(-speed); // 左方向なのでマイナス

            // ②生成回数をカウントアップ
            _cnt++;
            if (_cnt % 10 < 3)
            {
                // 0.1秒後にまた生成する
                _timer += 0.1f;
            }
            else
            {
                // 1秒後にまた生成する
                _timer += 1;
            }
        }
    }
}

using System.Collections;
using System.Collections.Generic;
using UnityEngine;

// ブロック
public class Block : MonoBehaviour
{
    Rigidbody2D _rigidbody;
    float _speed = -100; // 移動速度
   
    void Start()
    {
        // 物理挙動コンポーネントを取得
        _rigidbody = GetComponent<Rigidbody2D>();
        // 力を加える
        _rigidbody.AddForce(new Vector2(_speed, 0));
       
    }

    void Update()
    {
        Vector2 position = transform.position;
        if (position.x < GetLeft())
        {
            Destroy(gameObject); // 画面外に出たので消す.
        }
    }

    float GetLeft()
    {
        // 画面の左下のワールド座標を取得する
        Vector2 min = Camera.main.ViewportToWorldPoint(Vector2.zero);
        return min.x;
    }

    public void SetSpeed(float speed)
    {
        _speed = speed;
    }

  
}

###追記（Block.cs）
Block.csのUpdate関数内のdestroyを削除したらエラーは起こりませんでした。

void Update()
    {
        Vector2 position = transform.position;
        if (position.x < GetLeft())
        {
            //Destroy(gameObject); // 画面外に出たので消す.
        }
    }

補足情報（FW/ツールのバージョンなど）

unity20203.1f1

Comments

[sakura_hana]

エラーメッセージを読みましょう。「Destroyされたオブジェクトにアクセスしようとしている」というエラーが「at Assets/Scripts/BlockMgr2.cs:51」、つまりBlockMgr2の51行目で起こっています。
その行を見るに「block」がどこかでDestroyしていそうです。対象のオブジェクトが最初からシーン上にありBlockクラスUpdate内Destroyで消えている可能性が高そうですがどうなっていますか？　他の理由で消えていたりしませんか？

[unitykun]

追記に書かせていただきました。
ただ、重くなるとは思うので思うので、別の方法で画面外に出た場合のブロック削除方法を教えていただきたいです。

Answer 1

Instantiateでの生成元となるオブジェクト（＝block変数にセットされているオブジェクト）がDestroyされるのが問題です。
それ以外の、Instantiateで生成した後のオブジェクトはDestroyしても問題ありません。
「コピー元が存在しなければコピーはできない」「コピー後のオブジェクトを削除してもコピー元に影響は与えない」とイメージするとわかりやすいと思います。

ということは、「生成元だったらDestroyしない」「生成元をシーン上に配置しない」「そもそもDestroyしない」のいずれかで解決できます。

■生成元だったらDestroyしない
Destroy(gameObject);の前にif (gameObject.name != "生成元のオブジェクト名")という条件分岐を付け足すなどして、自身が生成元のオブジェクトである場合は削除されないようにしましょう。

■生成元をシーン上に配置しない
生成元のオブジェクトをProjectパネルにドラッグ＆ドロップし、そのProjectパネル内のオブジェクトをBlockMgrクラスの変数にドラッグ＆ドロップし、シーン上にある生成元オブジェクトは削除してください。
これで生成元オブジェクトはプレハブ化され、初期状態ではシーン上に存在しないのでUpdateも実行されず、Destroyすることもありません。（プレハブについての説明は割愛するので必要に応じて調べてください）

■そもそもDestroyしない
「ブロックが目的地に到着したら初期位置に移動する」「ブロックが既にある場合は新規生成しない」というコードに変更します。
考え方としては「オブジェクトプール」というものになります。こちらもググってみてください。

なお、いずれの場合もBlockMgr・BlockMgr2と複数の同内容のクラスを用意する必要性が現状見当たらないのでその点は再検討した方がよいかと思います。

Comments

[unitykun]

回答ありがとうございます。
一つ目の案を参考させていただきました。
BlockMgr・BlockMgr2の件についてもアドバイスありがとうございます。すごく修正しやすいスクリプトに変えることができました。

```
using System.Collections;
using System.Collections.Generic;
using UnityEngine;

public class BlockMgr : MonoBehaviour
{
// 生成するBlockオブジェクト
public GameObject block;

// 0になったらBlockオブジェクトを生成
float _timer = 0;
// トータルの経過時間を保持
float _totalTime = 0;
// ①ブロック生成回数
int _cnt = 0;

int mord = 1;

void Start()
{
Invoke("normal", 3);
Invoke("hard", 6);
Invoke("difficult", 9);
Invoke("impossible", 12);
}

void normal()
{
mord = 2;
}

void hard()
{
mord = 3;
}

void difficult()
{
mord = 4;
}

void impossible()
{
mord = 5;
}

void Update()
{
for (int i = 0; i < mord; i++)
{
// 経過時間を差し引く
_timer -= Time.deltaTime;
// トータル時間を加算
_totalTime += Time.deltaTime;

if (_timer < 0)
{
// 0になったのでBlock生成
// BlockMgrの場所から生成
Vector3 position = transform.position;
// ※上下(±3)のランダムな位置に出現させる
position.y = Random.Range(-3, 5);
// プレハブをもとにBlock生成
GameObject obj = Instantiate(block, position, Quaternion.identity);
// Blockオブジェクトの「Block」スクリプトを取得する
Block blockScript = obj.GetComponent<Block>();
// 速度を計算して設定
// 基本速度100に、経過時間x10を加える
float speed = 100 + (_totalTime * 20);
blockScript.SetSpeed(-speed); // 左方向なのでマイナス

// ②生成回数をカウントアップ
_cnt++;
if (_cnt % 10 < 3)
{
// 0.1秒後にまた生成する
_timer += 0.1f;
}
else
{
// 1秒後にまた生成する
_timer += 1;
}
}

}

}
}
```