:-( Why does my code register two mouse clicks per click? :-(

Howdy ya'll,

When I run the following code & click the screen any number of times I get one undefined mouse click to start with then for each click I get a duplicate one.

Why is this?

Code:

---

/* main.cpp */
#include <SFML/System.hpp>
#include <SFML/Window.hpp>
#include <SFML/Graphics.hpp>
#include <SFML/Audio.hpp>
#include <iostream>
#include <string>
using namespace std;

int main( void )
{
    // set gameWindow parameters
    sf::RenderWindow gameWindow( sf::VideoMode( 1024, 600, 32 ), "Adam's Fantasy", sf::Style::Fullscreen );

    // fill gameWindow with black background
    gameWindow.Clear();

    // include buttonOne.jpg
    sf::Image Image;
    Image.LoadFromFile( "buttonOne.jpg" );
    sf::Sprite buttonOne;
    buttonOne.SetImage( Image );
    gameWindow.Draw( buttonOne );

    // include buttonTwo.jpg
    Image.LoadFromFile( "buttonTwo.jpg" );
    sf::Sprite buttonTwo;
    buttonTwo.SetImage( Image );
    buttonTwo.SetPosition( 512.f, 300.f );
    gameWindow.Draw( buttonTwo );

    // display the game window
    gameWindow.Display();

    // begin interaction loop
    while ( gameWindow.IsOpened() )
    {
        sf::Event Event;
        while ( gameWindow.GetEvent( Event ) )
        {
            // Press escape key to close the game
            if ( ( Event.Type == sf::Event::KeyPressed ) && ( Event.Key.Code == sf::Key::Escape ) )
                gameWindow.Close();

            // Left click
            if ( Event.MouseButton.Button == sf::Mouse::Left )
            {
                cout << "The left mouse button was clicked at " << Event.MouseButton.X << ", " << Event.MouseButton.Y << endl;
                    if ( Event.MouseButton.X >= 512 && Event.MouseButton.X <= 822 )
                    {
                        if ( Event.MouseButton.Y >= 300 && Event.MouseButton.Y <= 460 )
                            cout << "You clicked the picture" << endl;
                    } // end if
            } // end if
        } // while events are recieved
    } // while gameWindow.IsOpened
} // end main
//////////////////////////////////////////////////////////////////////////////////////////////////////////

---

Does your Event.MouseButton also tell you whether it was a "press" or "release".

If you do get down/up events for mouse clicks, you need to add this distinction to your code.

Thanks for your input but after checking YouTube I realized that the double clickings are solved by using a switch statement rather than an if statement.
I'm still struggling to stop the "out of bounds" coordinates when the mouse if moved outside of the renderWindow area in case you can help with that.

Quote:

---

Originally Posted by JM1082

Thanks for your input but after checking YouTube I realized that the double clickings are solved by using a switch statement rather than an if statement.
I'm still struggling to stop the "out of bounds" coordinates when the mouse if moved outside of the renderWindow area in case you can help with that.

---

Salem's right, JM. Your loop will trigger when any event is detected - and though the check for the "Key Press & Escape Key" is correct, the second check is incomplete. You're not checking for a mouse event -- you simply check the value of the field as though a mouse event fired.

Compare how KeyPressed, KeyReleased checks IF a key press event occurred, and is followed in short-circuit logic by a check of the event structure to determine if the key is the ESC key.

Similarly, MouseButtonPressed, MouseButtonReleased catch mouse events - then can be followed by your check of the Event structure for the Event.MouseButton.Button field to see if it equals sf::Mouse::Left, sf::Mouse::Right, sf::Mouse::Middle, etc.

Reference:
SFML - Simple and Fast Multimedia Library

Leaving before I ramble.

I'm pretty sure you must be doing something else.

Because just switching if to switch won't do a damn thing.

Code:

---

if ( var == const )

// and

switch ( var ) {
  case const:
}

---

Are functionally equivalent.