片付けよう

CoddyのUnityジャーニー「Unityスクリプトの基礎」セクションの一部。レッスン 78/85。

foreachでリストを走査している間にリストから項目を削除すると、スクリプトはInvalidOperationExceptionで停止します。代わりにforで逆順にループします。項目iを削除すると、その後ろにある項目だけが移動しますが、それらはループですでに処理済みです。

for (int i = coins.Count - 1; i >= 0; i--)
{
    if (coins[i].transform.position.y > 20f)
    {
        Destroy(coins[i]);
        coins.RemoveAt(i);
    }
}

チャレンジ

中級

床の下に落ちるブロックを取り除きましょう。

foreachをfor (int i = blocks.Count - 1; i >= 0; i--)に置き換え、これまでと同じようにblocks[i]を下へ動かします。
移動後、blocks[i].transform.position.y < -1fのとき、Destroy(blocks[i]);を呼び出し、その後blocks.RemoveAt(i);を呼び出します。

「プレイ」を押します。Hierarchyにブロックが数個以上残ることはありません。その後「確認」を押します。

自分で試してみよう

using System.Collections.Generic;
using UnityEngine;

public class Runner : MonoBehaviour
{
    [SerializeField] private float speed = 6f;
    [SerializeField] private float laneEdge = 4f;
    [SerializeField] private GameObject blockPrefab;
    [SerializeField] private float spawnEvery = 1f;
    [SerializeField] private float fallSpeed = 5f;

    private List<GameObject> blocks = new List<GameObject>();
    private float timer;

    void Update()
    {
        Vector3 p = transform.position;
        p.x += Input.GetAxisRaw("Horizontal") * speed * Time.deltaTime;
        p.x = Mathf.Clamp(p.x, -laneEdge, laneEdge);
        transform.position = p;

        timer += Time.deltaTime;
        if (timer >= spawnEvery)
        {
            timer -= spawnEvery;
            Vector3 spot = new Vector3(Random.Range(-laneEdge, laneEdge), 10f, 0f);
            blocks.Add(Instantiate(blockPrefab, spot, Quaternion.identity));
        }

        foreach (GameObject block in blocks)
        {
            block.transform.position += Vector3.down * fallSpeed * Time.deltaTime;
        }
    }
}