「定義されていない識別子です。」というエラーの原因が分からない

Question

前提・実現したいこと

Visual Studio上でDXライブラリを使用してアクションゲームを作っています。
エラーなく動いていたファイルを分割し、新たに当たり判定を実装したところ「ビルド エラーが発生しました。続行して、最後に成功したビルドを実行しますか？」というダイアログが出て以下のエラーメッセージが発生しました。

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

重大度レベル	コード	説明	プロジェクト	ファイル	行	抑制状態
エラー	C2065	'player': 定義されていない識別子です。	Action_Game	Collision.cpp	8
エラー	C2065	'player': 定義されていない識別子です。	Action_Game	Collision.cpp	9	
エラー	C2065	'enemy': 定義されていない識別子です。	Action_Game	Collision.cpp	10	
エラー	C2065	'enemy': 定義されていない識別子です。	Action_Game	Collision.cpp	11	
エラー	C2065	'Font': 定義されていない識別子です。	Action_Game	Collision.cpp	17	
エラー	C2065	'player': 定義されていない識別子です。	Action_Game	Collision.cpp	24	
エラー	C2065	'player': 定義されていない識別子です。	Action_Game	Collision.cpp	25	
エラー	C2065	'enemy': 定義されていない識別子です。	Action_Game	Collision.cpp	26	
エラー	C2065	'enemy': 定義されていない識別子です。	Action_Game	Collision.cpp	27	
エラー	C2065	'Font': 定義されていない識別子です。	Action_Game	CCollision.cpp	33	
エラー	C2065	'Key': 定義されていない識別子です。	Action_Game	Player.cpp	49	
エラー	C2065	'Key': 定義されていない識別子です。	Action_Game	Player.cpp	57	
エラー	C2065	'Key': 定義されていない識別子です。	Action_Game	Player.cpp	82	
エラー	C2065	'Key': 定義されていない識別子です。	Action_Game	Player.cpp	86	

該当のソースコード

//main.cpp

#include "DxLib.h"
#include "Player.h"
#include "Enemy.h"
#include "Collision.h"

//初期ウィンドウサイズは640x480

//キー押下フレーム数
char Key[256];

int WINAPI WinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance,
	LPSTR lpCmdLine, int nCmdShow) {
	if (ChangeWindowMode(TRUE) != 0) {
		return 0;
	}
	if (DxLib_Init() != 0) {
		return 0;
	}
	if (SetDrawScreen(DX_SCREEN_BACK) != 0) {
		return 0;
	}
	if (SetBackgroundColor(0, 255, 0) != 0) {
		return 0;
	}

	//音楽ファイルロード
	int Handle_bgm = LoadSoundMem("./Audio/famipop3.mp3");

	//音楽再生
	PlaySoundMem(Handle_bgm, DX_PLAYTYPE_LOOP);

	Player_Initialize();

	Enemy_Initialize();

	while (1) {
		if (ScreenFlip() != 0) {
			break;
		}
		if (ProcessMessage() != 0) {
			break;
		}
		if (ClearDrawScreen() != 0) {
			break;
		}
		if (GetHitKeyStateAll(Key) != 0) {
			break;
		}

		Player_Update();

		Enemy_Update();

		CheckHitWidth();

		CheckHitHeight();

		Player_Draw();

		Enemy_Draw();
	}

	Player_Finalize();

	Enemy_Finalize();

	//DXライブラリ終了処理
	DxLib_End();

	return 0;
}

//--------------------------------------------------

//Player.cpp

#include "DxLib.h"
#include "Player.h"

//プレイヤー画像用
typedef struct {
	int x, y, y_prev, y_temp, flag, fflag, img;
} player_t;
player_t player;
int Handle_p[16];
bool jflag = false;
bool sflag = false;

//フォントデータロード
int Font = CreateFontToHandle("メイリオ", 32, 4, DX_FONTTYPE_ANTIALIASING_EDGE);

//初期化
void Player_Initialize() {
	//画像ファイルロード
	LoadDivGraph("./Image/player.png", 16, 4, 4, 64, 64, Handle_p);

	//プレイヤー座標
	player.x = 64 / 2;
	player.y = 480 - (64 / 2);
	player.y_prev = 0;
	player.y_temp = 0;

	//フラグ
	player.flag = 0;
	player.fflag = 0;
}

