Question

我用C ++和SFML制作游戏。我有一个奇怪的错误，每当我在屏幕上只有1个敌人和3个镜头并且子弹与敌人发生碰撞时，游戏就会崩溃。我得到的错误是：

Expression: vector subscript out of range

这是我检查镜头和敌人之间碰撞的方法：

for (int i = 0; i < enemies.size(); i++)
{
    for (int s = 0; s < shots.size(); s++) {
        if (Collision::PixelPerfectTest(enemies[i].getSprite(), shots[s].getSprite())) {
            enemies[i].setHealth(enemies[i].getHealth() - player.getDamage());
            if (enemies[i].getHealth() <= 0) {
                enemies.erase(enemies.begin() + i);
            }
            shots.erase(shots.begin() + s);
        }
    }
}

＆＃34;敌人＆＃34;和＃34;镜头＆＃34;是我在主循环中每隔x秒插入的向量。这是代码：

    if (enemySpawner.getElapsedTime().asSeconds() >= 1.5f) {
        enemies.push_back(Enemy(spriteManager.enemySprite));
        std::cout << enemies.size() << " enemies" << std::endl;
        enemySpawner.restart();
    }

    if (shotSpawner.getElapsedTime().asSeconds() >= 0.3f &&      sf::Mouse::isButtonPressed(sf::Mouse::Left)) {
        shots.push_back(Shot(spriteManager.shotSprite, player.getPosition(), *window));
        std::cout << shots.size() << " shots" << std::endl;
        shotSpawner.restart();
    }

正如我所说，这只发生在我有1个敌人和3个射击时，否则它似乎工作正常。

编辑：现在当第一个敌人产生时，游戏会冻结。这是更新后的代码：

//Shot vs enemy
for (auto eit = enemies.begin(); eit != enemies.end();)
{
    for (auto sit = shots.begin(); sit != shots.end();) {
        if (Collision::PixelPerfectTest((*eit).getSprite(), (*sit).getSprite())) {
            (*eit).setHealth((*eit).getHealth() - player.getDamage());
            if ((*eit).getHealth() <= 0) {
                enemies.erase(eit);
                shots.erase(sit);
            }
            shots.erase(sit);
            eit = eit++;
            sit = sit++;
        }
    }
}

Answer 1

你不能只是按照你正在做的方式迭代它们的东西，因为它会产生不再在向量范围内的i值（在它的大小因擦除调用而减小之后）），这将基本上导致尝试索引不再存在的数组值。如果您需要这种行为，最好使用迭代器：

auto eit = enemies.begin();
for (; eit != enemies.end(); )
{
    if((*eit).isDead())
    {
         eit = enemies.erase(eit);
    }
    else
    {
         eit++;
    }
}

你会对镜头阵列做同样的事情。

Answer 2

@ moka的回答解释了代码的核心问题，他/她使用迭代器的代码将解决您的问题。但是，还有另一种方法可以解决这个问题：将代码拆分为＆＃34;计算通道＆＃34;和＃34;数组修改传递。＆＃34;代码看起来像：

for (int i = 0; i < enemies.size(); i++)
{
    for (int s = 0; s < shots.size(); s++) {
        if (!enemies[i].alive || !shots[s].active) {
            continue;
        }
        if (Collision::PixelPerfectTest(enemies[i].sprite(), shots[s].sprite())) {
            shots[s].active = false;
            enemies[i].setHealth(enemies[i].getHealth() - player.getDamage());
            if (enemies[i].getHealth() <= 0) {
                enemies[i].alive = false;
            }
        }
    }
}

enemies.erase(std::remove_if(enemies.begin(), enemies.end(),
    [](Enemy const &e) { return !e.alive; }), enemies.end());

shots.erase(std::remove_if(shots.begin(), shots.end(),
    [](Shot const &s) { return !s.active; }), shots.end());

这种风格也很好，因为它将游戏逻辑与实现分开了一点。例如，如果你想允许射击对一个以上的敌人造成伤害，那么无论如何你都必须对射击阵列使用这样的方法。

优秀游戏编程模式书中的

Chapter on Double Buffering更深入。

SFML奇怪的射击bug

2 个答案: