C# - >索引超出范围错误|使用列表

时间:2014-11-03 20:30:45

标签: c# list count xna

我正在使用XNA进行坦克游戏。我已经实现了一个使用列表拍摄子弹的功能。拍摄后,我想测试子弹是否接近屏幕边界。如果是这样,请从列表中删除该特定项目符号。

只有在任何给定时间屏幕上有多个项目符号时才会出现错误。这是代码:


坦克等级:

List<Bullet> bulletList = new List<Bullet>();
bool spacebarPrepared = true;    //for shooting every 0.5 seconds
short count = 0;


//Shoot
if (Keyboard.GetState().IsKeyDown(Keys.Space) && spacebarPrepared == true)
   {
        bulletList.Add(new Bullet(sprBullet, position, turretDirection, turretAngle));
        spacebarPrepared = false;
   }


if (spacebarPrepared == false)
    {
        spacebarCount += (float)gameTime.ElapsedGameTime.TotalSeconds;

        if (spacebarCount > 0.5)
        {
            spacebarPrepared = true;
            spacebarCount = 0;
        }
    }


//Update bullets
foreach (Bullet bullet in bulletList)
{
    bullet.Update(bounds);
}


count = (short)bulletList.Count;

//Remove unwanted bullets
for (short i = 0; i < count; i++)
{
    if (bulletList[i].Alive == false)
    {
        bulletList.Remove(bulletList[i]);
    }
 }

Bullet Class:

class Bullet
{
    Texture2D spr;
    Vector2 origin, pos, dir, turretLength;
    float rotation, scale, turretLeng;
    short speed;
    bool alive = true;


    public Bullet(Texture2D sprite, Vector2 position, Vector2 direction, float angle)
    {
        spr = sprite;
        scale = 0.15f;
        turretLeng = (110 + spr.Width) * scale;
        speed = 5;
        rotation = angle;
        pos = position;
        dir = direction;
        origin = new Vector2(spr.Width / 2, spr.Height / 2);
        FindTurretLength();
    }

    public void Draw(SpriteBatch spriteBatch)
    {
        Matrix bulletTranslation = Matrix.CreateRotationZ(rotation) * Matrix.CreateTranslation(pos.X + turretLength.X, pos.Y + turretLength.Y, 0);

        spriteBatch.Begin(SpriteSortMode.BackToFront, null, null, null, null, null, bulletTranslation);
        spriteBatch.Draw(spr, Vector2.Zero, null, Color.White, 0, origin, 0.15f, SpriteEffects.None, 1f);
        spriteBatch.End();
    }

    public void Update(Vector2 boundary)
    {
        pos += dir * speed;


        if (pos.X < 50 || pos.X > boundary.X - 50 || pos.Y < 50 || pos.Y > boundary.Y - 50)
        {
            alive = false;
        }
    }

    public void FindTurretLength()
    {
        turretLength = new Vector2(turretLeng * dir.X, turretLeng * dir.Y);
    }

    public Vector2 Pos
    {
        get
        {
            return pos;
        }
        set
        {
            pos = value;
        }
    }

    public bool Alive
    {
        get
        {
            return alive;
        }
        set
        {
            alive = value;
        }
    }
}

我在调试时注意到的是我自己的'count'变量= 2,但bulletList.Count = 1.这可能是问题吗?这是怎么发生的?

非常感谢任何帮助。

1 个答案:

答案 0 :(得分:1)

问题在于您的for循环中删除了项目符号。

让我们假设您在启动循环时有10个项目符号列表(索引0 - 9)。第1个子弹(在索引0处)被删除....现在你的列表有9个子弹(索引0 - 8),但是计数变量尚未更新,所以你的for循环仍然认为它有10个。

当你达到“i”大于实际的子弹量时,你会得到“指数超出范围”。错误。

有多种方法可以解决此错误。

我会选择:

bulletList.RemoveAll(x => !x.Alive);