在未修改项目的地方修改了集合

Question

我正在使用 C# 的 XNA 库开发一个简单的游戏。在下面的代码 sn-p 我得到了

集合已修改；枚举操作可能无法执行。

第二个 foreach 循环顶部的错误。在我（相对有限的）C# 经验中，尝试在循环期间修改基础集合时会发生这种情况。但是，据我所知，我并没有以任何方式修改enemy_positions 集合。此代码中的所有集合都是List<Vector2> 类型。

这里发生了什么？

//defines collision behaviour when enemy is hit
int be_no = 0;
List<Vector2> tmp_bullets = bullet_i_position;
List<Vector2> tmp_enemy = enemy_positions;

foreach (Vector2 bullet in bullet_i_position)
{
    //get bullet collision box
    Rectangle bullet_col = new Rectangle(Convert.ToInt32(bullet.X - 12), Convert.ToInt32(bullet.Y - 12), 25, 26);

    int en_no = 0;

    foreach (Vector2 enemy in enemy_positions)
    {
        //get enemy collsion box
        en_box = new Rectangle(Convert.ToInt32(enemy.X), Convert.ToInt32(enemy.Y), 75, 75);

        if (temp_r.Intersects(en_box))
        {
            //remove all colliding elements
            tmp_enemy.RemoveAt(en_no);
            tmp_bullets.RemoveAt(be_no);
            bullet_direction.RemoveAt(be_no);

        }
        en_no++;
    }
    be_no++;
}

//update actual lists
bullet_i_position = tmp_bullets;
enemy_positions = tmp_enemy;

Answer 1

线条：

List<Vector2> tmp_bullets = bullet_i_position;
List<Vector2> tmp_enemy = enemy_positions;

没有克隆列表，它们只是创建对同一列表的本地引用。直接的解决方案是将这两行更改为：

List<Vector2> tmp_bullets = new List<Vector2>(bullet_i_position);
List<Vector2> tmp_enemy = new List<Vector2>(enemy_positions);

但这将在每次调用该方法时分配一个新列表，这对于垃圾收集（尤其是在游戏中）来说是很糟糕的，因此更好的解决方案是删除您的 foreach 循环并用 reverse for 替换它们循环。这是有效的，因为只有在使用枚举器迭代集合时才会出现该异常。同样的问题不适用于常规 for 循环。例如：

for (int i = bullet_i_position.Count - 1; i >= 0; i--)
{
    Vector2 bullet = bullet_i_position[i];

    // ...
}

reverse 迭代的另一个原因是，在定期迭代时删除元素意味着您将在删除元素之后跳过该元素（因为索引向下移动 1）

Answer 2

List<Vector2> tmp_enemy = enemy_positions;

不复制enemy_positions 并将其分配给tmp_enemy。相反，tmp_enemy 指向enemy_positions。因此，tmp_enemy 的任何变化都会反映在enemy_positions 中。 如果你想制作一个实际的副本，这是一个更好的方法：

List<Vector2> tmp_enemy = new List<Vector2>(enemy_positions);

Answer 3

正如其他人提到的，您遇到了引用类型和值类型之间的区别

我建议您将 foreach 循环替换为类似于以下内容的内容

for(int i = 0; i<bullet_i_position.count;i++)
  {
     bool hascollided = false;
     for(int j = 0; j<enemy_positions.count;j++)
        {
          if(collisionOccurs)
               {
                hasCollided = true;
                enemy_positions.RemoveAt(j);
                j--;
               }
       }
   if(hasCollided)
      {
         bullet_i_position.RemoveAt(i);
         i--;
      }
}

你需要考虑如果子弹在上面的例子中同时撞到两个敌人会发生什么