dynamic_cast causing run time error

**Micko** · 02-27-2004

Please look at the following code:

Code:

#include <iostream>
using namespace std;

class Pet
{
public:
	virtual void speak()
	{
		cout<<"Pet speak: "<<endl;
	}
	virtual ~Pet(){}
};
class Dog : public Pet
{
public:
	void speak()
	{
		Pet::speak();
		cout<<"AV AV"<<endl;
	}
	virtual void show()
	{
		cout<<"This one is new!"<<endl;
	}
	virtual ~Dog(){}
};

int main()
{
	Pet *p=new Dog;
	Pet *pb=new Pet;
	Dog *dg;
	dg=dynamic_cast<Dog*>( p);
	if(!dg)
	{
		cout<<"Zero!"<<endl;
		return 1;
	}
	dg->show();

}

I'm using Visual Studio .net
and when compiling get this warning:
warning C4541: 'dynamic_cast' used on polymorphic type 'Pet' with /GR-; unpredictable behavior may result
and when I execute program crashes.
I don't know why this happens and where could be mistake in this code.

**MrWizard** · 02-27-2004

You have to enable RTTI in project settings.

**Micko** · 02-28-2004

Thanks, I didn't know that.
Interesting when use
dg=(Dog *) p;
it works just fine?
What type of casting is this?
Is it C-style cast?

**skorman00** · 02-28-2004

I could be wrong, but I'm pretty sure that thats a type of static casting, because its explicitly done by you. That kind of cast is decided at compile time, so there is no reason to turn it "on".

**Polymorphic OOP** · 02-28-2004

Originally posted by Micko
Thanks, I didn't know that.
Interesting when use
dg=(Dog *) p;
it works just fine?
What type of casting is this?
Is it C-style cast?

Yeah, that's a C-style cast. It's result is like a mix of static_cast, reinterpret_cast, and const_cast -- by that I mean it will perform proper pointer arithmetic when casting via a hierarchy which makes it like static_cast, it can convert from pointer-to-const to pointer-to-nonconst like const_cast, and can convert between completely unrelated types, like reinterpret_cast. Try to stay away from C-style casts and use C++ casts instead. They're more specific, can make the intention clearer, and will disallow you from accidently doing more in the cast than you'd like (IE accidently casting away const with a static_cast).

So, using the c-style cast will work, however it won't check if the conversion is valid, whereas a dynamic_cast will. In general, you want to stay away from dynamic_cast anyways (preferably don't do any casting and solve your problem polymorphically if possible). If you're using dynamic_cast then theres pretty much a 99% chance that you have some constrewed logic in your program and theres probably a better solution. Use dynamic_cast sparingly, and try to only use it during development (try not to us it in a completed project, much like const_cast).

Aside from this, you should also realize that it's perfectly valid to use a static_cast to go both up and down the class hierarchy and it's preferable over a C-style cast.

dg=static_cast<Dog *>( p );

Thread: dynamic_cast causing run time error

Thread Tools

Search Thread

Display

dynamic_cast causing run time error

Similar Threads

How to monitor process creation?

Crazy errors caused by class, never seen before..

We Got _DEBUG Errors

Dikumud

pointer to array of objects of struct