//計算
void Player_Update() {
	//プレイヤー移動制御
	if (player.x < (64 / 2)) {
		player.x = (64 / 2);
	}
	if (player.x + (64 / 2) > 640) {
		player.x = 640 - (64 / 2);
	}
	if (player.y < (64 / 2)) {
		player.y = (64 / 2);
	}
	if (player.y + (64 / 2) > 480) {
		player.y = 480 - (64 / 2);
	}

	//下キーが押下された場合
	if (Key[KEY_INPUT_DOWN] == 1) {
		//しゃがみフラグ
		sflag = true;
	} else {
		sflag = false;
	}

	//上キーが押下され現在のy座標が480 - (64 / 2)でしゃがみフラグが立っていない場合
	if (Key[KEY_INPUT_UP] == 1 && player.y == 480 - (64 / 2) && sflag == false) {
		//ジャンプフラグ
		jflag = true;
		//現在のy座標を前回分に保存
		player.y_prev = player.y;
		//現在のy座標を24減らす
		player.y = player.y - 24;
	}

	//ジャンプフラグが立っている場合
	if (jflag == true) {
		//現在のy座標を一時分に保存
		player.y_temp = player.y;
		//現在y += (現在y - 前回y) + 1
		player.y += (player.y - player.y_prev) + 1;
		//一時保存のy座標を前回分に保存
		player.y_prev = player.y_temp;
		//現在のy座標が480 - (64 / 2)の場合
		if (player.y == 480 - (64 / 2)) {
			//ジャンプフラグを元に戻す
			jflag = false;
		}
	}

	//右キーが押下された場合
	if (Key[KEY_INPUT_RIGHT] == 1) {
		//右フラグ
		player.flag = 1;
		//左キーが押下された場合
	} else if (Key[KEY_INPUT_LEFT] == 1) {
		//左フラグ
		player.flag = 2;
		//押下されなかった場合
	} else {
		//停止フラグ
		player.flag = 0;
	}

	if (player.flag == 2) {
		if (sflag == true) {
			player.img = Handle_p[13];
		} else {
			//xを減らす
			player.x--;
			player.fflag = 2;
			if (jflag == true) {
				player.img = Handle_p[12];
			} else {
				//x座標に合わせた画像を指定
				player.img = Handle_p[(player.x % 64) / 16 + 8];
			}
		}
	} else if (player.flag == 1) {
		if (sflag == true) {
			player.img = Handle_p[5];
		} else {
			//xを増やす
			player.x++;
			player.fflag = 1;
			if (jflag == true) {
				player.img = Handle_p[4];
			} else {
				//x座標に合わせた画像を指定
				player.img = Handle_p[(player.x % 64) / 16];
			}
		}
	} else if (player.fflag == 2) {
		if (sflag == true) {
			player.img = Handle_p[13];
		} else if (jflag == true) {
			player.img = Handle_p[12];
		} else {
			player.img = Handle_p[8];
		}
	} else {
		if (sflag == true) {
			player.img = Handle_p[5];
		} else if (jflag == true) {
			player.img = Handle_p[4];
		} else {
			player.img = Handle_p[0];
		}
	}
}

//描画
void Player_Draw() {
	//タイトル描画
	DrawStringToHandle(20, 20, "アクションゲーム", GetColor(0, 0, 0), Font, GetColor(255, 255, 255));

	//画像描画
	DrawGraph(player.x - (64 / 2), player.y - (64 / 2), player.img, TRUE);
}

//終了処理
void Player_Finalize() {
	DeleteGraph(player.img);
}

//--------------------------------------------------

//Player.h

#ifndef DEF_PLAYER_H

#define DEF_PLAYER_H

//初期化
void Player_Initialize();

//計算
void Player_Update();

//描画
void Player_Draw();

//終了処理
void Player_Finalize();

#endif

//--------------------------------------------------

//Enemy.cpp

#include "DxLib.h"
#include "Enemy.h"

//エネミー画像用
typedef struct {
	int x, y, img;
} enemy_t;
enemy_t enemy;
int Handle_e[16];

//初期化
void Enemy_Initialize() {
	//画像ファイルロード
	LoadDivGraph("./Image/enemy.png", 16, 4, 4, 64, 64, Handle_e);

	//エネミー座標
	enemy.x = 640 - (64 / 2);
	enemy.y = 480 - (64 / 2);
}

//計算
void Enemy_Update() {
	//エネミー移動制御
	if (enemy.y < (64 / 2)) {
		enemy.y = (64 / 2);
	}
	if (enemy.y + (64 / 2) > 480) {
		enemy.y = 480 - (64 / 2);
	}

	//xを減らす
	enemy.x--;
	//x座標に合わせた画像を指定
	enemy.img = Handle_e[(enemy.x % 64) / 16];
}

