イテレーターが破壊される原因と対処法が知りたい。

Question

提示コードですが。コメント部のコードでアクセスエラーになります。どうすればいいのでしょうか？stage->getMapObject();はマップのベクターなのですがその要素を指定した要素を削除するコードなのですが削除する例外が出てしまいます。どうすればいいのでしょか？

void Scene::Update()
{
	stage->Update();
	player->Update();


	// プレイヤーとブロックの当たり判定 
	for (std::vector<MapObject>::iterator itr = stage->getMapObject().begin(); itr != stage->getMapObject().end(); itr++)
	{
		//printf("%d\n",itr->mCol.getSize().x);
		if (Intersect_Rect_2D(itr->mCol,player->mCol) == true)
		{
			if (itr->getObjectType() == ObjectType::Item)
			{
				printf("あああ\n");
				itr = stage->getMapObject().erase(itr);//////////
			}

			player->Fix(itr->mCol.getPosition(), itr->getObjectType());

			
		}
	}

}

#ifndef ___STAGE_HPP_
#define ___STAGE_HPP_

#include "MapObject.hpp"
#include "Object.hpp"
#include "../Entry.hpp"


#include <iostream>

// 画面のセルの数
#define SCREEN_WIDTH_CELL ((int)((int)SCREEN_WIDTH / CELL))
#define SCREEN_HEIGHT_CELL ((int)((int)SCREEN_HEIGHT / CELL))


class Stage : Actor_2D
{
public:
	Stage(Entry* e);
	
	~Stage();

	void Update() override;
	void Draw()override;

	std::vector<MapObject> getMapObject();	//マップオブジェクトを取得

private:


	
	std::vector<MapObject> Area;



};

#endif;

#include "../../Header/Game/Stage.hpp"


//コンストラクタ
Stage::Stage(Entry* e) : Actor_2D()
{

	std::shared_ptr<TextureData> sprite = LoadTexture("Assets/Block.png");
	std::shared_ptr<TextureData> Item_sprite = LoadTexture("Assets/Item.png");
	


//	printf("X: %f\n", SCREEN_WIDTH / (float)CELL);
//	printf("Y: %f\n",SCREEN_HEIGHT / (float)CELL);

	
	// ステージ
	int Map[SCREEN_HEIGHT_CELL][SCREEN_WIDTH_CELL] =
	{
		{0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0},
		{0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0},
		{0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0},
		{0,0,0,0,0,0,3,0,0,0,0,1,1,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0},
		{0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0},
		{0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,3,0,0,0,0,3,0,0,0,0,0,0,0,0,0,0,0,0},
		{0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0},
		{0,0,0,0,2,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0},
		{0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,2,0,0,0,0,2,0,0,0,0,0,0,0,0},
		{0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0},
		{0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0},
		{0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0},
		{0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,2,0,0,2,0,0,0,0,0,0,0,0},
		{0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0},
		{0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0},
		{0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0},
		{0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0},
		{0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0},
	};

	

	for (int y = 0; y < SCREEN_HEIGHT_CELL; y++)
	{
		for (int x = 0; x < SCREEN_WIDTH_CELL; x++)
		{

			// ブロック
			if (Map[y][x] == 1)
			{			
				Box_Collision_2D col;
				col.setEnable(true);
				col.setPosition(glm::ivec2(CELL * x,( -CELL * y)));
				col.setSize(glm::ivec2(CELL, CELL));
				TextureUV uv;
				uv.start = glm::vec2(0, 0);
				uv.end = glm::vec2(CELL, CELL);

				Area.push_back(MapObject(e, ObjectType::Brick, sprite, uv, glm::ivec2(CELL * x, (-CELL * y)), col));

			//レンガ
			}else if (Map[y][x] == 2)
			{
				Box_Collision_2D col;
				col.setEnable(true);
				col.setPosition(glm::ivec2(CELL * x, (-CELL * y)));
				col.setSize(glm::ivec2(CELL, CELL));
				TextureUV uv;
				uv.start = glm::vec2(CELL, CELL);
				uv.end = glm::vec2(CELL * 2, CELL * 2);

				Area.push_back(MapObject(e, ObjectType::Block, sprite, uv, glm::ivec2(CELL * x, (-CELL * y)), col));
			}

			// アイテム
			else if (Map[y][x] == 3)
			{

				Box_Collision_2D col;
				col.setEnable(true);
				col.setPosition(glm::ivec2(CELL * x, (-CELL * y)));
				col.setSize(glm::ivec2(CELL, CELL));
				TextureUV uv;
				uv.start = glm::vec2(0, 0);
				uv.end = glm::vec2(CELL , CELL);

				Area.push_back(MapObject(e, ObjectType::Item, Item_sprite, uv, glm::ivec2(CELL * x, (-CELL * y)), col));
			}
		}
	}
}


std::vector<MapObject> Stage::getMapObject()
{
	return Area;
}


void Stage::Update()
{	
	for (std::vector<MapObject>::iterator itr = Area.begin(); itr != Area.end(); itr++)
	{
		itr->Update();
	}	
}

void Stage::Draw()
{	
	for (std::vector<MapObject>::iterator itr = Area.begin(); itr != Area.end(); itr++)
	{
		itr->Draw();
	}
}

//デストラクタ
Stage::~Stage()
{

}

Comments

[int32_t]

stage->getMapObject() の実装はどうなってますか?

[退会済みユーザー]

ベクター型です。

[int32_t]

それは現状の質問文を見ればわかります。コピーを返してますか、参照を返してますか。

[退会済みユーザー]

実態です。

[int32_t]

コピーなのですね? というか、素直にコードを開示してください。

[退会済みユーザー]

開示しました。

Answer 1

以前の質問とまったく同じ問題があります。

で、今回は Stage::getMapObject()::erase() を呼んでいますが、Stage::getMapObject() が返すのは Stage::Area のコピーなので、erase() しても何も意味がなさそうです。
getMapObject() は消してしまって、stage->CheckCollision(player) みたいなメソッドを足すのが簡単だと思います(vectorがコピーかどうかを気にする必要がなくなるので)。

Comments

[退会済みユーザー]

参照型にすれば削除可能でしょうか？

[int32_t]

参照で可能ですけども、とてもトラブルを起こしやすい設計で、避けるべきです。
あるクラスのデータメンバは、そのクラスのメソッドのみが変更するのが理想的です。

[fana]

リンク張られてる前の質問に書いたけども，
erase()で要素削除する以前に，一時オブジェクトから取得したitrがもうダメなんじゃないかと．

[退会済みユーザー]

ベクターを引数に取る関数を使って消したい要素を消すというプログラムで解決しました。

[退会済みユーザー]

ベクターを引数に取るやり方はベクトルが例外エラーを出すので違いました。
どうすればstageクラスのベクトルで要素を消せるのでしょうか？

[int32_t]

どういうコードを書いて失敗しているのかわかりませんが、既にこの回答に書いたとおり、Stage にメソッドを足してその中でStage::Areaに対してループ&erase()すれば問題ないはずです。