C++ Class Definitions

[lolxdflyx3]

Hallo,
ich sitzt vor nem Fehler und komm einfach nicht weiter. Mittlerweile denke ich schon fast, dass es am Compiler liegt

Hier sind 3 Versuche mit Errors:

Versuch 1:

 Spoiler

Code:

class CConsignmentMng
{
private:
	map<DWORD,char> m_mapItems;
	
	CConsignmentMng();
	~CConsignmentMng();

public:
	static CConsignmentMng *Instance();

	public bool IsBuyable(DWORD dwSEQ);
	public void BuyItem(DWORD dwSEQ);
	public void RemoveItem(DWORD dwSEQ);
	public void AddItem(DWORD dwSEQ, char cState = 'O');
}

Error:

error C2144: syntax error : 'bool' should be preceded by ':'
error C2144: syntax error : 'void' should be preceded by ':'
error C2144: syntax error : 'void' should be preceded by ':'
error C2144: syntax error : 'void' should be preceded by ':'

Versuch 2:

 Spoiler

Code:

class CConsignmentMng
{
private:
	map<DWORD,char> m_mapItems;
	
	CConsignmentMng();
	~CConsignmentMng();

public:
	static CConsignmentMng *Instance();

	bool IsBuyable(DWORD dwSEQ);
	void BuyItem(DWORD dwSEQ);
	void RemoveItem(DWORD dwSEQ);
	void AddItem(DWORD dwSEQ, char cState = 'O');
}

Error:

error C2533: 'CConsignmentMng::__ctor' : constructors not allowed a return type

==>

CConsignmentMng::CConsignmentMng()
{
	m_mapItems.clear();
}

Versuch 3:

 Spoiler

Code:

class CConsignmentMng
{
private:
	map<DWORD,char> m_mapItems;
	
	CConsignmentMng();
	~CConsignmentMng();

public:
	static CConsignmentMng *Instance();

	bool IsBuyable(DWORD dwSEQ);
	void BuyItem(DWORD dwSEQ);
	void RemoveItem(DWORD dwSEQ);
	void AddItem(DWORD dwSEQ, char cState = 'O');
};

Error:

error C2653: 'CConsignmentMng' : is not a class or namespace name
error C2227: left of '->AddItem' must point to class/struct/union
type is ''unknown-type''
//...

Was mach ich falsch?

mfg lolxdfly

[LcPlayer1]

1. Versuch: Entferne die ganzen publics vor den Methodendeklarationen, die stehen ja schon im Publicbereich.

Alternative(wenn auch unnötig): hinter jedes public ein ":"

Edit: Vergiss meinen Beitrag, hab mir die anderen Versuche nicht angeschaut...

[Dr. Coxxy]

hast die memberdefinitionen i-wo vor der deklaration oder vergessen den header in der cpp einzubinden.

[lolxdflyx3]

Originally Posted by Dr. Coxxy

hast die memberdefinitionen i-wo vor der deklaration oder vergessen den header in der cpp einzubinden.

der Header ist in der Cpp drinne!

Was genau meinst du mit

Originally Posted by Dr. Coxxy

hast die memberdefinitionen i-wo vor der deklaration

[Dr. Coxxy]

bei versuch 3:

error C2653: 'CConsignmentMng' : is not a class or namespace name
error C2227: left of '->AddItem' must point to class/struct/union

er kennt offensichtlich an der stelle wo du CConsignmentMng->AddItem definierst noch nicht die CConsignmentMng klasse.
außerdem sollte man afaik bei nem singleton den destruktor public machen und noch den = operator private überladen.

[Tyrar]

Jede Methode einzeln als public zu deklarieren ist nicht nötig.
Hinter der abschließenden Klammer sollte ein Semikolon stehen.
Nutzt nur die "Instance" Methode den Konstruktor? Zumindest erlaubst du dem Compiler nicht, den Konstruktor von woanders aufzurufen. Wobei das auch einen anderen Fehler verursachen würde, wie wird der genau aufgerufen?