”アクセス違反”のせいで製作が進まない・・・！

Question

前提・実現したいこと

Dxlibにて2Dアクションを作っています。
二次元配列に沿ってマップを描画するようなコードを書いていて、1ステージ目の走破は出来るのですが、2ステージ目に進む前にゲームが強制終了してしまいます。
エラーメッセージには
”ハンドルされない例外が 0x00007FF6E90BD6E5 (2DActionGame.exe) で発生しました: 0xC0000005: 場所 0x0000000000000105 の読み取り中にアクセス違反が発生しました。”
と書いており、ローカルや呼び出し履歴を見ても明らかにおかしいのはわかるのですが、対処の仕方がわかりません。
他の方の質問を色々調べてみたのですが、当方のプログラミングスキルはアクセス違反の意味を理解することがやっとのレベルです。
どなたか教えていただけると助かります。

発生している問題・エラーメッセージ

ハンドルされない例外が 0x00007FF6E90BD6E5 (2DActionGame.exe) で発生しました: 0xC0000005: 場所 0x0000000000000105 の読み取り中にアクセス違反が発生しました。

該当のソースコード

#include "Stage.h"
#include "DxLib.h"

//サンプルマップデータ
char g_MapDataA[STAGE_H][STAGE_W] = {

	0, 0, 0, 0, 0,  0, 0, 0, 0, 0,  0, 0, 0, 0, 0,  0, 0, 0, 0, 0,
	1, 0, 0, 0, 0,  0, 0, 0, 0, 0,  0, 0, 0, 0, 0,  0, 0, 0, 0, 5,
	1, 0, 0, 3, 0,  0, 0, 0, 0, 0,  0, 0, 0, 0, 0,  0, 0, 0, 0, 1,
	1, 0, 0, 3, 0,  0, 0, 0, 0, 0,  0, 0, 0, 0, 0,  0, 0, 3, 0, 1,
	1, 0, 0, 3, 3,  3, 3, 0, 0, 0,  0, 0, 0, 0, 0,  0, 0, 3, 0, 1,

	1, 0, 0, 0, 0,  0, 0, 0, 3, 3,  0, 0, 0, 0, 0,  0, 0, 3, 0, 1,
	1, 0, 0, 0, 0,  0, 0, 0, 0, 0,  0, 0, 3, 3, 0,  0, 0, 3, 0, 1,
	1, 0, 0, 0, 0,  0, 0, 0, 0, 0,  0, 0, 0, 0, 0,  0, 0, 3, 0, 1,
	1, 0, 0, 0, 0,  0, 0, 0, 0, 0,  0, 0, 0, 0, 0,  0, 0, 0, 0, 1,
	1, 0, 0, 0, 0,  0, 0, 3, 3, 0,  0, 0, 0, 0, 0,  3, 0, 0, 0, 1,

	1, 0, 0, 0, 0,  3, 3, 3, 3, 3,  0, 0, 0, 0, 3,  3, 0, 0, 0, 1,
	1, 0, 0, 0, 0,  3, 3, 3, 3, 3,  0, 0, 0, 3, 3,  3, 0, 0, 0, 1,
	1, 0, 0, 0, 0,  0, 0, 0, 0, 0,  0, 0, 0, 3, 3,  3, 0, 0, 0, 1,
	1, 4, 0, 0, 0,  0, 0, 0, 0, 0,  0, 0, 0, 0, 0,  0, 0, 0, 0, 1,
	2, 2, 2, 2, 2,  2, 2, 2, 2, 2,  2, 2, 2, 2, 2,  2, 2, 2, 2, 2,
};

//クラスに宣言した関数の実装法
//戻り値　クラス名::関数名（引数）
Stage::Stage()			//コンストラクタのため戻り値なし
	:m_PlayerStartPos()
{
	for (int y = 0; y < STAGE_H; y++)
	{
		for (int x = 0; x < STAGE_W; x++)
		{
			switch (g_MapDataA[y][x])
			{
			case ChipType_StartPos:
				m_PlayerStartPos.x = (CHIP_SIZE * x) + (HALF_CHIP);
				m_PlayerStartPos.y = (CHIP_SIZE * y) + (HALF_CHIP);
				m_Data[y][x] = ChipType_Non;
				break;
			default:
				m_Data[y][x] = g_MapDataA[y][x];
				break;
			}
		}
	}
}

Stage::~Stage()
{
}

void Stage::Draw()
{
	for (int y = 0; y < STAGE_H; y++)
	{
		for (int x = 0; x < STAGE_W; x++)
		{
			//処理しない条件で判定
			if (m_Data[y][x] == ChipType_Non)
			{
				continue;
			}
			/*
			ここを通過した時点で、何かしら描画する必要がある
			→DrawBoxを共通化できる
			*/
			unsigned int color = 0;			//色はマップチップごとに違う
			
			//マップチップごとに分岐処理
			switch (m_Data[y][x])
			{
			case ChipType_Wall:				color = GetColor(125, 125, 125);		break;
			case ChipType_Ground:			color = GetColor(0, 200, 0);		break;
			case ChipType_Footing:			color = GetColor(128, 0, 0);		break;
			case ChipType_GoalPos:		color = GetColor(255, 0, 0);		break;
			}

			DrawBox(
				x * CHIP_SIZE, y * CHIP_SIZE,
				x * CHIP_SIZE + CHIP_SIZE, y * CHIP_SIZE + CHIP_SIZE,
				color,
				TRUE
			); 
		}
	}
}

