Jump to content

  • Log In with Google      Sign In   
  • Create Account

SDL Keyboard input


Old topic!
Guest, the last post of this topic is over 60 days old and at this point you may not reply in this topic. If you wish to continue this conversation start a new topic.

  • You cannot reply to this topic
7 replies to this topic

#1 KotoR`eZ   Members   -  Reputation: 228

Like
0Likes
Like

Posted 06 July 2014 - 05:34 PM

Basically the problem in my program is when I move my character for example North/East and I hold W and D at the same time my character goes it a correct direction but when I let go W and continue holding D my character moves in D direction but it only plays the first frame of W animation for 1-2 seconds before switching to the W animation. Also when I was trying to "debug" the program I wrote some couts for which animation gets called (ACTIVE/IDLE) and I noticed that when I press move my active animation sometimes gets called 5 times but when ever I release the button the Idle animation gets called about 20-30 times. Can somebody please explain to me this behavior and also is there "industry standard" on setting up keyboard input? I was following lazyfoo's tutorials but those dont explain much in details.


void Game::GameLoop()

{


    while (!quit && sdl_setup->GetMainEvent()->type != SDL_QUIT)

    {

        sdl_setup->Begin();

        walkingGround->Draw();


        player->Draw();


        //float angle = atan2(Follow_Point_Y - player->GetY(), Follow_PointX - player->GetX());

        //angle = angle * (180/3.14);


        if(sdl_setup->GetMainEvent()->type == SDL_KEYDOWN)

        {

            switch(sdl_setup->GetMainEvent()->key.keysym.sym)

            {

            case SDLK_w:

                moveUp = true;

                break;

            case SDLK_s:

                moveDown = true;

                break;

            case SDLK_a:

                moveLeft = true;

                break;

            case SDLK_d:

                moveRight = true;

                break;

            default:

                break;

            }

        }

        if(sdl_setup->GetMainEvent()->type == SDL_KEYUP)

        {

            switch(sdl_setup->GetMainEvent()->key.keysym.sym )

            {

            case SDLK_w:

                std::cout GetX() + 1);

               // moveRight = false;

            }

	}

        sdl_setup->End();

    }

}



Sponsor:

#2 KotoR`eZ   Members   -  Reputation: 228

Like
0Likes
Like

Posted 06 July 2014 - 05:48 PM

The site crops my source code for some reason. Here is full source code pastebin.com/NHCNTsG4

#3 fastcall22   Crossbones+   -  Reputation: 4334

Like
1Likes
Like

Posted 06 July 2014 - 06:22 PM

You're only handling one event per frame. There can be anywhere from 0 to a few hundred of events per frame. If you only process one event per frame, then a keyboard or mouse event could take up to several seconds before it is processed.

A typical game loop looks something like this:
while running:
  event e
  while nextEvent(&e):
    processEvent(e)

  update()
  render()

c3RhdGljIGNoYXIgeW91cl9tb21bMVVMTCA8PCA2NF07CnNwcmludGYoeW91cl9tb20sICJpcyBmYXQiKTs=

#4 fastcall22   Crossbones+   -  Reputation: 4334

Like
1Likes
Like

Posted 06 July 2014 - 06:27 PM

Also, it does not appear your Play_annimatedSprite routine doesn't have the ability to be interrupted. If Play_animatedSprite were called for an animation such as (1 3 1 125), then (1 3 2 125) cannot be used until "animationDelay + speed < SDL_GetTicks()" is true
c3RhdGljIGNoYXIgeW91cl9tb21bMVVMTCA8PCA2NF07CnNwcmludGYoeW91cl9tb20sICJpcyBmYXQiKTs=

#5 KotoR`eZ   Members   -  Reputation: 228

Like
0Likes
Like

Posted 06 July 2014 - 08:03 PM

Thanks for response! Your suggestion did fix my problem however now I am getting a weird bug when if I just simply move character it works fine and then suddenly it just keeps going into random direction as if SDL did not notice that I release the button (SDL_KEYUP). In order to interrupt my character from moving uncontrollably I have to tap the button twice for sdl to register the SDL_KEYUP. At first I thought my keyboard was glitched so I rebind-ed the buttons. Same thing. I checked everywhere but I think I am doing something totally wrong with event handling. Any suggestions? pastebin.com/mGv4ZLmJ (updated)

#6 SimonForsman   Crossbones+   -  Reputation: 6111

Like
4Likes
Like

Posted 06 July 2014 - 09:30 PM

Thanks for response! Your suggestion did fix my problem however now I am getting a weird bug when if I just simply move character it works fine and then suddenly it just keeps going into random direction as if SDL did not notice that I release the button (SDL_KEYUP). In order to interrupt my character from moving uncontrollably I have to tap the button twice for sdl to register the SDL_KEYUP. At first I thought my keyboard was glitched so I rebind-ed the buttons. Same thing. I checked everywhere but I think I am doing something totally wrong with event handling. Any suggestions? pastebin.com/mGv4ZLmJ (updated)

 

