C++ 控制台井字游戏：检查获胜条件

Question

游戏板存储为 2D 字符数组。玩家使用数字键盘在棋盘上移动光标，并使用 enter 键选择 - 光标的当前位置存储在两个整数中。

每次移动后，使用以下方法评估棋盘是否获胜。

void checkwin()
{
    //look along lines from current position
    int x = cursorPosX;
    int y = cursorPosY;
    int c = playerTurn ? 1 : 2; //which mark to look for

    for (int xAxis = 0; xAxis <= 2; xAxis++) //look along x axis
    {
        x = WrapValue(0, sizeof(squares[0]), x + 1);
        if (CheckPos(x, y) != c) //if we don't find the same mark, must not be a horizontal line, otherwise, break out.
        {
            x = cursorPosX; //reset x
            for (int yAxis = 0; yAxis <= 2; yAxis++) //look along y axis
            {
                y = WrapValue(0, sizeof(squares[0]), y + 1);
                if (CheckPos(x, y) != c) 
                {
                    y = cursorPosY;
                    //look for diagonal
                    for (int i = 0; i <= 2; i++ )
                    {
                        x = WrapValue(0, sizeof(squares[0]), x + 1);
                        y = WrapValue(0, sizeof(squares[0]), y + 1);
                        if (CheckPos(x, y) != c)
                        {
                            //failed everything, return
                            winConditions = -1;
                            return;
                        }
                    }
                    break;
                }
            }
            break;
        }
    }
    //if we make it out of the loops, we have a winner.
    winConditions = playerTurn ? 0 : 1;
}

我得到错误的结果 - 在不合适的情况下返回平局或获胜。我几乎可以肯定 x 和 y 在某些时候会得到错误的值并开始检查错误的位置。

Visual Studio 在进入 yAxis 循环后停止更新 x 和 y 上的手表 - 我不知道为什么，但它阻止了我跟踪这些值。我是否违反了某处范围界定的规则？这是我唯一使用 x 和 y 作为变量名的地方。

下面的相关换行方法。我的目标是始终能够通过添加来检查其他 2 个空格，无论我在棋盘的哪个位置

int WrapValue(int min, int max, int value)
{
    auto range = max - min;

    while (value >= max)
    {
        value -= range;
    }
    while (value < min)
    {
        value += range;
    }

    return value;
}

我很感激训练有素的眼睛告诉我我在这里做错了什么。非常感谢您的宝贵时间。

Answer 1

嵌套 for 循环是一个糟糕的主意。我通过将代码重构为多个单独的循环来解决这个问题，每个循环只做一件事，而不是通过彼此陷入更深层次的地狱。

for (int xAxis = 0; xAxis <= 2; xAxis++) //look along x axis
{
    x = WrapValue(0, sizeof(squares[0]), x + 1);
    if (CheckPos(x, y) != c) //if we don't find the same mark, must not be a horizontal line, otherwise, break out.
    {
        x = cursorPosX; //reset x
        break;
    }
    else if (xAxis == 2)
    {
        winConditions = playerTurn ? 0 : 1;
        return;
    }
}

for (int yAxis = 0; yAxis <= 2; yAxis++) //look along y axis
{
    y = WrapValue(0, sizeof(squares[0]), y + 1);
    if (CheckPos(x, y) != c)
    {
        y = cursorPosY;
        break;
    }
    else if (yAxis == 2)
    {
        winConditions = playerTurn ? 0 : 1;
        return;
    }
}
...ect

这违反了 DRY，但它确实按预期的方式工作，我相信我以后可以简化它。

虽然我不完全确定为什么以前的方法行不通，但我确实意识到这只是一个糟糕的设计。