Runtime error pri mazani z vectoru – C / C++ – Fórum – Programujte.com

Dobry den,
Kupil som si knihu 'Naucte se programovat pocitacove hry za 24 hodin' a pri testovani prikladov, po tom co som ich skompiloval z prilozenych zdrojakov dava runtime error ked sa v kode maze sprite z vectoru. Snazil som sa prist na to, co to moze sposobovat a na 90 % je to kod

// Handle the SA_KILL sprite action

if (saSpriteAction & SA_KILL)

{

    // Notify the game that the sprite is dying

    SpriteDying(*siSprite);

    // Kill the sprite

    delete (*siSprite);

    m_vSprites.erase(siSprite);

    siSprite--;

    continue;

}

Jo, to je super přístup učit se tak, že člověk opisuje kusy kódu a přitom z tomu vůbec ničemu nerozumí... Takhle nikam nedojdeš. Ta kniha už podle názvu vypadá nedůvěryhodně...

Runtime error ti to hází asi proto, že přistupuješ v paměti někam, kam nemáš, to znamená na místo, které sis nealokoval. To je obvykle způsobené použitím ukazatele, který ukazuje na neplatné místo, protože buď nebyl inicializován nebo objekt, na který ukazoval, už neexistuje.

od pohledu bych tipl ze to bude proto ze siSprite bude nulovy (popr. nenulovy ale neplatny) pointer - pak by ale pricina chyby byla nekde jinde.
ukazal jsi dost malo kodu, z toho nelze jiste vyvodit nic, pokud neznam kontext, tedy jake hodnoty maji vstupni promenne a co delaji volane funkce. doporucuji ti pustit to nejakym debuggerem (treba gdb), ktery ti ukaze na kterem radku kodu to spadlo a muzes si nechat vypsat hodnoty promennych v dobe havarie.

hlavne nevim k cemu se vaze to continue - dost dobre muzes mit chybu v ridici podmince toho cyklu, ktera nezkontroluje ze uz si smazal posledni validni siSprite...

a presne jak rikal Quiark, zjisti si co ty kusy kodu maji delat, protoze i kdyz tu chybu najdeme, tobe to vlastne vubec nic neda. to uz si rovnou stahni zdrojak OpenAreny, zmen jmeno v titulcich a muzes machrovat zes nakodil hru.

už to ide :D Naozaj išlo o zneplatnenie iterátoru, thx much, stačilo pridať riadok

if(siSprite -1 > m_vSprites.begin());

	siSprite--;

nazdar, tu knihu som mal tiez v ruke a v tom engine je chyba. Uz si to uplne presne nepametam, pretoze to bolo rok dozadu, ale upravoval som tieto dve metody:

void GameEngine::UpdateSprites()

{

	// Update the sprites in the sprite vector

	RECT rcOldSpritePos;

	SPRITEACTION saSpriteAction;

	vector<Sprite*>::iterator siSprite = m_vSprites.begin();



	while (siSprite != m_vSprites.end())

	{

		// Save the old sprite position in case we need to restore it

		rcOldSpritePos = (*siSprite)->GetPosition();



		// Update the sprite

		saSpriteAction = (*siSprite)->Update();



		// Handle the SA_KILL sprite action

		if (saSpriteAction & SA_KILL)

		{

			// Notify the game that the sprite is dying

			SpriteDying(*siSprite);



			// Kill the sprite

			delete (*siSprite);



			siSprite = m_vSprites.erase(siSprite);

			continue;

		}			



		// See if the sprite collided with any others

		if (CheckSpriteCollision(*siSprite))

			// Restore the old sprite position

			(*siSprite)->SetPosition(rcOldSpritePos);

		

		++siSprite;

	}

}

void GameEngine::CleanupSprites()

{

	// Delete and remove the sprites in the sprite vector

	vector<Sprite*>::iterator siSprite;

	for (siSprite = m_vSprites.begin(); siSprite != m_vSprites.end(); 

		siSprite++)

	{

		delete (*siSprite);

	}



	m_vSprites.clear();

}

Dakujem.