If you are using this code: https://github.com/gratholio/sdlscroller as your base: please don't. it is making a huge mess of things.

the sdl_setup::begin method gets a single event from the queue and stores it as mainevent (which is then returned by all calls to GetMainEvent) and clears the renderer(big wtf), it also discards the return value from SDL_PollEvent, there is no way to use that wrapper correctly since the wrapper itself is completely broken.

 

Just use standard SDL instead.

 

[source]

 

while(!quit) {

    SDL_Event event;

    while(SDL_PollEvent(&event) != 0) {

        quit = event.type == SDL_QUIT;

        //handle the rest of the events here       

    }

    update();

    render();

}


Edited by SimonForsman, 06 July 2014 - 09:38 PM.

I don't suffer from insanity, I'm enjoying every minute of it.
The voices in my head may not be real, but they have some good ideas!

#7 KotoR`eZ   Members   -  Reputation: 228

Like
0Likes
Like

Posted 07 July 2014 - 08:02 AM

 

Thanks for response! Your suggestion did fix my problem however now I am getting a weird bug when if I just simply move character it works fine and then suddenly it just keeps going into random direction as if SDL did not notice that I release the button (SDL_KEYUP). In order to interrupt my character from moving uncontrollably I have to tap the button twice for sdl to register the SDL_KEYUP. At first I thought my keyboard was glitched so I rebind-ed the buttons. Same thing. I checked everywhere but I think I am doing something totally wrong with event handling. Any suggestions? pastebin.com/mGv4ZLmJ (updated)

 

If you are using this code: https://github.com/gratholio/sdlscroller as your base: please don't. it is making a huge mess of things.

the sdl_setup::begin method gets a single event from the queue and stores it as mainevent (which is then returned by all calls to GetMainEvent) and clears the renderer(big wtf), it also discards the return value from SDL_PollEvent, there is no way to use that wrapper correctly since the wrapper itself is completely broken.

 

Just use standard SDL instead.

 

[source]

 

while(!quit) {

    SDL_Event event;

    while(SDL_PollEvent(&event) != 0) {

        quit = event.type == SDL_QUIT;

        //handle the rest of the events here       

    }

    update();

    render();

}

 

Thanks a lot! Now it works perfect. I was actually following "Lets Make an RPG C++/SDL" tutorials on youtube but it appears that the code is exactly the same as the link you provided. Can you suggest a good resource to learn from or I should just stick to lazyfoo?



#8 SimonForsman   Crossbones+   -  Reputation: 6111

Like
2Likes
Like

Posted 07 July 2014 - 08:38 AM

 

 

Thanks for response! Your suggestion did fix my problem however now I am getting a weird bug when if I just simply move character it works fine and then suddenly it just keeps going into random direction as if SDL did not notice that I release the button (SDL_KEYUP). In order to interrupt my character from moving uncontrollably I have to tap the button twice for sdl to register the SDL_KEYUP. At first I thought my keyboard was glitched so I rebind-ed the buttons. Same thing. I checked everywhere but I think I am doing something totally wrong with event handling. Any suggestions? pastebin.com/mGv4ZLmJ (updated)

 

If you are using this code: https://github.com/gratholio/sdlscroller as your base: please don't. it is making a huge mess of things.

the sdl_setup::begin method gets a single event from the queue and stores it as mainevent (which is then returned by all calls to GetMainEvent) and clears the renderer(big wtf), it also discards the return value from SDL_PollEvent, there is no way to use that wrapper correctly since the wrapper itself is completely broken.

 

Just use standard SDL instead.

 

[source]

 

while(!quit) {

    SDL_Event event;

    while(SDL_PollEvent(&event) != 0) {

        quit = event.type == SDL_QUIT;

        //handle the rest of the events here       

    }

    update();

    render();

}

 

Thanks a lot! Now it works perfect. I was actually following "Lets Make an RPG C++/SDL" tutorials on youtube but it appears that the code is exactly the same as the link you provided. Can you suggest a good resource to learn from or I should just stick to lazyfoo?

 

 

The Lazyfoo tutorials are fairly high quality so they would be a good choice, i would recommend avoiding video tutorials (videos are a bad format for tutorials).

 

Video lectures however work great, these are worth watching(Not game specific):

http://ocw.mit.edu/courses/electrical-engineering-and-computer-science/6-00-introduction-to-computer-science-and-programming-fall-2008/video-lectures/


I don't suffer from insanity, I'm enjoying every minute of it.
The voices in my head may not be real, but they have some good ideas!




Old topic!
Guest, the last post of this topic is over 60 days old and at this point you may not reply in this topic. If you wish to continue this conversation start a new topic.



PARTNERS