从向量数组中删除敌人

Question

这是我在这个网站上的第一个问题。我学了一年编程，总能在项目中找到问题的答案。我知道这是一个非常简单的问题，这里有一个代码：

#include "stdafx.h"
#include "Wave.h"

Wave::~Wave()
{
}

void Wave::setEnemyCount(int count) {
    nemyCount = count;
}

bool Wave::createWave() {
    for (int i = 0; i < enemyCount; i++) {
        enemyArray.push_back(Enemy());
    }

    for (int i = 0; i < enemyArray.size(); i++) {
        int randomPositionX = rand() % 150 + 10;
        int randomPositionY = rand() % 50 + 10;

        if (!enemyArray.at(i).init(randomPositionX, randomPositionY)) {
            std::cout << "Create wave(init enemies) - failed\n";
            return false;
        }
    }

    return true;
}

void Wave::drawWave(Window* window) {
    window->setCursor({ 0, 2 });
    window->setColor(Window::Color::RED);
    std::cout << "Enemies: " << enemyArray.size();
    window->setColor(Window::Color::WHITE);

    for (int i = 0; i < enemyArray.size(); i++) {
        enemyArray.at(i).draw(window);
        //std::cout << i;
    }
}

void Wave::update(Bullet* bullet, Player* player) {

    for (int i = 0; i < enemyArray.size(); i++) {
        if (player->checkShoot()) {
            if ((bullet->getPositionX() >= enemyArray.at(i).getPositionX() && bullet->getPositionX() <= enemyArray.at(i).getPositionX() + 5)) {
                if ((bullet->getPositionY() >= enemyArray.at(i).getPositionY() && bullet->getPositionY() <= enemyArray.at(i).getPositionY() + 5)) {
                    player->addScore(10);
                    enemyArray.erase(enemyArray.begin() + i);
                }
            }
        }
    }

    if (enemyArray.size() <= 0) {
        std::cout << "WAVE COMPLETE";
        Sleep(2000);
        enemyCount += 5;
        createWave();
    }
}

int Wave::getEnemyCount() const {
    return enemyCount;
}

碰撞工作正确，但是当子弹碰到索引的敌人时，例如：5，代码总是删除最后一个索引的敌人，所以我可以删除每个敌人只射击索引为 5 或 1 或 7 的敌人。

Answer 1

当您执行enemyArray.erase(...) 时，您不应增加i，因为您将跳过一个元素。

所以你应该像这样重写循环：

for (int i = 0; i < enemyArray.size();) {
    if (player->checkShoot()) {
        if ((bullet->getPositionX() >= enemyArray.at(i).getPositionX() && bullet->getPositionX() <= enemyArray.at(i).getPositionX() + 5)) {
            if ((bullet->getPositionY() >= enemyArray.at(i).getPositionY() && bullet->getPositionY() <= enemyArray.at(i).getPositionY() + 5)) {
                player->addScore(10);
                enemyArray.erase(enemyArray.begin() + i);
                continue;
            }
        }
    }
    ++i;
}

或者，使用更惯用的风格：

for (auto it = enemyArray.begin(), end = enemyArray.end();it!=end;) {
    if (player->checkShoot()) {
        if ((bullet->getPositionX() >= it->getPositionX() && bullet->getPositionX() <= it->getPositionX() + 5)) {
            if ((bullet->getPositionY() >= it->.getPositionY() && bullet->getPositionY() <= it->getPositionY() + 5)) {
                player->addScore(10);
                enemyArray.erase(it);
                continue;
            }
        }
    }
    ++it;
}

当然，如果你想一次移除一个敌人，你应该在删除元素后在循环中break 而不是continue。

为了完整起见，如果你想移除所有被子弹触及的敌人，你也可以使用std::remove_if，它比上面的2个循环更优化，因为当移除多个元素时它会减少元素的移动。