警告 [C26815 このポインターは、破棄された一時インスタンスを指しているため、ダングリング状態です。] の解決方法が知りたい。

Question

提示コードですが下記の警告とデバッグエラーが発生するのですが警告コード対処法がわかりません。なぜでしょうか？
(*itr)->getWeapon().begin();　部にて下記の警告文

警告[ 重大度レベル コード 説明 プロジェクト ファイル 行 抑制状態
警告 C26815 このポインターは、破棄された一時インスタンスを指しているため、ダングリング状態です。 Dungeon_Game C:\Users\KenMatsuzawa\Desktop\Dungeon_Game\Dungeon_Game\Game.cpp 260
]

//	std::vector<std::shared_ptr<Weapon>> getWeapon(); //攻撃を取得
//攻撃を取得
std::vector<std::shared_ptr<Weapon>> Enemy::getWeapon()
{
	return attack;
}

if ( (windowContext->getFrame() % 45) == 0)
	{
		int t = FrameWork::GetRandom(0,4);
		if (t == 4)
		{
			attack.push_back(std::make_shared<Magic>(windowContext, position, moveVec));
		}
		else {
			moveVec = t;
		}

	}

	
	//プレイヤーダメージ判定　エネミー
	for (std::vector<std::shared_ptr<Enemy>>::iterator itr = enemy.begin(); itr != enemy.end(); itr++)
	{
		for (std::vector<std::shared_ptr<Weapon>>::iterator it = (*itr)->getWeapon().begin(); it != (*itr)->getWeapon().end();)
		{
			if (player->EnemyAttackCollision((*it)->mCollision) == true)
			{
				it = (*itr)->getWeapon().erase(it);
			}
			else {
				it++;
			}
		}
	}

Comments

[asm]

> (*itr)->getWeapon()
ここで呼び出されているgetWeapon関数についての詳細をおねがいします。

[退会済みユーザー]

追加しました。

Answer 1

std::vector<std::shared_ptr<Weapon>> Enemy::getWeapon()

これではattackのコピーを返しています。

std::vector<std::shared_ptr<Weapon>>& Enemy::getWeapon()
{
    return attack;
}

とする必要があります。

Answer 2

以下の2つの質問が同様の問題なので、参考になると思います。

• std:;vector<> std::arrayという二重のイテレーター二重for文がアクセスエラーとなる原因が知りたい。
• イテレーターが破壊される原因と対処法が知りたい。

「インタンスのコピーを避ける」「あるクラスのデータメンバに対する操作は、そのクラスの関数メンバにする」というのを気を付けるとトラブルを避けられます。