//描画
void Enemy_Draw() {
	//画像描画
	enemy.img = Handle_e[0];
	DrawGraph(enemy.x - (64 / 2), enemy.y - (64 / 2), enemy.img, TRUE);
}

//終了処理
void Enemy_Finalize() {
	DeleteGraph(enemy.img);
}

//--------------------------------------------------

//Enemy.h

#ifndef DEF_ENEMY_H

#define DEF_ENEMY_H

//初期化
void Enemy_Initialize();

//計算
void Enemy_Update();

//描画
void Enemy_Draw();

//終了処理
void Enemy_Finalize();

#endif

//--------------------------------------------------

//Collision.cpp

#include "DxLib.h"
#include "Player.h"
#include "Enemy.h"
#include "Collision.h"

//横の当たり判定
int CheckHitWidth() {
	int L1 = player.x;
	int R1 = player.x + 64;
	int L2 = enemy.x;
	int R2 = enemy.x + 64;
	if (R1 < L2 || R2 < L1) {
		//当たっていない
		return 0;
	} else {
		//当たっている
		DrawStringToHandle(20, 400, "Game Over", GetColor(0, 0, 0), Font, GetColor(255, 255, 255));
		return 1;
	}
}

//縦の当たり判定
int CheckHitHeight() {
	int U1 = player.y;
	int D1 = player.y + 64;
	int U2 = enemy.y;
	int D2 = enemy.y + 64;
	if (D1 < U2 || D2 < U1) {
		//当たっていない
		return 0;
	} else {
		//当たっている
		DrawStringToHandle(20, 400, "Game Clear", GetColor(0, 0, 0), Font, GetColor(255, 255, 255));
		return 2;
	}
}

//--------------------------------------------------

//Collision.h

#ifndef DEF_COLLISION_H

#define DEF_COLLISION_H

//横の当たり判定
int CheckHitWidth();

//縦の当たり判定
int CheckHitHeight();

#endif

試したこと

「#include」に何らかの問題があるのかと思い文を見直してみましたがおかしいと思う部分は発見できず、インターネット上で調べても解決に役立ちそうなページは発見できませんでした。

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

Microsoft Visual C++ 2019
DXライブラリ Windows版 Visual C++用 Ver3.22c

Answer 1

Collision.cpp 内の関数内にて

int L1 = player.x;

等と書かれていますが，この記述では，

• player とは何なのか？　（→　Player.cpp内に定義されている外部変数の名前であるが…）
• player とはどんな型なのか？　（→　Player.cpp内にて定義されているstructだが…）

の２点を解決する必要があります．
enemy側も同様です．
（それが解決できていないのがエラーの原因）

---

Collision.cpp 内の関数の実装を見るに，
外部変数をダイレクトに触らなくとも，player.x や player.y の値さえ知ることができればよい様子ですから，

• int GetPlayerX() : player.xの値を返す
• int GetPlayerY() : player.yの値を返す

のような関数を player.cpp に用意してやり（そして player.h　にそれらの宣言を書き），Collision.cpp側では，これらの関数を呼べば済むのではないでしょうか．

Comments

[fkcz_yfm_carp]

ありがとうございます。
Player.cppに int GetPlayerX = player.x;
Player.hに int GetPlayerX;
と加え(GetPlayerYやGetEnemyX, Yも同様です)Collision.cppの当たり判定のplayer.xもGetPlayerXに修正した結果、エラーを表すコード上の赤い波線は消えましたがデバッグビルドしてみたところ「再定義されました。」というエラーが表示されました。。。

重大度レベル コード 説明 プロジェクト ファイル 行 抑制状態
エラー C2086 'int GetEnemyX': 再定義されました。 Action_Game Enemy.cpp 12
エラー C2086 'int GetEnemyY': 再定義されました。 Action_Game Enemy.cpp 13
エラー C2086 'int GetPlayerX': 再定義されました。 Action_Game Player.cpp 16
エラー C2086 'int GetPlayerY': 再定義されました。 Action_Game Player.cpp 17

[fana]

うーん，この様子だと，物作る前にちょっとC/C++言語の基礎的な事柄の復習をした方がいいんじゃないか？と思えます．

「関数を用意すればよくね？」という話なので，その話を試すならば関数を書きましょう．

> int GetPlayerX = player.x;

これでは，新しい変数 GetPlayerX が増えただけですし，player.x の値が変化しても GetPlayerX の値がそれに追従するわけでもないですから，役に立たないでしょう．

