У меня есть вектор умных указателей для хранения самолетов героя.
Я хочу удалить их, как только они не будут видны на экране, или они достигают цели. Двойное удаление или вызов функции на nullptr < /code>.
Есть предложения?void HeroShoot::CleanupHeroShoots() {
if (!burst_.empty()) {
for (auto it = burst_.begin(); it != burst_.end(); ++it) {
if (*it == nullptr ||
(*it)->IsHit() ||
!((*it)->IsVisible((*it)->GetProtonBounds()))) {
it = burst_.erase(it);
}
}
}
}
< /code>
и < /p>
bool Collisions::IsTargetHit(Target& target) {
bool scored{};
std::vector* p_bullets = HeroShoot::GetBurst();
for (const auto& bullet : *p_bullets) {
scored = CheckCollisionRecs(bullet->GetProtonBounds(),target.GetTargetBounds());
if (scored) {
bullet->SetHit(scored);
target.SetHit(scored);
// receive damage from bullet and reduce it from target
}
}
p_bullets = nullptr;
return scored;
}
< /code>
Редактировать: < /p>
Сначала, Burts_ - вектор:
static std::vector burst_;
Второе,
Я изменил чистые чистки на то, что ниже, и теперь он работает. Это намного больше кода, который необходимо улучшить, но теперь он работает. Как справедливо указано, я стирал во время итерации вектора. < /P>
void HeroShoot::CleanupHeroShoots() {
if (!burst_.empty()) {
burst_.erase(std::remove_if(burst_.begin(),
burst_.end(),
[](auto& s_ptr) { return s_ptr->IsHit(); }), burst_.end());
}
}
Подробнее здесь: https://stackoverflow.com/questions/797 ... rom-vector
