ich habe einen auto_ptr:

std::auto_ptr<MeineKlasse> m_apInstance;

Dann habe ich noch eine Methode:

MeineKlasse &MeineKlasse::GetInstance()
{
	if (m_apInstance == NULL)
	{
		m_apInstance = std::auto_ptr<MeineKlasse> (new MeineKlasse());
	}

	return *m_apInstance;
}

Der Compiler (VC++ 6) meldet folgendes:
Error C2678: Binaerer Operator '==' : Kein Operator definiert, der einen linksseitigen Operator vom Typ 'class std::auto_ptr<class CDBManager>' akzeptiert (oder keine geeignete Konvertierung moeglich)

Ich habe gesehen, dass bei auto_ptr keinen "=="- oder "!="-operator gibt.

Kann ich denn irgendwie abfragen, ob m_apInstance schon initialisiert wurde, außer dass ich eine Flag-Variable like: static bool m_bInitialized verwenden muss?

if (m_apInstance.get() == NULL)

m_apInstance ist ja ein auto_ptr der per default = 0 gesetzt wird.
Daher wollte ich ja ihm auch it new MeineKlasse() den Speicher zuweisen.

Ich habe im Moment ein Fehler in der Klasse, daher kann ich
if (m_apInstance.get() == NULL)
oder
if (m_apInstance->get() == NULL)
noch nicht testen.

Es sollte aber nicht funktionieren, da m_apInstance = 0 vom auto_ptr gesetzt ist.

Zumindest wenn du z.B. CString *pStr = NULL; machst und dann pStr->GetLength() oder was anderes aufriufst gibt's Ärger, da noch kein Speicher mit new oder alloc zugewiesen ist.

Zumindest wenn du z.B. CString *pStr = NULL; machst und dann pStr->GetLength() oder was anderes aufriufst gibt's Ärger, da noch kein Speicher mit new oder alloc zugewiesen ist.

Die get-Memberfunktion von auto_ptr gibt doch nur den Zeiger auf das unterliegende Objekt zurück. Und diesen prüfst du doch dann nur auf NULL. Da wird ja nichts illegales aufgerufen.

Also std::auto_ptr<MeineKlasse>* pAutoPtr;

MeineKlasse &MeineKlasse::GetInstance()
{
	if (!m_apInstance)
		m_apInstance.reset(new MeineKlasse());

	return *m_apInstance;
}

Kann ich denn irgendwie abfragen, ob m_apInstance schon initialisiert wurde, außer dass ich eine Flag-Variable like: static bool m_bInitialized verwenden muss?

Ja klar, genauso wie in GetInstance.

BTW eignet sich boost::scoped_ptr hier eigentlich noch besser, weil std::auto_ptr ein sehr unintuitives Kopier- und Zuweisungsverhalten hat, was sich auf die Klasse überträgt. scoped_ptr läßt Kopieren und Zuweisen einfach gar nicht zu. Das mag aber nicht so entscheidend sein, weil deine Klasse (als Singleton) wahrscheinlich sowieso privaten Kopierkonstruktur und Zuweisungsoperator hat.

vielleicht! schrieb:

Die get-Memberfunktion von auto_ptr gibt doch nur den Zeiger auf das unterliegende Objekt zurück. Und diesen prüfst du doch dann nur auf NULL. Da wird ja nichts illegales aufgerufen.

Du hattest vollkommen Recht
if (c_apInstance.get() == 0) funktioniert. Ich musste bloss die Dokumentation von std::auto_ptr gründlicher lesen..
Dann wäre ich auch für meine eigene Faulheit nicht bestrafft ;=)

@Bashar
Ich habe das bisschen anders gelöst. Nachdem ich if (c_apInstance.get() == 0) geprüft habe und in dem if scope mit new neuen Speicher zugewiesen habe, anschließend: ASSERT(c_apInstance.get()); gemacht.
Meine Klasse ist ein singleton und da nur eine Instance gleichzeitig erlaubt ist,
macht das absolut keinen Sinn die einzige Instance Variable c_apInstance zu deleten und neuen Speicher zu zuweisen

Danke an alle!

Das Forum ist ein bisschen buggy. Dich trifft kaum schuld.