Player.cppに書く関数定義としては，例えば

int GetPlayerX(){ return player.x; }

といったものになります．
この関数を呼べば，結果として player.x の（その時点における）値を関数の戻りとして得られる，という点は大丈夫ですか？

で，この関数を呼ぶためには
「関数の宣言（プロトタイプ宣言）がその関数を呼ぶ箇所よりも前に存在する必要がある」というC/C++言語のルールを満たす必要がある（という話もOK？）．
これは，最も単純には，Collision.cpp の先頭の方に

int GetPlayerX(); //プロトタイプ宣言

とかダイレクトに書いてしまえばまぁ良いわけですが，
回答文内では（それと同等のことになる話として）このプロトタイプ宣言は player.h の方に書いてやればいいんじゃないか，としています．
（現状，Collision.cppで player.h を include してますし．）

[fana]

#include とは何か？　という話も不安であれば把握された方がよいかと．
https://teratail.com/questions/324710#reply-450241

[dodox86]

別回答でのfanaさんの[2021/04/12 17:45]のコメント

> player_t という型の定義は Player.cpp にある．
> なので，Collision.cpp 内に，「player_t 型の player っていうインスタンスがどこかにあってさー…」と書いても「player_t 型って何やねん？」という話になる．
> player_t 型の定義を何らかのヘッダにでも移して，player_t型を知る必要があるcpp群でそれをincludeするとかすれば良いかと．
> （enemy_t の側も同様）

が言わば本質的な部分での指摘だと思えるのですが、質問者さんはまだピンと来ていないのでしょうね。いちコンパイル単位で、C++コンパイラはその型を知っている必要があるが、それが分からないから「定義されていない識別子です。」エラーになる。場当たり的に修正しても解決できない（或いは解決できても、理由が分からない）かもしれません。

[fana]

一旦，今のプログラムは置いといて，cppが２つくらいの小さいテストプログラムか何かで諸々復習すると良いんじゃないかな？

[fkcz_yfm_carp]

Player.cppで
int GetPlayerX() {
return player.x;
}
Player.hで
int GetPlayerX();
とそれぞれ修正しました、その結果
//12行目
int L1 = GetPlayerX;
重大度レベル コード 説明 プロジェクト ファイル 行 抑制状態
エラー C2440 '初期化中': 'int (__cdecl *)(void)' から 'int' に変換できません。 Action_Game Collision.cpp 12
//13行目
int R1 = GetPlayerX + 64;
重大度レベル コード 説明 プロジェクト ファイル 行 抑制状態
エラー C2296 '+': 無効です。左オペランドには型 'int (__cdecl *)(void)' が指定されています。 Action_Game Collision.cpp 13
このようになりました。

> この関数を呼べば，結果として player.x の（その時点における）値を関数の戻りとして得られる，という点は大丈夫ですか？
> で，この関数を呼ぶためには「関数の宣言（プロトタイプ宣言）がその関数を呼ぶ箇所よりも前に存在する必要がある」というC/C++言語のルールを満たす必要がある（という話もOK？）．
> #include とは何か？　という話も不安であれば把握された方がよいかと．

このあたりの話は分かります、添付して頂いた質問の回答にも初めて見るようなものは特にありませんでした。。。

ちなみに試しにplayer_t型の定義をヘッダーファイルに移してみたところ
重大度レベル コード 説明 プロジェクト ファイル 行 抑制状態
エラー LNK2005 "struct player_t player" (?player@@3Uplayer_t@@A) は既に Collision.obj で定義されています。 Action_Game main.obj 1
これと同じようなエラーがいくつも出ました。

本質的、根本的なエラーの原因はまだピンと来ないです。。。

[fkcz_yfm_carp]

epistemeさんの別回答についてですが
extern player_t player;
とすると
重大度レベル コード 説明 プロジェクト ファイル 行 抑制状態
エラー C4430 型指定子がありません - int と仮定しました。メモ: C++ は int を既定値としてサポートしていません Action_Game Collision.cpp 6
エラー C2146 構文エラー: ';' が、識別子 'player' の前に必要です。 Action_Game Collision.cpp 6
とエラーが出るためエラーが出ない
extern struct player_t player;
としています。

[fana]

int GetPlayerX(); なる関数を呼ぶ記述は

> int L1 = GetPlayerX;

