Why is this code detecting the key twice?

[bobtran12]

Every time I hit the "x" key, it shows two outputs instead of one.

Code:

#include <SFML/Graphics.hpp>
#include <iostream>

int main()
{
        sf::Vector2f screenDimensions(800, 600);
        sf::RenderWindow Window;
        Window.create(sf::VideoMode(screenDimensions.x, screenDimensions.y), "Test", sf::Style::Titlebar | sf::Style::Close);

        Window.setKeyRepeatEnabled(false);

        int i = 0;

        while(Window.isOpen())
        {
                sf::Event Event;
                while(Window.pollEvent(Event))
                {
                        if(sf::Keyboard::isKeyPressed(sf::Keyboard::X))
                        {
                                std::cout << i << std::endl;
                                i++;
                        }
                }
        }
}

Output (when pressing "x" key once):

0
1

How do I fix it so that it only detects and outputs one?

EDIT: I'm on Windows 7 and using SFML 2.1.

[Ixrec]

You probably want events, not real-time input.  You should read the tutorials on both input methods so you understand the difference and why you want events in this case.

http://www.sfml-dev.org/tutorials/2.2/window-events.php
http://www.sfml-dev.org/tutorials/2.2/window-inputs.php

Incidentally, your current code is mixing real-time input with event-based input, which is always a really bad idea, as this example proves.  In this case, I believe you're getting exactly two outputs because isKeyPressed is being called after the KeyPressed event and the KeyReleased event.  Any other event (say, moving the mouse) would probably cause it to output some more.

[bobtran12]

You probably want events, not real-time input.  You should read the tutorials on both input methods so you understand the difference and why you want events in this case.

http://www.sfml-dev.org/tutorials/2.2/window-events.php
http://www.sfml-dev.org/tutorials/2.2/window-inputs.php

Incidentally, your current code is mixing real-time input with event-based input, which is always a really bad idea, as this example proves.  In this case, I believe you're getting exactly two outputs because isKeyPressed is being called after the KeyPressed event and the KeyReleased event.  Any other event (say, moving the mouse) would probably cause it to output some more.

Got it. Thanks.

So, would something like this have any problems?

#include <SFML/Graphics.hpp>
#include <iostream>

int main()
{
        sf::RenderWindow Window;
        Window.create(sf::VideoMode(800, 600), "Test", sf::Style::Titlebar | sf::Style::Close);

        Window.setKeyRepeatEnabled(false);

        int i = 0;

        while(Window.isOpen())
        {
                sf::Event Event;
                while(Window.pollEvent(Event))
                {
                        if (Event.type == sf::Event::KeyPressed)
                        {
                                if (Event.key.code == sf::Keyboard::X)
                                {
                                        std::cout << i << std::endl;
                                        i++;
                                }
                        }
                }
        }
}

[Ixrec]

>So, would something like this have any problems?

That code looks perfectly fine to me.

[Hapax]

Could do with being able to close the window, though...