C++骰子游戏输入合法y/n时validate校验函数仍持续运行

核心死循环根因 #

你validate函数中的循环条件存在布尔逻辑错误，这是循环无论输入什么都无法退出的直接原因：

while (ch != 'n' || ch != 'y')

根据逻辑运算规则，这个表达式对任意字符ch的计算结果永远为真：

• 当ch为合规输入'y'：ch != 'n'成立，或运算结果为真，循环持续运行
• 当ch为合规输入'n'：ch != 'y'成立，或运算结果为真，循环持续运行
• 当ch为任意无效输入：两个不等判断均成立，循环同样持续运行

要实现“输入非y/n时要求重输”的目标，正确的判断条件是同时不满足y、n输入，需把逻辑或替换为逻辑与：

while (ch != 'n' && ch != 'y')

其余影响程序正常运行的逻辑问题 #

修正循环条件后，当前代码还存在几处会导致功能不符合预期的问题，按影响程度排序：

• 程序流程缺失关键调用
  driver函数目前仅调用prompt()和display()，从未触发实现投掷、积分计算的rolls()函数，即使用户输入合规的y，程序也不会执行任何投掷逻辑。
  需调整流程为：完成欢迎提示与输入校验 → 根据用户输入触发投掷逻辑 → 投掷结束后输出最终统计结果。
• 非标准的输入缓冲清理实现
  你使用的fseek(stdin, 0, SEEK_END)是C风格的终端操作，跨平台兼容性差，不属于C标准流操作范畴。
  如需清理输入残留（例如用户输入test时，cin >> ch仅读取第一个t，剩余est会留在缓冲区干扰后续读取），可使用C标准流接口实现：

  cin.ignore(numeric_limits<streamsize>::max(), '\n');
  

  使用时需在代码头部引入头文件#include <limits>。
• 骰子点数仅生成一次
  类构造函数中仅在程序启动时生成一次固定的随机骰子点数，后续投掷循环中从未重新生成随机值，就算连续多次投掷，使用的也是第一次的固定点数，不符合随机投掷的规则。需把随机点数生成逻辑移到每次投掷的步骤开头，保证每轮结果独立随机。
• 输出语句笔误
  case 12的输出语句中多写了左移符号<<，会导致输出格式混乱：

  // 错误写法
  cout << "Sorry, you rolled a << " << dice << ", you lose.";
  // 正确写法
  cout << "Sorry, you rolled a " << dice << ", you lose.";
  

修正后的核心代码参考 #

#include <iostream>
#include <cstdlib>
#include <ctime>
#include <limits>
using namespace std;

class Game
{
private:
  int die1;
  int die2;
  int dice;
  int totalPoints;
  int totalRolls;
  char ch;

  void prompt();
  void display();
  void validate();
  void rolls();

public:
  void driver();
  Game();
};

// 核心validate函数修正参考
void Game::validate()
{
  while (ch != 'n' && ch != 'y')
  {
      cin.clear();
      // 标准方式清理输入缓冲区残留
      cin.ignore(numeric_limits<streamsize>::max(), '\n');
      cout << "Invalid response. Please input (y = yes, n = no) ";
      cin >> ch;
  }
  // 校验通过后清理一次残留，避免后续读取受影响
  cin.ignore(numeric_limits<streamsize>::max(), '\n');
}

// 核心driver函数流程参考
void Game::driver()
{
  prompt(); // 内部完成输入与校验
  rolls();  // 执行投掷逻辑
  display(); // 输出最终统计
}

内容的提问来源于stack exchange，提问作者Jonathan Gonzalez