Почему юнити жалуется на код?

Question

[Юра Майллер]

Я пытаюсь перебрать список, но почему-то юнити выдает ошибку:



Вот место, где происходит ошибка:

foreach (int item in sv.ItsNum) //жалуется здесь 
        {
            Debug.Log(item);
            if(Num == item)
            {
                StartCoroutine(Skin());
            }
            else
            {
                sv.ItsNum.Add(Num);
                
            }
                
        }

думаю это из за foreach но не факт

Comments

[Василий Банников]

Что вообще этот код делает? Может можно тут обойтись без цикла и добавления?

Answer 1

[freeExec]

Нет, ему плохо от этого
sv.ItsNum.Add(Num);
Нельзя изменять коллекцию, которую вы перебираете. Если бы вы тупо перевели сообщение об ошибке, не пришлось бы вообще заводить топик.

PS
Да тут сам код с багой, чем длиннее массив, тем больше Нумов в него добавится. А так бы его можно было переписать на две строки с Linq.

Comments

[GavriKos]

Ну вообще ТС тоже частично прав - это из за энумератора. Обращаясь по индексу - можно модифицировать коллекцию, не забывая модифицировать индекс

Answer 2

[Griboks]

Думаю это из-за того, что "Collection was modified". Иными словами, вы перебираете и изменяете коллекцию одновременно. А шарп не хочет разбираться в ваших махинациях с коллекцией, поэтому отказывается работать и плюётся в вас ошибкой.

Для особо ленивых, есть секретный метод, который заставит шарп работать в этой каше. Надо всего лишь... подсунуть ему копию коллекции:

foreach (int item in sv.ItsNum.ToList()) //тупо копируем
        {
            Debug.Log(item);
            if(Num == item)
            {
                StartCoroutine(Skin());
            }
            else
            {
                sv.ItsNum.Add(Num);
                
            }
                
        }

Также следует обратить внимание, что тупо (и даже умно) копировать коллекцию не очень хорошо во всех смыслах.

Comments

[GavriKos]

А еще можно обращаться по индексу )) И не забивать память

[Griboks]

GavriKos,

if(sv.ItsNum.Contains(Num)) 
{
    StartCoroutine(Skin());
    sv.ItsNum.AddRange(sv.ItsNum.Where(item=>Num != item));
}
else sv.ItsNum.AddRange(sv.ItsNum);

[GavriKos]

Griboks, мне то это зачем ))) Но в контексте задачи это решение, да

Answer 3

[Василий Банников]

думаю это из за foreach но не факт

Вы правы.
Нельзя модифицировать лист, по которому вы сейчас итерируетесь.