ich habe eine funktion, die instanzen für asteroiden erstellt, die man mit einem raumschiff abschießen kann (ich weiß, unschlagbares konzept). rein optisch funktioniert auch alles. um zu schauen, wieviele instanzen vorhanden sind habe ich einen zähler eingebaut, der statisch ist und beim aufruf des konstruktors hochzählt und beim destruktur runterzählt. selbst wenn ich die ausgabe ständig aktualisiere und sehe, wie neue asteroiden erzeugt werden ändert sich der zählstand nicht, sondern bleibt auf 0. (die ausgabe des zählstandes wird natürlich permanent aktualisiert)

was mir noch auffiel ist, dass in der zeile:

CAsteroid tempAsteroid(&render);

der name der instanz rot markiert ist. ich verwende msvc6 und visual assist x. rot werden normalerweise nur funktionen angezeigt, instanzen jedoch grau. ich habe auch geschaut, dass ich sicher keine funktion mit diesem name habe.

ich hoffe, jemand kann mir sagen, was hier schiefläuft. ich habe jetzt zwei tage lang jede zeile nochmal durchgeschaut und geändert. nichts hilft.

vielen dank schonmal

CAsteroid tempAsteroid(&render);

Das sieht doch nach dem Aufruf eines benutzerdefinierten Konstruktors aus; und das ist sehr wohl ein Funktionsaufruf.

So kann dir keiner helfen^^

void CGame::spawnAsteroids(void)
{
        // drawarea ist eine instanz einer klasse, die für das rendern 
        // verantwortlich ist
	CAsteroid tempAsteroid(&DrawArea);

	m_lAsteorid.push_back(tempAsteroid); 
}

hier wird also ein asteroid erzeugt.

CAsteroid::CAsteroid(CDrawEngine* de)
{
        // zähler für instanzen 
	CAsteroid::m_instanceCntr++;

	// zeiger übergeben
	m_pDrawArea = de;

	// ein asteroid hat ein leben
	addLives(1);

	// einen zufälligen spawnpoint festlegen
		m_fPosX = (float)((rand() % (m_pDrawArea->m_screenWidth - 2))+    1);
		m_fPosY = 1;

	m_pDrawArea->drawSprite(m_index, (int)(m_fPosX), (int)(m_fPosY));
}

und hier noch ein ausschnitt aus der updatefunktion, die die asteroiden bewegt

if (m_lAsteorid.empty() == false)
	{
		// jeden asteroid abfragen
		for (Itr = m_lAsteorid.begin(); Itr != m_lAsteorid.end(); ++Itr)
		{
                         // diese funktion bewegt den asteroid weiter
			(Itr)->updateAsteroid();

                        // wen der asteroid tot ist...
			if ((Itr)->isAlive() == false)
                                // ...aus der liste entfernen	
				Itr = m_lAsteorid.erase(Itr);

		}
	}

und hier noch der destruktor des sprites

CAsteroid::~CAsteroid(void)
{
	CAsteroid::m_instanceCntr--;
	m_pDrawArea->eraseSprite(m_fPosX, m_fPosY);
}

also wie schon gesagt, der zähler funktioniert nicht und wenn ich in den konstruktor etwas einbaue wie
cout << "Konstruktor aufgerufen";
dann erscheint auch das nicht. das programm schein meinen konstruktor einfach zu übersehen, obwohl alles richtig in der headerdatei angegeben ist.

meinst du das hier?

CAsteroid asteroid1(&DrawArea);
CAsteroid asteroid2(asteroid1);

Ist die statische Varibale initialisiert?

/edit/
erste Frage nehme ich zurück. Hab es gerade entdeckt

int CAsteroid::sm_instanceCntr = 0;

scheint ja auch zu funktionieren. der zähler steht wie gewollt zu beginn auf 0.

Was passiert wenn du die Syntax leicht abänderst, also statt

CAsteroid tempAsteroid(&render);

CAsteroid tempAsteroid = CAsteroid(&render);

verwendest?

Nur als Test, ob zumindest die rote Kennzeichnung als Funktion verschwindet... *grübel*

ich glaube ich werds wohl so hinnehmen müssen. solange der rest in den funktionen wie erwartet funktioniert ist das spiel ja funktionsfähig.

danke für euer bemühen

Du erzeugst ja ein Objekt, dass nicht lange erhalten bleibt. Das Schieben in den Container wird ja mit dem Copy-Konstruktor erledigt. D.h. das Objekt wird gleich wieder gelöscht. Könnte es daran liegen?

Wenn ja, hätte camper das Problem eigentlich schon erwähnt.

/edit/
Der negative Zähler lässt sich dadurch erklären, dass der Destruktor die Objekte im Container runterzählt, die der Copy-Konstruktor nicht hochgezählt hat.

Wenn es das auch nicht ist, gebe ich auch auf.

Folgendes könnte passieren:

Der Konstruktor zählt hoch.
Das Objekt kommt in den Container mit dem Copy-Konstruktor.
Der Destruktor zählt runter, weil das Objekt ja nur lokal gültig ist und beim Verlassen der Methode zerstört wird, d.h. die Variable m_instanceCntr bleibt unverändert.

Wird das Objekt im Container zerstört zählt der Destruktor nochmal runter.

/edit/
Am Quellcode verdeutlicht:

void CGame::spawnAsteroids(void)
{
        // drawarea ist eine instanz einer klasse, die für das rendern
        // verantwortlich ist
        // hier ist m_instanceCntr noch 0
    CAsteroid tempAsteroid(&DrawArea); 
        // nun ist m_instanceCntr 1
    m_lAsteorid.push_back(tempAsteroid);
        // für das Objekt im Container bleibt m_instanceCntr unverändert, 
        // da der Copy-Konstruktor wahrscheinlich m_instanceCntr nicht ändert

       // m_instanceCntr wir wieder auf 0 gesetzt, 
       // da tempAsteroid hier zerstört wird
       // und es somit zum Aufruf des Destruktors kommt
}

Irgendwann später im Quellcode wird auch das Objekt im Container m_lAsteorid zerstört, was zum Destruktoraufruf führt und dazu, dass m_instanceCntr runtergezählt und somti negativ wird.

OMG ...

Na, das ist doch kein Grund das höchste "Metaobjekt" anzurufen.

Du hast natürlich Recht, der Debugger wäre die schnellste Methode gewesen. Wenn ich das richtig sehe, hat camper die Lösung auch schon nach ca. 3 Stunden ins Forum gestellt. Leider zeigte sich der "Patient" uneinsichtig, was bis zur Stunde anhält.

t512 schrieb:

hmmm. erscheint mir nicht schlüssig. ...

1. Erezugen (Zähler++)
2. Kopie in den Container (passiert nicht mit zähler)
3. zestörung nach kopie (Zähler--)

-- objekt lebt

4. obekt wird ausm container gelöscht (Zähler--)