Python多线程问题：ASCII版Pong中对象方法交互异常

嘿，我一眼就看出问题出在哪了——线程竞态条件！你用两个线程分别处理球和球拍逻辑，但它们肯定在共享游戏状态（比如球的位置、球拍位置），而这些共享资源的访问没有正确同步，导致多个线程同时修改/读取变量时出现数据混乱。不输入时球拍线程基本没动作，冲突少；一输入就频繁修改球拍位置，直接触发了严重的竞态，球的行为自然就失控了。

下面给你具体的修复方案：

1. 给所有共享资源的访问加锁 #

你已经定义了lock = threading.Lock()，但关键是不管读还是写共享变量，都要在锁的保护下进行。比如球拍的移动、球的位置更新、碰撞检测，甚至屏幕刷新时读取状态，都得用锁包裹。

修改球拍类的移动方法 #

class Stick(object):
    def __init__(self, x, y, screen_height):
        self.x = x
        self.y = y
        self.screen_height = screen_height

    def move_up(self):
        # 用with上下文管理器自动释放锁，比手动acquire/release更安全
        with lock:
            if self.y > 0:
                self.y -= 1

    def move_down(self):
        with lock:
            if self.y < self.screen_height - 3:  # 假设球拍占3行高度
                self.y += 1

修改球的线程逻辑 #

球的位置更新、碰撞检测必须在锁内完成，避免和球拍线程的修改冲突：

def ball_update_thread(ball, stick_left, stick_right, screen_width, screen_height):
    while True:
        with lock:
            # 更新球的位置
            ball.x += ball.dx
            ball.y += ball.dy

            # 碰撞顶部/底部边界
            if ball.y <= 0 or ball.y >= screen_height - 1:
                ball.dy *= -1

            # 碰撞球拍
            if (ball.x == stick_left.x + 1 and 
                stick_left.y <= ball.y <= stick_left.y + 2):
                ball.dx *= -1
            elif (ball.x == stick_right.x - 1 and 
                  stick_right.y <= ball.y <= stick_right.y + 2):
                ball.dx *= -1

            # 球出界逻辑（这里可以加得分判断）
            if ball.x < 0 or ball.x >= screen_width:
                ball.reset_position()  # 重置球的位置

        # 把sleep放在锁外面，别占用锁的时间，让其他线程能及时获取锁
        time.sleep(0.05)

修改输入线程逻辑 #

输入处理时修改球拍位置，同样要加锁：

def input_thread(stick):
    while True:
        key = getch()
        with lock:
            if key.lower() == 'w':
                stick.move_up()
            elif key.lower() == 's':
                stick.move_down()
            elif key.lower() == 'q':
                # 安全退出所有线程
                os._exit(0)

2. 额外注意事项 #

• 别用递归代替循环：你设置了sys.setrecursionlimit(100000000)，但线程里用递归很容易栈溢出，用while True循环才是正确的做法。
• 屏幕刷新也要加锁：当你打印ASCII画面时，需要读取球和球拍的位置，这时候也要加锁，否则可能读到一半被修改的状态，导致画面错乱：

  def render_screen(ball, stick_left, stick_right, screen_width, screen_height):
      with lock:
          os.system('cls' if os.name == 'nt' else 'clear')
          # 构建屏幕内容的逻辑，比如用二维列表生成画面
          screen = [[' ' for _ in range(screen_width)] for _ in range(screen_height)]
          # 绘制球拍
          for i in range(3):
              screen[stick_left.y + i][stick_left.x] = '|'
              screen[stick_right.y + i][stick_right.x] = '|'
          # 绘制球
          screen[ball.y][ball.x] = 'O'
          # 打印屏幕
          for row in screen:
              print(''.join(row))
  

• 控制线程的执行频率：每个线程的sleep时间要合理，避免某个线程占用过多CPU时间，导致其他线程响应不及时。

这样修改后，你再输入控制球拍时，线程间的共享资源访问就同步了，球的行为应该就能恢复正常啦！

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