[SOLVED]Problem with Mouse.x stored in Event.key.code

[Husenap]

So, my application has been closing all the time and driving me crazy, and I've thought that it crashed, but I later figured out that the problem lies here:

if(event.type == event.Closed || (event.key.code == sf::Keyboard::Escape)){
    m_window.close();
}
 

So what's happening is that the mouse.x position is stored in event.key.code every time the mouse is moved.
Now, sf::Keyboard::Escape == 36
So, every time my mouse is at pixel 36, the application closes, does anyone know how I can rewrite the above code, if possible, to make it work with the escape button?

Appreciate all answers!

[Jesper Juhl]

You are reading "event.key.code" before checking "event.type" to see if the event is a type where that field is valid. That's a straight up bug in your code.
The sf::Event is a union and you must check the 'type' member first before accessing other members. And then only access members that are actually valid for that specific type of event.

[Husenap]

Thanks for the quick answer

Silly of me not to check

if(event.type == event.KeyPressed)

[Jesper Juhl]

That should be

if (event.type ==  sf::Event::KeyPressed)

[Husenap]

What is the difference between using sf::Event::KeyPressed and event.KeyPressed when they return the same value?

[Laurent]

event.KeyPressed just happens to work, but what you mean is really sf::Event::KeyPressed, since the value is not tied to the event instance.

[Jesper Juhl]

What Laurent said.
Or phrased differently: What do you want to rely on? A guarantee given to you by the library you use or something that happens to work (for now) by sheer dumb luck?