//プレイヤーの初期座標の取得
//constメンバ関数…関数名にconstが後置されている関数
//関数内でメンバ変数の変更を行っていないことを保証する
const Position& Stage::GetPlayerStartPos() const
{
	return m_PlayerStartPos;
}

//マップとの当たり判定
//もし当たっていた場合、めり込まないように移動量 moveX_, moveY_ を補正する
//当たっている辺の情報をbitフラグにして返す
int Stage::CheckCollision(const Position& pos_, float* moveX_, float* moveY_) const
{
	//当たった辺の種類
	int hitFlg = 0;

	//現在座標に移動量を加算して、移動後の座標を出す
	float afterX = pos_.x;
	float afterY = pos_.y;
	afterX += (moveX_ == nullptr ? 0.0f : *moveX_);
	afterY += (moveY_ == nullptr ? 0.0f : *moveY_);
	
	//移動後の座標にどのようなマップチップがあるかをチェック
	//もし何かあれば、必要な処理を行う
	if (GetChipType(afterX, afterY) != ChipType_Non && GetChipType(afterX, afterY) != ChipType_GoalPos)
	{
		//2020/12/14現在、この中にはChipType_Wallの時しか来ない
		//すなわち、壁に接触している間の処理をここで行う

		//マップチップの上下左右の座標を計算
		float chipLeft		= int (afterX / CHIP_SIZE) * CHIP_SIZE;			//左端
		float chipRight		= chipLeft + CHIP_SIZE;  									//右端
		float chipTop		= int (afterY / CHIP_SIZE) * CHIP_SIZE;			//上端
		float chipBottom	= chipTop  + CHIP_SIZE;									//下端

		//y方向のチェック
		if (moveY_ != nullptr)
		{
			//マップチップの上端に当たっていた場合の処理
			if (*moveY_ > 0.0f)
			{
				*moveY_ = chipTop - pos_.y - 1.0f;
				hitFlg |= (1 << CollisionDir_Up);
			}
			//マップチップの下端に当たっていた場合の処理
			else if (*moveY_ < 0.0f)
			{
				*moveY_ = chipBottom - pos_.y + 1.0f;
				hitFlg |= (1 << CollidionDir_Down);
			}
		}

		//x方向のチェック
		if (moveX_ != nullptr)
		{
			//マップチップの左端に当たっていた場合の処理
			if (*moveX_ > 0.0f)
			{
				*moveX_ = chipLeft - pos_.x - 1.0f;
				hitFlg |= (1 << CollisionDir_Left);
			}
			//マップチップの右端に当たっていた場合の処理
			else if (*moveX_ < 0.0f)
			{
				*moveX_ = chipRight - pos_.x + 1.0f;
				hitFlg |= (1 << CollisionDir_Right);
			}
		}
	}
	return hitFlg;
}

/*
	@param	x_		スクリーン座標ｘ
	y_		スクリーン座標ｙ
*/
ChipType Stage::GetChipType(float x_, float y_) const
{
	//画面上の座標から、配列上の座標へ変換する
	const int x = int(x_ / CHIP_SIZE);
	const int y = int(y_ / CHIP_SIZE);

	/*
	例題：配列[1][10]はマップチップ上のどこに描画されるか？
	マップチップの縦横はCHIP_SIZEとする。
	y = CHIP_SIZE * 1  = 32
	x = CHIP_SIZE * 10 = 320
	*/

	//配列のx、ｙが計算出来たら…
	if (x < 0 || y < 0 || x >= STAGE_W || y >= STAGE_H)
	{
		//何もなかったことにする
		return ChipType_Non;
	}

	//指定の座標に該当するマップの情報を返す
	return ChipType(m_Data[y][x]);				//←ここでアクセス違反が起こる
}

試したこと

補足情報（FW/ツールのバージョンなど）

使用ツール：
Visual Studio 2019
Dxlib

Answer 1

エラーの原因なのですが、今回提示して頂いている場所とは違うように思えます。
C++言語において、マイナスの数値が入らない限り、STAGE_WやSTAGE_Hを少し超えたぐらいの値ならメモリの壁をぶち破ってアクセスすることが可能です（ようはそのようなエラーにはならない）
そして一番の原因として考えられるのは、インスタンス側でDeleteした後にも関わらず呼び出しを行っているケースです。

二次元配列に沿ってマップを描画するようなコードを書いていて、1ステージ目の走破は出来るのですが、2ステージ目に進む前にゲームが強制終了してしまいます。

と発言していますので、１ステージ目の最後にStageクラスを解放していないか、ご確認をお願いします。

Comments

[kogarashi0917]

朝早くからご回答いただきありがとうございます。

「Delete後に呼び出す」というのは、つまりはdeleteしたのちに、nullptrを代入するなということでしょうか？

