SetCursorPos problem.

[time to c]

i was trying to create a small program.
the program moves the cursor key whenever the arrow keys (up,down,left,right) are pressed.

sometime the thing goes smooth and sometimes it just slows down too much.
here is the code

#include<windows.h>

int main()
{

    int x=0,y=0;
    while(1)
    {
        Sleep(1);
        if(GetAsyncKeyState(39))
        {
            if(x<1280)
            x++;
            SetCursorPos(x,y);
            Sleep(1);
        }
        if(GetAsyncKeyState(37))
        {
            if(x>0)
            x--;
            SetCursorPos(x,y);
            Sleep(1);
        }
        if(GetAsyncKeyState(40))
        {
            if(y<800)
            y++;
            SetCursorPos(x,y);
            Sleep(1);
        }
        if(GetAsyncKeyState(38))
        {
            if(y>0)
            y--;
            SetCursorPos(x,y);
            Sleep(1);
        }
    }
    return 0;
}

any advice to make things better?
using GNU compiler and codeblocks OS is windows 7.

[Computergeek01]

The problem here is with how fast you are iterating through your loop. I rewrote a few things and got it to work a little better:

#include<windows.h>

int main()
{
    POINT cursor;
   
    while(!GetAsyncKeyState(VK_ESCAPE))
    {
        GetCursorPos(&cursor);
        
        if(GetAsyncKeyState(39))
        {
            if(cursor.x<1280)
            cursor.x++;
            SetCursorPos(cursor.x,cursor.y);
        }
        if(GetAsyncKeyState(37))
        {
            if(cursor.x>0)
            cursor.x--;
            SetCursorPos(cursor.x,cursor.y);
        }
        if(GetAsyncKeyState(40))
        {
            if(cursor.y<800)
            cursor.y++;
            SetCursorPos(cursor.x,cursor.y);
        }
        if(GetAsyncKeyState(38))
        {
            if(cursor.y > 0)
            cursor.y--;
            SetCursorPos(cursor.x,cursor.y);
        }
        
        Sleep(1);
    }
    return 0;
}

The major change is me consolidating all of your separate "Sleep()" calls into one spot. Sleep() isn't what you want to use here but for some reason I'm having trouble remembering the right function.

[webJose]

SwitchToThread? http://msdn.microsoft.com/en-us/library/windows/desktop/ms686352(v=vs.85).aspx

[Texan40]

Is there some way to key WaitForSingleObject() off of keyboard input?

[webJose]

No, but if that's the route then I'd say: Keyboard hook.

[beginning]

Using the SetWindowsHookEx function to install a key board hook.

[time to c]

Thanks for help.