2

我一直在用 kbhit() 测试一些东西,并发现延迟无限循环的奇怪行为。在此代码示例中,我将循环延迟为每秒运行 30 次。

#include <stdio.h>
#include <stdlib.h>
#include <windows.h>
#include <conio.h>

_Bool IsKeyDown(char c)
{
    if(kbhit())
    {
        char ch1 = getch();
        if(ch1 == c)
            return 1;
    }
    return 0;
}

/*
 * 
 */
int main(int argc, char** argv) {
    while(1)
    {
        if(IsKeyDown('a'))
        {
            printf("HELLO\n");
        }
        if(IsKeyDown('b'))
        {
            printf("HI\n");
        }
        Sleep(1000 / 30);
    }
    return (EXIT_SUCCESS);
}

问题是,虽然循环中的第一个 if 语句工作正常,但第二个 if 语句几乎不起作用。如果在此示例中按住“a”键,则会无限期打印 HELLO。如果您按住“b”键,则几乎不会打印 HI(如果有的话)。

为什么会出现这种行为?

4

2 回答 2

5

发生这种情况是因为您在返回时IsKeyDown消耗缓冲区中的下一个字符。由于您连续调用两次,第一次调用“吃掉” ,所以当运行第二次调用时,缓冲区中没有数据。kbhittrueIsKeyDown'b'

您需要重新组织代码,以便IsKeyDown每次循环迭代仅调用一次。您的循环应该存储它的返回值,并将存储的值与您需要的字符(即'a''b')进行比较:

int GetKeyDown() {
    if(kbhit())
    {
        return getch();
    }
    return -1;
}

while(1)
{
    int keyDown = GetKeyDown();
    if(keyDown == 'a')
    {
        printf("HELLO\n");
    }
    if(keyDown == 'b')
    {
        printf("HI\n");
    }
    Sleep(1000 / 30);
}
于 2014-01-12T12:00:42.520 回答
0

因为第一个 IF 消耗了正在按下的键。更改您的代码,以便读取一次密钥:

int main(int argc, char** argv) {
    char key;

    while(1)
    {
        if (khbit())
            key = getch();
        else
            key = 0;

        if(key == 'a')
        {
            printf("HELLO\n");
        }
        if(key == 'b')
        {
            printf("HI\n");
        }
        Sleep(1000 / 30);
    }
    return (EXIT_SUCCESS);
}
于 2014-01-12T12:01:38.753 回答