Stageクラスのdeleteにかかわるスクリプトも添付しますので、ご指摘いただけると助かります。

#include "InGameScene.h"
#include "DxLib.h"
#include "../Manager/GameManager.h"
#include "../Manager/InputManager.h"
#include "../Manager/SceneManager.h"
#include "../Manager/FadeManager.h"

//もし複数ステージor複数キャラクターがいるなら、同様にインクルードする
#include "../Stage/Stage.h"
#include "../Object/Player/Player.h"

//実行ステップ定義
enum
{
STEP_FADE_IN,
STEP_INPUT,
STEP_FADE_OUT,
STEP_END,
};
/*
無名の列挙体を用いて、このcpp内のみで扱う定数を宣言しておくと、
マジックナンバーによる混乱を防げる。
*/

//コンストラクタ
InGameScene::InGameScene()
:SceneBase()
,m_pStage(nullptr)
,m_pPlayer(nullptr)
{
m_Step = STEP_END;

//コンストラクタでnewすることで、
//クラス内でm_pStage, m_pPlayerが存在していることを保証する
m_pStage = new Stage();
m_pPlayer = new Player();

/*
switch (選択したキャラ番号)
{
case マリオ: m_pPlayer = new Mario();
case ルイージ: m_pPlayer = new Luigi();
・
・
・
以下、キャラクター分を逐次追加していく
}
*/

//ゲーム管理クラスへプレイヤーとステージを登録
GameManager* gameMng = GetGameManager();
gameMng->ResisterPlayer(m_pPlayer);
gameMng->ResisterStage(m_pStage);
}

//デストラクタ
InGameScene::~InGameScene()
{
//ゲーム管理クラスへ登録したプレイヤーとステージを解除
//登録した順とは逆順に解除
GameManager* gameMng = GetGameManager();
gameMng->UnresisterStage();
gameMng->UnresisterPlayer();

delete m_pPlayer;
m_pPlayer = nullptr;
delete m_pStage;　　　　←おそらくここのことでしょうか？
m_pStage = nullptr;

}

//実行
void InGameScene::Exec()
{
switch (m_Step)
{
case STEP_FADE_IN: step_FadeIn(); break;
case STEP_INPUT: step_Input(); break;
case STEP_FADE_OUT: step_FadeOut(); break;
default: break;
}

m_pPlayer->Exec();
}

//描画
void InGameScene::Draw()
{
m_pStage->Draw();
m_pPlayer->Draw();

//デバッグ表示
DrawString(20, 20, "Stage 1", GetColor(0, 0, 0));
}

//シーン初期化
void InGameScene::Init()
{
//スタート位置を設定した地点に
m_pPlayer->SetPos(m_pStage->GetPlayerStartPos());
GetFadeManager()->StartFadeIn(DEFAULT_FADE_FRAME);
set_Step(STEP_FADE_IN);
}

//シーンが終了しているか
bool InGameScene::IsEnd() const
{
return (m_Step == STEP_END);
}

//フェードイン
void InGameScene::step_FadeIn()
{
if (!GetFadeManager()->IsFadeFinished())
{
return;
}
set_Step(STEP_INPUT);
}

//入力受付
void InGameScene::step_Input()
{
const Position& playerPos = m_pPlayer->GetPos();
if (m_pStage->GetChipType(playerPos.x, playerPos.y) == ChipType_GoalPos)
{
GetFadeManager()->StartFadeOut(DEFAULT_FADE_FRAME);
set_Step(STEP_FADE_OUT);
}
}

//フェードアウト
void InGameScene::step_FadeOut()
{
if (!GetFadeManager()->IsFadeFinished())
{
return;
}
GetSceneManager()->SetNextScene(SceneID_InGameB);
set_Step(STEP_END);
}

[stdio]

> おそらくここのことでしょうか？
そこが呼ばれてからCheckCollision又はGetChipTypeが呼ばれていないでしょうか？

1. deleteのところにブレークポイントをおいた状態で、ゲームを開始
2. ブレイクポイントで止める
3. GetChipTypeにブレイクポイントをおいて通っていないかチェック

この手順に従って試してみてください。多分通ってしまっていると思います。
又はm_pStageのインスタンスを見直して下さい。今の状態ではポインター定義をしている意味がさほどないように思えます。

[kogarashi0917]

教えていただいた手順で確認してみたところ、確かにdeleteした後にGetChipTypeに通ってしまっています。
ただ、エラーが起こった時点で確認できた値が、最初はm_pStage = 0x0000000000000000だったのに対し、GetChipTypeにブレイクポイントを置いた場合はm_pStage = 0xccccccccccccccccとなっています。
この場合はどう対処すればよいのでしょうか？

[stdio]

それはアドレスの値ですね。deleteした直後はnullptrが入っているので、0となっています。
私にも詳しい原因は分かりませんが、C/C++の言語の特徴であるメモリの壁を破壊した結果、よく分からない値が取れてきてしまっているのだと考えられます。
つまり、対処方はありません。プログラムの動きを確認して、削除後は絶対にそこを通らないように設計してあげて下さい。