Ошибка с листом [РЕШЕНО]

[trololoid]

Ошибка такова:

Скрытый текст: показать

InvalidOperationException: Collection was modified; enumeration operation may not execute.
System.Collections.Generic.List`1+Enumerator[System.Int32].VerifyState ()
System.Collections.Generic.List`1+Enumerator[System.Int32].MoveNext ()
TerrainCreater.CreateTerrain () (at Assets/_Script/TerrainCreater.cs:20)

Код главного действующего лица:

Синтаксис:

Синтаксис: [ Показать ]

Используется csharp

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

public class TerrainCreater : MonoBehaviour
{
    public  Manager mn;
    public  MeshFilter Filter;
    public  List<int> bases =new List<int>();

    [ContextMenu("Create Terrain")]
    void CreateTerrain()
    {
        Debug.Log("1");
        for (int i=0;i<mn.BaseAmount;i++)
        {
            Debug.Log("2");
            int temp= Random.Range(0, mn.TerrainLength);
            Debug.Log(temp);
//20 -снизу
           foreach (int bs in bases)
            {
                Debug.Log("3");
                int size = mn.granitsa;
                while(bs-size<temp&&bs+size>temp)
                {
                    Debug.Log("4");
                    temp = Random.Range(0, mn.TerrainLength);
                }
                bases.Add(temp);
                Debug.Log(temp+"Added-Down");
            }
        }
    }
}

Код склада переменных:

Синтаксис:

Синтаксис: [ Показать ]

Используется csharp

using UnityEngine;
using System.Collections;

public class Manager : MonoBehaviour
{
    public int TerrainLength=2000;
    public int BaseAmount=3;
    public int granitsa = 20;
}

Консоль:

Скрытый текст: показать

Сижу в ступоре, что не так

[waruiyume]

Создайте дополнительный список и добавляйте в него, а потом слейте их вместе.

[Nicloay]

проблема в том что ты итерируешься по base и в него же добавляешь значение, возможен бесконечный цикл, если будешь использовать for - то такого не будет.

[trololoid]

Синтаксис:

Синтаксис: [ Показать ]

Используется csharp

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

public class TerrainCreater : MonoBehaviour
{
    public Manager mn;
    public MeshFilter Filter;
    public List<int> bases = new List<int>();
    public List<int> tmp = new List<int>();

    [ContextMenu("Clear Lists")]
    void ClearLists()
    {
        bases.Clear();
        tmp.Clear();
    }

    [ContextMenu("Create Terrain")]
    void CreateTerrain()
    {
        for (int i = 0; i < mn.BaseAmount; i++)
        {
            int temp = Random.Range(0, mn.TerrainLength);
            Debug.Log(temp);
            if (bases.Count > 0)
            {
                foreach (int bs in bases)
                {
                    //Заполняем временный лист
                    tmp.Add(bs);
                }
                foreach (int bs in tmp)
                {
                    int size = mn.granitsa;
                    while (bs - size < temp && bs + size > temp)
                    {
                        //Если между базами не хватает места, то пересчёт
                        temp = Random.Range(0, mn.TerrainLength);
                    }
                    bases.Add(temp);
                    Debug.Log(temp + " Added-Down");
                }
                tmp.Clear();
            }
            else
            {
                //Первая база
                bases.Add(temp);
            }

        }
    }

}

Появился забавный эффект.
Если кол-во баз должно быть 2, то всё работает как надо.

Если 3, то:

Если 4, то:

Вообщем с трёх баз начинает возводить кол-во одинаковых чисел в квадрат.

Решил проверкой на наличие этого числа в главном листе, чувствую, что индус поступил бы также.

Вдруг оказалось, что с третей базы начинает добавляться лишняя база, решил проверкой на кол-во чисел. Теперь мне можно пить чай с индусами.
Финальная версия такова:

Синтаксис:

Синтаксис: [ Показать ]

Используется csharp

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

public class TerrainCreater : MonoBehaviour
{
    public Manager mn;
    public MeshFilter Filter;
    public List<int> bases = new List<int>();
    public List<int> tmp = new List<int>();

    [ContextMenu("Clear Lists")]
    void ClearLists()
    {
        bases.Clear();
        tmp.Clear();
    }

    [ContextMenu("Create Terrain")]
    void CreateTerrain()
    {
        bases.Clear();
        for (int i = 0; i < mn.BaseAmount; i++)
        {
            int temp = Random.Range(0, mn.TerrainLength);
            Debug.Log(temp);
            if (bases.Count > 0)
            {
                foreach (int bs in bases)
                {
                    //Заполняем временный лист
                    tmp.Add(bs);
                }
                foreach (int bs in tmp)
                {
                    int size = mn.granitsa;
                    while (bs - size < temp && bs + size > temp)
                    {
                        //Если между базами не хватает места, то пересчёт
                        temp = Random.Range(0, mn.TerrainLength);
                    }
                    if (!bases.Contains(temp)&&bases.Count<mn.BaseAmount)
                    {
                        bases.Add(temp);
                        Debug.Log(temp + " Added-Down");
                    }
                }
                tmp.Clear();
            }
            else
            {
                //Первая база
                bases.Add(temp);
            }

        }
    }

}