C++井字棋核心函数异常求助：Validation与DetermineWinner失效

一、Validation函数修复（阻止重复落子） #

核心问题： #

1. 条件判断逻辑错误：用位或运算符|代替逻辑或||，'X' | 'O'是二进制运算，结果为127，永远不等于棋盘上的X/O/*，导致验证完全失效。
2. 参数传递无效：采用值传递，函数内修改的row/column无法同步到main函数，即使重新输入坐标，PlayerAction仍会使用原无效位置。
3. 缺少边界校验：未检查row和column是否在0-2范围内，输入非法值会触发数组越界。
4. 返回值无意义：返回double类型且用逗号表达式返回column，完全冗余，改用void即可。

修改后的代码： #

void TicTacToe::Validation(int &row, int &column) // 引用传递参数，同步修改main中的变量
{
    // 循环校验坐标合法且未被占用
    while (row < 0 || row >= 3 || column < 0 || column >= 3 || 
           board[row][column] == 'X' || board[row][column] == 'O')
    {
        cout << "请输入合法的空位（行和列范围0-2）：" << endl;
        cin >> row >> column; // 正确读取两个坐标，弃用逗号运算符
    }
}

二、DetermineWinner函数修复（终止游戏） #

核心问题： #

1. 参数传递无效：采用值传递，函数内修改的winner无法同步到main函数，导致win变量始终为0，循环无法终止。
2. 获胜判断漏洞：未校验获胜的三个格子是否为非空（即不是初始的*），初始状态下三个*相等会错误触发获胜判定。

修改后的代码： #

int TicTacToe::DetermineWinner() // 无需传入参数，直接返回结果
{
    // 检查行获胜
    for (int i = 0; i < 3; i++)
    {
        if (board[i][0] != '*' && board[i][0] == board[i][1] && board[i][1] == board[i][2])
        {
            return board[i][0] == 'X' ? 1 : 2;
        }
    }
    // 检查列获胜
    for (int i = 0; i < 3; i++)
    {
        if (board[0][i] != '*' && board[0][i] == board[1][i] && board[1][i] == board[2][i])
        {
            return board[0][i] == 'X' ? 1 : 2;
        }
    }
    // 检查对角线获胜
    if (board[0][0] != '*' && board[0][0] == board[1][1] && board[1][1] == board[2][2])
    {
        return board[0][0] == 'X' ? 1 : 2;
    }
    if (board[0][2] != '*' && board[0][2] == board[1][1] && board[1][1] == board[2][0])
    {
        return board[0][2] == 'X' ? 1 : 2;
    }
    // 检查平局
    for (int i = 0; i < 3; i++)
    {
        for (int j = 0; j < 3; j++)
        {
            if (board[i][j] == '*')
            {
                return 0; // 还有空位，游戏继续
            }
        }
    }
    return -1; // 平局
}

三、main函数对应调整 #

1. 调用Validation时直接传变量（因用了引用，会同步修改main中的值）：ttt.Validation(row, column);
2. 接收DetermineWinner的返回值，更新win变量：win = ttt.DetermineWinner();
3. 修正输出字符串的换行错误：

// 原错误写法：cout << "Player 1 is the winner!
";
// 修改为：
cout << "玩家1获胜！" << endl;

其他小修复 #

• 构造函数中的cout << endl;会在初始化时输出空行，可移除避免冗余输出。
• PlayerAction函数可保留现有逻辑，因Validation已完成合法性校验。

修改后，重复落子的验证会生效，获胜或平局时win变量会被正确更新，循环会终止，游戏逻辑恢复正常。

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