getchar()输入后需再次按ENTER才继续的问题排查

问题描述 #

使用getchar()处理用户输入时遇到程序卡顿等待输入的异常：

• 程序中共有4处用循环清空输入缓冲区的逻辑，但仅2处while循环会出现卡顿，需要按两次ENTER才能继续
• 对比正常工作的循环，卡顿的循环多了else if分支，但移除该分支后问题仍未解决
• 输入非'y'/'Y'内容时，"invalid input"提示功能正常；但输入'y'/'Y'并按ENTER后，程序仅换行无响应，再次输入无效内容会触发错误提示，输入正确内容仍需再按一次ENTER才能继续执行

非工作实例代码 #

printf("Play a game? (Y/N): ");
while (!is_valid) {
    user_choice = getchar();
    char ch;
    while ((ch = getchar()) != '\n' && ch != EOF) {} // clear the buffer

    if (user_choice == 'y' || user_choice == 'Y') {
        is_valid = true;
        play_game();
    }
    else if (user_choice == 'n' || user_choice == 'N') break;
    else {
        printf("Invalid input. Try again.\n");
    }
} 

工作实例代码 #

printf("Roll dice ('R')\n");
while (!is_valid) {
    first_roll = getchar();     
    char ch;
    while ((ch = getchar()) != '\n' && ch != EOF) {} // clear the buffer

    if (first_roll == 'r' || first_roll == 'R') {
        is_valid = true;
    }
    else {
        printf("Invalid input. Try again.\n");
    }
} 

错误原因分析 #

1. 多余的getchar()调用：在play_game()函数开头存在一行无意义的getchar();，这会直接吃掉输入缓冲区中的一个字符（通常是用户输入'y'后按下的换行符），导致后续输入处理逻辑错位，必须额外输入才能填补这个被消耗的字符，表现为需要按两次ENTER。
2. 递归调用的上下文冲突：当用户选择继续游戏时，play_game()会递归调用自身，递归返回后，原循环的输入处理与递归中的缓冲区清理逻辑产生冲突，进一步加剧了输入等待的异常。
3. 缓冲区清理逻辑的一致性问题：虽然四处清理逻辑代码一致，但由于递归和多余getchar()的影响，导致部分场景下缓冲区状态异常，触发卡顿。

修复方案 #

1. 移除多余的getchar() #

删除play_game()函数开头的getchar();语句，避免无端消耗输入字符：

bool play_game(void) {
    bool result = false;
    bool is_valid = false;
    bool game_has_ended = false;
    char first_roll;
    char user_choice;
    int roll_result;
    
    // 移除这一行：getchar();

    // 后续代码保持不变
}

2. 替换递归为循环控制 #

将play_game()内部的递归调用改为在main()中用循环控制游戏重复，避免递归导致的输入上下文混乱：
修改后的main()函数：

int main() {
    int roll = 0;
    char user_choice;
    bool is_valid = false;
    bool play_again = true;

    while (play_again) {
        // 游戏开始选择
        printf("Play a game? (Y/N): ");
        is_valid = false;
        while (!is_valid) {
            user_choice = getchar();
            clear_input_buffer(); // 调用通用清理函数

            if (user_choice == 'y' || user_choice == 'Y') {
                is_valid = true;
                play_again = play_game();
            }
            else if (user_choice == 'n' || user_choice == 'N') {
                is_valid = true;
                play_again = false;
            }
            else {
                printf("Invalid input. Try again.\n");
            }
        } 
    }
    
    return 0;
}

同时修改play_game()的结尾部分，移除递归，改为返回是否继续的状态：

// 替换play_game()结尾的递归逻辑
is_valid = false;
printf("Play another game? (Y/N): ");
while (!is_valid) {
    user_choice = getchar();
    clear_input_buffer();
    if (user_choice == 'y' || user_choice == 'Y') {
        is_valid = true;
        return true;
    }
    else if (user_choice == 'n' || user_choice == 'N') {
        is_valid = true;
        return false;
    }
    else {
        printf("Invalid input. Try again.\n");
    }
}

3. 封装通用的缓冲区清理函数 #

创建一个通用函数统一处理缓冲区清理，确保代码一致性：

void clear_input_buffer(void) {
    int ch;
    while ((ch = getchar()) != '\n' && ch != EOF) {}
}

在所有需要清理缓冲区的地方调用该函数，替换原来的重复代码。

额外优化点 #

• roll_dice()函数的实现有误：当前rand() % 12 + 1会生成1-12的随机数，但两个骰子的正确范围是2-12，且每个数的概率不符合实际骰子概率。正确实现应为：

int roll_dice(void) {
    int die1 = rand() % 6 + 1;
    int die2 = rand() % 6 + 1;
    return die1 + die2;
}

• 在程序开头添加srand(time(NULL));初始化随机数生成器，避免每次运行得到相同的随机序列。

内容的提问来源于stack exchange，提问作者Hexe Decima