EventReceiver question

[VeneX]

my eventreceiver:

Code: Select all

IAnimatedMeshSceneNode* node = 0;
IrrlichtDevice* device=0;

#pragma comment(lib, "Irrlicht.lib")

int main()
{
class MyEventReceiver : public IEventReceiver 
	{ 
	public: 

		virtual bool OnEvent(SEvent event) 
		{ 
		static bool model_anim = false;
		if (event.EventType == EET_KEY_INPUT_EVENT && !event.KeyInput.PressedDown) 
		{ 
            switch(event.KeyInput.Key) 
			{ 
				case KEY_KEY_W:
					{
						node->setMD2Animation(EMAT_RUN); 
					}
				case KEY_ESCAPE:
					{
						device->closeDevice();
					}
				default:
					{
						node->setMD2Animation(EMAT_STAND);
					}
			}
		 }
		 	
			if(event.EventType == EET_MOUSE_INPUT_EVENT && event.MouseInput.Event == EMIE_LMOUSE_PRESSED_DOWN )
			{
				node->setMD2Animation(EMAT_SALUTE);
			}
			return false;
		}
		 
	};

MyEventReceiver receiver;

	IrrlichtDevice *device = createDevice(EDT_DIRECTX8, dimension2d<s32>(1024, 768), 32, true, false, &receiver);

The node:

Code: Select all

SMaterial material;
	material.Texture1 = driver->getTexture("faerie2.bmp");
	material.Lighting = true;

	IAnimatedMesh* mesh = smgr->getMesh("faerie.md2");
	node = smgr->addAnimatedMeshSceneNode(mesh);

	if (node)
	{
		node->setPosition(vector3df(0,100,-90));
		node->setMD2Animation(EMAT_RUN);
		node->getMaterial(0) = material;
	}

If I use the W key the program exits hisself. What is wrong?
Thanx in advance.

[level1]

Ah, very easy, and just as easy to overlook:
you need to insert "break;" statements in the switch block

Code: Select all

switch(gugu)
{
case BLA:
  // dubidu
  break;
case BLU:
  // yeah baby yeah
  break;
default:
  // well...
}

[VeneX]

I feel stupid!

Thanks for your help dude.


What's wrong with this code:

Code: Select all

class MyEventReceiver : public IEventReceiver 
	{ 
	public: 

		virtual bool OnEvent(SEvent event) 
		{ 
		static bool model_anim = false;
		if (event.EventType == EET_KEY_INPUT_EVENT && !event.KeyInput.PressedDown) 
		{ 
            switch(event.KeyInput.Key) 
			{ 
				case KEY_KEY_W:
					{
						event.KeyInput.Key=KEY_UP;
						node->setMD2Animation(EMAT_RUN); 
						break;
					}
				case KEY_KEY_A:
					{
						event.KeyInput.Key=KEY_LEFT;
						break;
					}

[keless]

try putting the 'break;' outside of the code block:

Code: Select all

case KEY_KEY_W: 
               { 
                  event.KeyInput.Key=KEY_UP; 
                  node->setMD2Animation(EMAT_RUN); 
                   
               } 
break;

[Guest]

no. it useless cause 'break' breaks switch

1. I would recommend you ALWAYS use brackets when you are not sure about operators precedence. I have 7 years experience in C/C++ but I'm not sure which one is the first - 'comparison' or 'logical and'. Of course, you can use help but this makes your code difficult to read...

So I think you should set

Code: Select all

if ((event.EventType == EET_KEY_INPUT_EVENT) && !event.KeyInput.PressedDown)

After all I cannot see anything bad in this code. So nothing wrong with it.
Just don't forget that OnEvent function should return 0(false) if it doesn't process event, and nonzero (true) value if it does

[keless]

comparisson is done first, so he doesnt need parens here

[Guest]

You don't need the brackets in the case statements


class MyEventReceiver : public IEventReceiver
{
public:

virtual bool OnEvent(SEvent event) {
static bool model_anim = false;
if (event.EventType == EET_KEY_INPUT_EVENT && ! event.KeyInput.PressedDown) {
switch(event.KeyInput.Key) {
case KEY_KEY_W:
event.KeyInput.Key=KEY_UP;
node->setMD2Animation(EMAT_RUN);
break;
case KEY_KEY_A:
event.KeyInput.Key=KEY_LEFT;
break;

[Guest]

2keless: I have MSDN - thank you. But if you have ever tested your code for meeteing any code style then you should know that almost all of them give exactly this situation as an example how you don't have to write your code

2Guest: almost the same it's always better to set brackets than do not set (exception is only for one string expressions). BTW, if you declare variable in one case in switch without brackets - you'll get error (I think you know this)

2VeneX: Finally, can you tell me what's wrong with your code? I mean what goes with your program?

WBR,
Mike

[VeneX]

Sorry I forgot to say what didn't work. The code:

Code: Select all

event.KeyInput.Key=KEY_UP;

doesn't work....

I have a demo on my site now but the only pc he is working on is mine
I am trying to find the bug, expect some code...