エラーの発生時の原因理解と対処方法

Question

前提・実現したいこと

ここに質問の内容を詳しく書いてください。
C++を使って簡単なゲームを作っています。シーンマネージャークラスを作ってタイトル画面やプレイ画面等の描写をこのクラスに任せるつもりだったのですが、途中からシングルトンにしようと手を加えたらエラーが出てしまい困っています。どうにかしてエラーが出ないように直したいです。

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

Player* player;に対して
c2143 ';'が'*'の前にありません。
c4430 型指定子がありません。 intと仮定します

std::list<Cat*>cats;等のlistに対して
c2065 Cat,Rabbit,Cancer定義されてない識別子です
c2059 構文エラー：'>'

list<>::iterator it
c2039 'iterator':'grobal namespace'のメンバーではありません
c3645 'C_it','R_it'...: 不明なオーバーライド指定子です
C2327	'SceneManager::iterator': 型名、スタティック、または列挙子ではありません。
C2086	'unknown-type SceneManager::iterator': 再定義されました。

//SceneManagerクラスを書き直したら消えました
C2011	'SceneManager': 'class' 型の再定義
C2027	認識できない型 'SceneManager' が使われています。

該当のソースコード

class SceneManager
{
private:
	//描画用ハンドル
	int title_bcg;
	int select_bcg;
	int cursour;
	int shotgun;
	int sniper;
	int pubg;
	int lock;
	int stage_1;
	int stage_2;
	int stage_3;

	int clear = 2;
	int stage = 1;


public:

	SceneManager()
	{
		sequence.change(&SceneManager::play);
	}

	void title(float t);
	void select(float t);
	void play(float t);
	void stage_select();
	void spawn(int limit);
	
//ここから下がエラーになってしまう
	Player* player;
	std::list<Cat*>cats;
	std::list<Cat*>::iterator C_it;

	std::list<Rabbit*>rabbits;
	std::list<Rabbit*>::iterator R_it;

	std::list<Cancer*>cancers;
	std::list<Cancer*>::iterator K_it;

	CSequence<SceneManager>sequence;
};

//CancerクラスやRabbitクラスも基本的にはPlayerクラスと一緒
class Player 
{
private:
	//描画用ハンドル
	int normal;
	int move;
	int snipe;
	int shot;
	int jump;
	int death;
	int cut;

	int x;
	int y;
	int radius = 34;

	

	int state = 0;
	int cooltime = 30;
	int count = 0;
	bool wait = false;
	bool cutin = false;
	bool cool = true;

public:
	Player()
	{
		sequence.change(&Player::draw_player);
		x = 100;
		y = 400;
	}
	int* get_X() { return &x; }
	int* get_Y() { return &y; }
	

	void update();
	void draw_player(float t);
	void Cutin();

//Playerクラスだとエラーが発生しない
	Rifle* rifle;
	

	CSequence<Player>sequence;
};

//これを継承させようとしてたらエラーが発生し始めた
template<class T>
class Singleton
{
public:
	static inline T& GetInstance()
	{
		static T instance;
		return instance;
	}

protected:
	Singleton() {} // 外部でのインスタンス作成は禁止
	virtual ~Singleton() {}

private:
	void operator=(const Singleton& obj) {} // 代入演算子禁止
	Singleton(const Singleton& obj) {} // コピーコンストラクタ禁止
};

試したこと

シングルトンを継承させようとするまでは問題なく動いたので、シングルトンの基底クラスの継承を止めて一度コードを書き直したりはしました

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

使用しているツールはVisualStudio2019になります。

Answer 1

C++コンパイラは、ソースコードの頭から順番にコードを解析していきます。自分で定義した型や関数などを使うまでに、その型や関数の名前が宣言されていなければ、エラーになります。たとえ、後の方で宣言してあっても、コンパイラはわざわざ後の方まで探してくれません。

上のコードでは、class SceneManagerの定義の中で、いきなりPlayer* player;とPlayerクラスを使おうとしています。しかし、実際の定義は下の方にあって、使用している場所ではまだ不明なのでエラーになっています。

ポインタの変数を定義する場合は、クラスの中身は、まだ必要ないので、名前だけ宣言することができます。class SceneManagerを定義する前に、class Playerの名前を宣言しましょう。

class Player;

class SeceneManger
{
    // 以下略

Cat,Rabbit,Cancerなども恐らく同じだと思います。

class Player;のような宣言については、「前方宣言」で検索すれば詳しい説明が見つけられると思います。

Comments

[Parsley_068]

返信おくれてすみません・・・
言われた通りに前方宣言した所問題なく動きました！ありがとうございます！
わかりやすい解説も大変助かりました！