c++でシングルトンクラスのインスタンスへの参照ができない。

Question

前提・実現したいこと

シングルトンクラスのインスタンスの参照を得たいのですが、できなくて困っています。
初めて投稿します。何か失礼があるかもしれませんがご容赦ください。c++初心者です。
シングルトンクラスを実装して、そのクラスのインスタンスの参照をenemyクラス内で得ようとしたのですが、エラーになってしまいます。
OSはwindows 10コンパイラはvisual studio 2019 communityを使用しています。
エラーはsingletonが定義されていない識別子です、と出ます。しかし、なぜかすべてのsingletonの文字は水色になっていて、クラス名と識別されているようですが。コンパイラが悪いのでしょうか？
自分の知識や理解が足りていないのは重々承知なのですが、ご教授いただければ幸いです。

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

singletonが定義されていない識別子です。e_Controlが定義されていない識別子です。singleton識別子がクラス名でも名前空間でもありません。getinstance()識別子が見つかりません。と出ます。

該当のソースコード

#include <iostream>
using namespace std;

class enemy {
private:
	int c;
public:
	void e_all();
};

void enemy::e_all() {
	cout << "e_all" << endl;

    singleton& e_Control = singleton::getinstance();
}

class singleton {
private:
	enemy* Enemy;
public:
	static singleton& getinstance() {
		static singleton Control;
		return Control;
	}
	void c_all();
	singleton();
};

void singleton::c_all() {
	cout << "c_all" << endl;
	Enemy->e_all();
}

singleton::singleton() {
	Enemy = new enemy;
}

int main() {
	singleton* con = new singleton;

	con->c_all();

	return 0;
}

試したこと

正直さっぱり分からなくて何もできていません。すいません。

Answer 1

C++ ではいくつかの例外を除いて使う識別子は使う場所よりも先に宣言されていなければなりません。

「前方宣言」で検索すれば良い例がたくさんヒットするので参考にしてください。

Comments

[dodox86]

なぜ低評価なのでしょうね。私も宣言順とclass enemy; の前方宣言の付与で解決できるものだと思いましたが。

C++のクラスの前方宣言の仕方がよくわからないです。- teratail
https://teratail.com/questions/94312

[ozwk]

enemy::e_all() の定義をclass singleton の宣言のあとに置けばコンパイル通るようになったんで別にこれでいいはず

[anitotsu411]

前方宣言ですか。わかりました。
参考にします。回答ありがとうございました。

Answer 2

ふるまいが正しいかはしらんけど、
これ↓でコンパイルエラーは消えます。

#include <iostream>
using namespace std;

class enemy {
private:
    int c;
public:
    void e_all();
};

class singleton {
private:
    enemy* Enemy;
public:
    static singleton& getinstance() {
        static singleton Control;
        return Control;
    }
    void c_all();
    singleton();
};

void enemy::e_all() {
    cout << "e_all" << endl;

    singleton& e_Control = singleton::getinstance();
}

void singleton::c_all() {
    cout << "c_all" << endl;
    Enemy->e_all();
}

singleton::singleton() {
    Enemy = new enemy;
}

int main() {
    singleton* con = new singleton;

    con->c_all();

    return 0;
}

※ singletonが複数個newできたらシングルトンじゃないけども。

Comments

[anitotsu411]

わざわざ回答ありがとうございます。参考にします。

Answer 3

singleton のクラスは別のヘッダーに置いて下さい。
それとシングルトンクラスの場合はデフォルトコンストラクタとデフォルトデストラクタは、privateに入れて下さい。

class singleton {
private:
    enemy* Enemy;

    singleton();
    ~singleton();
public:
    static singleton& getinstance() {
        static singleton Control;
        return Control;
    }
    void c_all();
};

詳しくは、こちら

Comments

[anitotsu411]

助言ありがとうございます。試してみます。