ではない，ということからわからない状況なのであれば，
さすがにもう「まずは使う言語の文法を一から学ぶべき」というアドバイスしかできない．
失礼な言い方かもしれないけども，このような場でやりとりできる状態に達していないものと見受ける．
先にC言語の入門的な本なりサイトなりで２時間～２日程度の学習を行った方が絶対早い．

（質問文に提示されているコードが書けるのならば，こんな状態はあり得ないと思えるのだが…？？）

[fana]

> 本質的、根本的なエラーの原因

は既に述べられている．

元々のエラー
> 'player': 定義されていない識別子です
というのは，変数(名) player の通用範囲の問題だ．
変数の名前が通用する範囲には制限がある．「通用範囲」とか「スコープ」とかで調べると良い．

ちなみに，その後のエラーについては，以下のような話であり，いずれも基礎中の基礎を抑えていれば他人に訊くまでもない話．
「宣言」とか「定義」とかいう話の理解が根本的にあやふやなのだと見える．

> エラー C2146 構文エラー: ';' が、識別子 'player' の前に必要です。

Collision.cpp に，player_t 型の定義が必要となる記述を行ったが，
その player_t 型の定義が Collision.cpp から見えていない．

> エラー C2440 '初期化中': 'int (__cdecl *)(void)' から 'int' に変換できません。

前述のように，関数を呼び出す記述の文法が間違っている．

> エラー LNK2005 "struct player_t player" (?player@@3Uplayer_t@@A) は既に Collision.obj で定義されています。

（コードがどんな状態になったのか不明瞭だが）多重定義になったのだろう．

[fana]

邪推するに，元々ファイルが複数のcppに分かれているコードをどこかから引っ張ってきて，それをスタート地点にしようとしているように見えるが，
そこから開始するのはやや難易度が高すぎると思う．

そうでなくても各種基礎的な事柄の習得が必要そうなので，
分割コンパイル（複数のcppに分けて書く話）の件が問題を複雑にしないように，まずは単一のソースコードで完結しているような例を探してスタート地点にした方がよいのではないだろうか．

[fkcz_yfm_carp]

このコードは学習サイトを参考にしながら自分で書いた後に複数に分割したものです。
元々JavaをやっていてC++に移ったのである程度は書けましたが。。。

一度C言語の基本的なところから復習してみます。
ありがとうございました。

Answer 2

グローバル変数 player と enemy の存在を Collision.cpp は知らないのでしょう。

Collision.cpp の #include 部の直後あたりに

extern player_t player;
extern enemy_t enemy;

を追加してみてはいかがかと。

Comments

[fkcz_yfm_carp]

追加してみたところplayerとenemyのエラーが以下のように変わりました。
FontとKeyについては変わらずでした。

重大度レベル コード 説明 プロジェクト ファイル 行 抑制状態
エラー C4430 型指定子がありません - int と仮定しました。メモ: C++ は int を既定値としてサポートしていません Action_Game Collision.cpp 6
エラー C2146 構文エラー: ';' が、識別子 'player' の前に必要です。 Action_Game Collision.cpp 6
エラー C4430 型指定子がありません - int と仮定しました。メモ: C++ は int を既定値としてサポートしていません Action_Game Collision.cpp 7
エラー C2146 構文エラー: ';' が、識別子 'enemy' の前に必要です。 Action_Game Collision.cpp 7

[episteme]

Font, Key についても同様、extern int Font; extern char Key[256];

[fkcz_yfm_carp]

ありがとうございます、Font, Keyのエラーは無事に消えました。

player. enemyの方は追加文を
extern struct player_t player;
extern struct enemy_t enemy;
としてみたところエラーは以下のようになりました。

重大度レベル コード 説明 プロジェクト ファイル 行 抑制状態
エラー C2027 認識できない型 'player_t' が使われています。 Action_Game Collision.cpp 12, 13, 28, 29
エラー C2027 認識できない型 'enemy_t' が使われています。 Action_Game Collision.cpp 14, 15, 30, 31

何度も質問してしまい申し訳ないです。

[episteme]

extern player_t player;
extern enemy_t enemy;

じゃなくて?

[fana]

player_t という型の定義は Player.cpp にある．
なので，Collision.cpp 内に，「player_t 型の player っていうインスタンスがどこかにあってさー…」と書いても「player_t 型って何やねん？」という話になる．
player_t 型の定義を何らかのヘッダにでも移して，player_t型を知る必要があるcpp群でそれをincludeするとかすれば良いかと．
（enemy_t の側も同様）

[episteme]

型の定義が(.hじゃなく).cppにあるんか。 それだとダメなんだが。