2

以下代码检查球是否与砖重叠。如果发生重叠,砖块会突出显示并从列表中删除。

仅擦除矢量brickWall中的最后一项时,我遇到了核心转储。评论erase()行,代码似乎运行良好。
在论坛上查看类似问题后,我相信我正在迭代并正确擦除矢量中的项目,但我怀疑情况可能并非如此。

对此代码的任何建议将不胜感激!

void Game::updateEntities()
{
    ballMother.update();
    for (std::vector<gdf::Entity>::iterator it = brickWall.begin(); it != brickWall.end(); ++it) 
    {
        if (it->getRect().intersects(ballMother.getRect())) {
            it->rect.setFillColor(sf::Color::Red);
            it = brickWall.erase(it);
        }
        else {
            it->rect.setFillColor(it->colour);                
        }
    }
}
4

2 回答 2

11

通常的成语是

for (std::vector<gdf::Entity>::iterator it = brickWall.begin(); it != brickWall.end(); /*note nothing here*/) 
{
    if (it->getRect().intersects(ballMother.getRect())) {
        it->rect.setFillColor(sf::Color::Red);
        it = brickWall.erase(it);
    }
    else {
        it->rect.setFillColor(it->colour);                
        ++it;
    }
}

否则,您将跳过元素,并最终通过结束而崩溃。

于 2012-06-09T00:50:08.020 回答
4

K-ballo 发现了错误,并提出了一个很好的习语,但您可能会发现使用标准算法和擦除删除习语更容易。

struct Pred {
    Pred(const BallMother& ballMother) { 
        //save refs to whatever you need for the compare 
    }

    bool operator()(const gdf::Entity& item) {
        //collision logic here, return true if you want it erased
    }
};

现在在您的功能中:

std::vector<gdf::Entity>::iterator end_iter =
    std::remove_if(brickWall.begin(), brickWall.end(), Pred(ballMother));

brickWall.erase(end_iter, brickWall.end());

或者更简洁地说:

brickWall.erase(std::remove_if(brickWall.begin(), brickWall.end(), Pred(ballMother)),
                brickWall.end());
于 2012-06-09T01:00:08.680 回答