Keyboard auto repeat

[akiross]

Yep! It helps Thanks i didn't notice it.

Ok, at least now i can use that function to check... But i still prefer to choose if enabling key repeat (well, i'm doing a game, and i come with the libSDL experience, where by default key repeat is off but you can enable and modify it as your wish). To me, disabling it at all will result in a more straightforward code...

... But for now this is ok

So, well, this is just mine sudgestion for qt developers: a function like QWidget::enableKeyRepeat(bool) would be even better

Fullmetalcoder, thanks a lot!

EDIT: Also thanks to wysota, the idea of using a set for multiple keypress is really nice! I used (but in C, without sets) to do something like char pressed[256];
But sets are far better for this.
Thanks again!

[wysota]

If you're making a game, then you have a very simple solution... As you probably want your game to be stable independent of the computer it is running on, you should use a QTimer object (or a timer event) to synchronise your game loop. And if you do, you can simply check the key state there instead of relying on key press/release events. QSet might be helpfull here as well.

QSet<Qt::Key> keysPressed;
void W::keyPressEvent(QKeyEvent *ev){
  keysPressed += (Qt::Key)ev->key();
}
void W::keyReleaseEvent(QKeyEvent *ev){
  keysPressed -= (Qt::Key)ev->key();
}
 
void W::timerEvent(QTimerEvent *ev){
  foreach(Qt::Key k, keysPressed){
    //...
  }
}

QSet<Qt::Key> keysPressed;
void W::keyPressEvent(QKeyEvent *ev){
  keysPressed += (Qt::Key)ev->key();
}
void W::keyReleaseEvent(QKeyEvent *ev){
  keysPressed -= (Qt::Key)ev->key();
}

void W::timerEvent(QTimerEvent *ev){
  foreach(Qt::Key k, keysPressed){
    //...
  }
}

To copy to clipboard, switch view to plain text mode 

[akiross]

Mmh well actually i won't need the set in this particular game, i just liked the idea

For this one i'll rely on events to handle user inputs. Main loop will be used only for drawing purpose. Actually, Qt is class based, so events are outside the main loop.
In SDL context, events were put in an event queue and polled at every refresh, so a set would be wonderful there.

But here i'm using a timer for refresh, events aren't syncronized with this timer because i'm keeping objects interact asyncronously: a physic engine will have its loop for integrate the simulation, GLwidget loop will have its loop for refreshing the screen, but the game under the hood isn't synced.

Well, i'm just trying this approach, since it's the first time i do a game with Qt (and C++)... If i'll found that doesn't work good, i'll use the classic method

Thanks anyway

[fullmetalcoder]

Actually, Qt is class based, so events are outside the main loop.

In Qt input events are fed by the system (X11, Window$, ...) to the event loop of the GUI thread (== main loop) which dispatch them to proper receivers.

[akiross]

yes yes, i know

I intended to say that usually you do it by hand (in SDL it's like)

while (1) { // main loop
  while (SDL_PollEvent(&ev)) {
    switch (ev.type) {
       // Handle here events
    }
  // Here game drawing loop
}

while (1) { // main loop
  while (SDL_PollEvent(&ev)) {
    switch (ev.type) {
       // Handle here events
    }
  // Here game drawing loop
}

To copy to clipboard, switch view to plain text mode 

While in qt the event polling is done by Qt; I'm saying that in Qt events are handled outside main loop because a Press method will be called on Press event.

To me, this is not only more readable and clear to write in code, but leaves also implementation freedom. If Qt a day will prefer to map directly system's event to my application, in this way there is no need to pass by an event queue, with SDL method it doesn't.

(well, it's quite ugly as example but i hope you got )