Python游戏引擎开发报错：NameError未定义topDownPlayerControls

首先来看你遇到的NameError: name 'topDownPlayerControls' is not defined报错，这是因为你对类方法的调用方式不对，再加上代码里还有变量作用域、重复初始化等问题，我们一步步来解决：

直接报错原因分析 #

1. 初始版本：topDownPlayerControls是playerMovement类内部的方法，但你在main.py里直接全局调用topDownPlayerControls()，Python找不到这个全局函数，自然报错。
2. 更新后版本：你虽然创建了player = playerMovement()实例，但调用时写的是player.topDownPlayerControls——这只是引用方法对象，没有实际执行调用（缺少()），而且就算加上括号，方法里的event、x、y等变量都是未定义的，因为它们不在方法的作用域内。

还有这些隐藏问题需要修复 #

• 变量作用域问题：playerSelection.py里的方法直接使用event、x、y这些外部变量，类方法无法访问全局或外部作用域的变量（除非显式传递或设置为实例属性）。
• 重复初始化Pygame窗口：在playerSelection.py里又调用了pygame.display.set_mode()，这会和main.py里的窗口冲突，导致异常。
• 未初始化的变量：x_change、y_change在方法里没有初始值，第一次运行会报错。
• 类方法的定义问题：初始版本里的类方法没有使用self（实例方法）或@staticmethod装饰器，不符合Python类的规范。

修复后的代码实现 #

优化后的playerSelection.py #

我们把玩家的状态（位置、速度）封装成实例属性，让方法通过self访问，同时接收event参数来处理输入：

import pygame

class PlayerMovement:
    def __init__(self, display, x, y, img_path='Player.png'):
        self.display = display  # 传入游戏窗口对象
        self.x = x
        self.y = y
        self.x_change = 0
        self.y_change = 0
        self.player_img = pygame.image.load(img_path)

    def display_player(self):
        # 直接使用实例属性绘制玩家
        self.display.blit(self.player_img, (self.x, self.y))

    def handle_controls(self, event):
        # 接收event参数处理按键事件
        if event.type == pygame.KEYDOWN:
            if event.key == pygame.K_LEFT:
                self.x_change = -5
                print("Left")
            elif event.key == pygame.K_RIGHT:
                self.x_change = 5
                print("Right")
            if event.key == pygame.K_DOWN:
                self.y_change = 5
                print("Down")
            elif event.key == pygame.K_UP:
                self.y_change = -5
                print("Up")
        elif event.type == pygame.KEYUP:
            if event.key in (pygame.K_LEFT, pygame.K_RIGHT):
                self.x_change = 0
            if event.key in (pygame.K_DOWN, pygame.K_UP):
                self.y_change = 0

    def update_position(self):
        # 更新玩家位置
        self.x += self.x_change
        self.y += self.y_change

优化后的main.py #

把玩家实例化时传入窗口对象和初始位置，在事件循环里把event传递给玩家的控制方法：

import pygame
from playerSelection import PlayerMovement
import ctypes
from ctypes.wintypes import HWND, LPWSTR, UINT

# Window Size
display_w = 1366
display_h = 768

# Initialize and Set Display
pygame.init()
game_display = pygame.display.set_mode((display_w, display_h))
pygame.display.set_caption("The Joker Engine (Version Undecided) - By Harrison Court")
clock = pygame.time.Clock()

# Ctypes ID's
IDYES = 6
IDNO = 7

# Colours
black = (0, 0, 0)
white = (255, 255, 255)
red = (255, 0, 0)

# 初始化玩家，传入游戏窗口和初始位置
player = PlayerMovement(
    display=game_display,
    x=display_w * 0.45,
    y=display_h * 0.8
)

running = True
while running:
    game_display.fill(white)
    
    for event in pygame.event.get():
        if event.type == pygame.QUIT:
            running = False
        if event.type == pygame.KEYDOWN:
            # 退出逻辑
            if event.key == pygame.K_ESCAPE:
                quit_window = ctypes.windll.user32.MessageBoxW(
                    0, 
                    "Are you sure you want to quit?", 
                    "Warning!", 
                    4
                )
                if quit_window == IDYES:
                    running = False
        # 把事件传递给玩家处理控制
        player.handle_controls(event)
    
    # 更新玩家位置并绘制
    player.update_position()
    player.display_player()
    
    pygame.display.update()
    clock.tick(60)

pygame.quit()

更优实现的核心思路 #

1. 封装性：把玩家的所有状态（位置、速度、图像）和行为（控制、绘制、更新）都封装在PlayerMovement类里，避免全局变量，代码更清晰。
2. 依赖注入：把游戏窗口对象传入玩家类，避免在子模块里重复初始化窗口，降低耦合度。
3. 事件传递：在主循环里把每个event传递给玩家的控制方法，让玩家自己处理输入，符合单一职责原则。
4. 规范的类方法：使用__init__初始化实例属性，方法通过self访问实例数据，符合Python面向对象的规范。

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