如何修复C# .NET Framework控制台版吃豆人游戏移动异常问题

架构老林

2026-10-25

问题定位

核心逻辑错误：吃豆人精灵方向定义完全与移动逻辑错位
当前PacMan类的精灵定义存在方向反向问题：
```
public static string upSprite = "v";
public static string downSprite = "^";
public static string rightSprite = "<";
public static string leftSprite = ">";
```
字符尖的朝向完全和预期相反：向右移动时显示尖朝左的<、向上移动时显示尖朝下的v、向下移动时显示尖朝上的^、向左移动时显示尖朝右的>。由于初始位置的精灵是尖朝左的<，刚好匹配用户对“向左移动”的视觉认知，因此用户会误以为只有向左的操作正常，其余方向因为朝向与操作预期不符，表现为“逻辑异常”。
边界校验缺失：现有移动判断仅校验目标位置是否为墙#，未对数组索引的合法性做校验。一旦出现getPlayerPos识别失败返回默认值(0,0)的情况，执行上/左方向移动时会访问数组负索引，直接触发程序崩溃。
渲染逻辑冗余：四个并列的if判断没有分支互斥逻辑，按键不匹配时会走到分支外的逻辑，重复执行清屏、重绘操作，导致画面闪烁、位置识别异常。
初始化逻辑边界错误：墙层绘制的循环上限为y < yNum、x < xNum，遍历范围没有覆盖数组最后一行、最后一列，导致这两个边界位置未被设置为墙，玩家可移动到最边缘位置，极易触发索引越界。

修复方案

修正精灵定义，让尖的朝向与移动方向匹配：

public static string upSprite = "^"; // 尖朝上，对应向上移动
public static string downSprite = "v"; // 尖朝下，对应向下移动
public static string rightSprite = ">"; // 尖朝右，对应向右移动
public static string leftSprite = "<"; // 尖朝左，对应向左移动

修正墙层初始化循环的边界，确保所有边界位置都被设置为墙：将循环上限调整为y <= yNum、x <= xNum，覆盖数组全部索引范围。
所有移动逻辑增加双校验：既校验目标位置不是墙，也校验索引在数组合法范围内，避免越界崩溃。
调整位置替换逻辑：吃豆人离开原位置时，不要直接将原位置设为null，替换为空地字符，避免后续位置识别时出现null值匹配失败的问题。
采用互斥分支结构重构移动判断，仅在移动成功时执行清屏重绘，降低画面闪烁。

修正后的runGame方法参考实现：

public static void runGame()
{
    (int y, int x) playerPos = PacMan.getPlayerPos();
    ConsoleKeyInfo ki = Console.ReadKey(true);
    string remainSprite = " ";

    // 移动前临时清空原位置，避免旧精灵残留
    gameField.gameArr[playerPos.y, playerPos.x] = remainSprite;

    // 分支处理不同按键的移动逻辑，增加双校验
    if (ki.Key == ConsoleKey.LeftArrow && playerPos.x > 0 
        && gameField.gameArr[playerPos.y, playerPos.x - 1] != "#")
    {
        gameField.gameArr[playerPos.y, playerPos.x - 1] = PacMan.leftSprite;
        remainSprite = PacMan.leftSprite;
    }
    else if (ki.Key == ConsoleKey.RightArrow && playerPos.x < gameField.xNum 
        && gameField.gameArr[playerPos.y, playerPos.x + 1] != "#")
    {
        gameField.gameArr[playerPos.y, playerPos.x + 1] = PacMan.rightSprite;
        remainSprite = PacMan.rightSprite;
    }
    else if (ki.Key == ConsoleKey.UpArrow && playerPos.y > 0 
        && gameField.gameArr[playerPos.y - 1, playerPos.x] != "#")
    {
        gameField.gameArr[playerPos.y - 1, playerPos.x] = PacMan.upSprite;
        remainSprite = PacMan.upSprite;
    }
    else if (ki.Key == ConsoleKey.DownArrow && playerPos.y < gameField.yNum 
        && gameField.gameArr[playerPos.y + 1, playerPos.x] != "#")
    {
        gameField.gameArr[playerPos.y + 1, playerPos.x] = PacMan.downSprite;
        remainSprite = PacMan.downSprite;
    }

    // 移动无效时把精灵放回原位置，避免位置丢失
    gameField.gameArr[playerPos.y, playerPos.x] = remainSprite;

    // 仅在发生位置变更时重绘画面
    if (remainSprite != " ")
    {
        Console.Clear();
        gameField.drawField();
    }
}

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