Back to For Beginners

Unexpected crash

For Beginners

Started by PumpkinPieman November 14, 2003 01:12 PM

12 comments, last by PumpkinPieman 20 years, 5 months ago

PumpkinPieman

382

Author

November 14, 2003 01:12 PM

Alright, I have no clue what causes this, but is there someone who can tell me what''s going wrong.

Unhandled exception at 0x00427c6b in Twilight.exe: 0xC0000005: Access violation reading location 0xabababab.


int main()
{
	CPlayer NewPlayer;
	delete NewPlayer.m_PlayerBelt; //Pointer to CBelt
	delete NewPlayer.m_PlayerBag; //Pointer to CBag, which uses std::multimap as a base class
	std::cout << sizeof(NewPlayer);
	return 0;
}; // Error occurs here.

The violation occurs at (Refined)


	iterator begin()
		{	// return iterator for beginning of mutable sequence
		return (_TREE_ITERATOR(_Lmost()));
		}

[PumpkinPieman.net] [Quadris]

Chozo

176

November 14, 2003 01:16 PM

What''s your constructor/destructor look like? You''re deleting something that we don''t know how got constructed, then you aren''t setting it to NULL so you''re probably accessing it in the destructor. Too little information with too many things it could be.

Just as an aside. With classes, you should make your members private, and only free them from within the class. And always, always, always set pointers to NULL when you free them.

JTippetts

13,307

November 14, 2003 01:18 PM

What does your CPlayer class look like? If m_PlayerBelt and m_PlayerBag are defined as pointers, then immediately calling delete on them after CPlayer is instanced is a bad idea, as they have not yet been allocated. If they are not pointers, then calling delete on them again is a bad idea. At any rate, deleting individual members of a class is bad form; that should be handled within the class, transparent to the user of the class.

EDIT: Heh. Too slow, Chozo beat me to it.

Josh
vertexnormal AT linuxmail DOT org

Check out Golem: Lands of Shadow, an isometrically rendered hack-and-slash inspired equally by Nethack and Diablo.

[edited by - VertexNormal on November 14, 2003 2:19:47 PM]

Accidental Noise Library
The Adventures of Goblinson Crusoe
Terrain Editor

PumpkinPieman

382

Author

November 14, 2003 01:27 PM

CPlayer::CPlayer(void): m_modHP(1.0), m_modChi(1.0), m_modWei(1.0){	m_PlayerBelt = new Belt;	m_PlayerBag = new CBag<int, CItem*>;}CPlayer::CPlayer(double m_pmodHP, double m_pmodChi, double m_pmodWei): m_modHP(m_pmodHP), m_modChi(m_pmodChi), m_modWei(m_pmodWei){	m_PlayerBelt = new Belt;	m_PlayerBag = new CBag<int, CItem*>;}CPlayer::~CPlayer(void){	if(m_PlayerBelt)		delete m_PlayerBelt;	if(m_PlayerBag)		delete m_PlayerBag;}

That''s the deconstructor of the CPlayer Class. I want to beable to delete those two pointers eventually.

[PumpkinPieman.net] [Quadris]

JTippetts

13,307

November 14, 2003 01:31 PM

Aha. If you delete those two pointers in your main() function but do not then set them to 0, they are deleted again when the program exits and the CPlayer destructor is called on NewPlayer. In main(), after you delete the pointers, set them to 0 and the crash should be fixed.

Josh
vertexnormal AT linuxmail DOT org

Check out Golem: Lands of Shadow, an isometrically rendered hack-and-slash inspired equally by Nethack and Diablo.

Accidental Noise Library
The Adventures of Goblinson Crusoe
Terrain Editor

Wildfire

154

November 14, 2003 01:32 PM

-----------------------------------------------------------
To expand on what VertexNormal said:

You should (usually) try to put all the classes resource management inside the class. First you should make sure that the constructor of CPlayer either new's the pointers, or sets them to NULL/0 explicitly. The destructor of CPlayer should then be responsible for deleting them.
If you're trying to go for OO, you should probably make the pointers private, and provide getter methods. This way a user of your class can't accidently delete resources of your class.
-----------------------------------------------------------

Edit: Wrote my reply before your answer. Looks like you're already doing this. Like VertexNormal said, you're deleting the pointers twice. This is a good examply why pointers inside the class should be private. That would've prevented that mistake.

Also, the "if(pointer) delete pointer"-construction, is unnecessary 'delete' already takes that case into account.

[edited by - Wildfire on November 14, 2003 2:38:37 PM]

How do I set my laser printer on stun?

Greatwolf

125

November 14, 2003 01:35 PM

It looks like you''re deleting each member variable twice... once in your main and another time in your destructor once main goes out of scope. As mentioned before, in your destructor set your variables to NULL once you delete them.

--{You fight like a dairy farmer!}

PumpkinPieman

382

Author

November 14, 2003 02:01 PM

Is there some way to check if a pointer has gone out of scope?

[edited by - PumpkinPieman on November 14, 2003 3:52:41 PM]

[PumpkinPieman.net] [Quadris]

sagwagon

122

November 14, 2003 06:13 PM

quote:
For Beginners - Unexpected crash

BWAHAHAHAHAHAHAHAHA

PumpkinPieman

382

Author

November 14, 2003 07:09 PM

quote:Original post by sagwagon
quote:
For Beginners - Unexpected crash

BWAHAHAHAHAHAHAHAHA

It's nice to see the topic amuses you, but unless you have anything constructive to say; then don't bother saying it.

[edited by - PumpkinPieman on November 14, 2003 8:10:52 PM]

[PumpkinPieman.net] [Quadris]

Unexpected crash

This topic is closed to new replies.

Popular Topics

Recommended Tutorials

Unexpected crash

This topic is closed to new replies.

Popular Topics

Recommended Tutorials

Reticulating splines