Rush Hour游戏单元测试中car坐标不更新的根源与修复方案-Python教程-PHP中文网

Rush Hour游戏单元测试中car坐标不更新的根源与修复方案

单元测试中move_car函数无法更新汽车坐标，是因为测试数据TEST_GAME_GAME['cars']被定义为不可变元组（tuple），而move_car内部试图通过赋值修改其元素——这在Python中静默失败，导致状态未变更。

单元测试中`move_car`函数无法更新汽车坐标，是因为测试数据`test_game_game['cars']`被定义为不可变元组（tuple），而`move_car`内部试图通过赋值修改其元素——这在python中静默失败，导致状态未变更。

问题核心在于数据结构的可变性（mutability）差异：虽然你在测试中使用了 deepcopy(TEST_GAME_GAME)，但 deepcopy 仅递归复制对象内容，不会改变原始容器的类型。观察你的测试常量定义：

TEST_GAME_GAME = {
    'width': 6,
    'height': 6,
    'max_moves': 40,
    'cars': (  # ← 注意：这里是 tuple，不是 list！
        [(0, 2), 'h', 2],
        [(2, 0), 'v', 3],
        # ... 其他 car 条目
    )
}

move_car 函数中关键逻辑如下（节选）：

cars[car_index] = ((x, y), orientation, size)  # ← 尝试对 cars 进行索引赋值

当 cars 是 tuple 时，此操作会抛出 TypeError: 'tuple' object does not support item assignment —— 但在你的测试中该异常可能被意外捕获、忽略或未触发断言，导致函数看似“成功”返回 True，实则坐标未更新。而主程序中 parse_game() 返回的 game['cars'] 是 list（由 append() 构建），因此赋值正常生效。

✅ 修复方法：将测试数据中的 cars 改为 list

TEST_GAME_GAME = {
    'width': 6,
    'height': 6,
    'max_moves': 40,
    'cars': [  # ← 改为方括号：list 类型
        [(0, 2), 'h', 2],  # Car A
        [(2, 0), 'v', 3],  # Car B ← 现在可被 move_car 安全修改
        [(3, 0), 'h', 3],  # Car C
        [(0, 3), 'v', 2],  # Car D
        [(3, 3), 'h', 2],  # Car E
        [(5, 3), 'v', 3],  # Car F
        [(4, 4), 'v', 2],  # Car G
        [(1, 5), 'h', 3]   # Car H
    ]
}

⚠️ 额外建议：增强健壮性与可观测性

在 move_car 开头添加类型检查，避免静默失败：

if not isinstance(game['cars'], list):
    raise TypeError(f"Expected game['cars'] to be a list, got {type(game['cars']).__name__}")

在单元测试中验证返回值，确保移动逻辑真正执行：

def test_move_car(self):
    game = deepcopy(TEST_GAME_GAME)
    # 断言移动前状态
    self.assertTupleEqual(game['cars'][1][0], (2, 0))
    # 执行移动并检查返回值
    result = move_car(game, 1, 'DOWN')
    self.assertTrue(result, "move_car should return True on valid move")
    # 断言移动后状态
    self.assertTupleEqual(game['cars'][1][0], (2, 1))