vector型変数のメンバ関数.size() == 0 という比較は危険なのかどうか？

Question

提示コードですがgetEnemyNum();関数を使ってステージにエネミーが何体居るかどうかを検出して０だったら新しいステージに移行するようにbool newStage 変数をtrueにしています。このif文ですがゲーム内の壁に当たるとどういうわけかnewStage変数がtrueになってしまいますつまり
if文が成立してしまっているのですがこのif文は危険なのでしょうか？

//現在のエネミーの数を得る
int Enemy_Mng::getEnemyNum()
{
//	std::shared_ptr<std::vector<Enemy>> getEnemy();					//エネミー情報

	return enemy->size();
}

//更新
void Enemy_Mng::Update()
{

	printf("Size: %d\n",getEnemyNum());


	if (getEnemyNum() == 0)
	{
		printf("新しいステージ\n");
		newStage = true;
	}

	for (std::vector<Enemy>::iterator itr = enemy->begin(); itr != enemy->end(); itr++)
	{
		itr->Update();
	}
}

Answer 1

std::vector<Enemy>で敵のインスタンスを管理されていますが、vectorの途中の敵が死んだ場合にその敵をvectorから取り除く方法に問題があるのかも知れません(vectorのsizeで敵の数を管理しているので、要素の再利用はしていないのだと推測しています)。例えば、vectorを回しながら要素を削除したりすると、未定義の動作になるので注意が必要です。

Comments

[退会済みユーザー]

iterator を回しているところに問題がありました。

[episteme]

enemy数が変化する個所を"すべて"確認したんじゃなかったんかい...

[退会済みユーザー]

調べたのですがまさかiteratorがおかしな場所をさしてるとは想像もしませんでした。そいう場合は例外が発生するものだと思ってたんだで...

Answer 2

このif文ですがゲーム内の壁に当たるとどういうわけかnewStage変数がtrueになってしまいます

ゲーム内の壁に当たるときenemyがクリアされて空になるのなら、その挙動となるでしょう。
心当たりはありませんか?

Comments

[退会済みユーザー]

それは既に把握済みです。当たり判定を無効にして実験しましたがやはり書き換えられてしまいます。
質問の通り問題はないのでしょうか？

[episteme]

if (getEnemyNum() == 0) これ自体に何の問題もありません。

[episteme]

> 当たり判定を無効にして実験しましたがやはり書き換えられてしまいます。

enemy数が変化する個所を"すべて"確認しましたか?

[退会済みユーザー]

確認しました。.....なんでや...