从模块函数调用主文件对象报错NameError：button_w未定义问题排查求助

Let's break down why you're hitting that NameError and fix it properly.

The Root Cause #

Your use of global button_w in the module doesn't work because global only refers to the global namespace of the module itself, not the main script's global variables. The button_w you defined in main.py lives in the main script's scope, which the module can't reach through a simple global declaration.

Practical Solutions #

1. Pass the Button Object as a Parameter (Simplest Fix)

This is the most straightforward approach—just pass the button directly to your module functions when binding the events.

Update the event bindings in main.py:

root.bind('<KeyPress-w>', lambda event: w_down(event, button_w))
root.bind('<KeyRelease-w>', lambda event: w_up(event, button_w))

Then modify the functions in your module to accept the button as an argument:

from RemoteCarActions import *

def w_down(event, btn):
    btn.config(bg='yellow')
    w_gedrueckt()

def w_up(event, btn):
    btn.config(bg='lightgrey')
    # Add your w_up logic here

2. Encapsulate Your App in a Class (Best for Scalability)

Since you mentioned you'll have other objects like car later, wrapping your entire app in a class will make managing shared objects much cleaner.

Refactor main.py into a class:

from tkinter import *
from tkinter import ttk
from PIL import Image, ImageTk
from buttonActions import *
from Car import car

class RemoteCarApp:
    def __init__(self, root):
        self.root = root
        self.root.title("RemoteCar")
        self.root.state('zoomed')
        
        # Configure grid layout
        self.root.rowconfigure(0, weight=1)
        self.root.rowconfigure(1, weight=1)
        self.root.columnconfigure(0, weight=1)
        self.root.columnconfigure(1, weight=1)
        
        # Initialize car object
        self.auto = car()
        
        # Create UI frames
        self.stream = ttk.Frame(root, borderwidth=5, relief="ridge")
        self.stream.grid(row=0, column=0, rowspan=3, columnspan=2)
        
        pil_image = Image.open("Bild.jpg")
        self.tk_image = ImageTk.PhotoImage(pil_image)
        self.imgLabel = Label(self.stream, image=self.tk_image)
        self.imgLabel.pack()
        
        self.keys = ttk.Frame(root, borderwidth=5, relief="ridge")
        self.keys.grid(row=4, column=0)
        
        # Create buttons
        buttonWidth = 8
        buttonHight = 4
        self.button_w = Button(self.keys, text='W', width=buttonWidth, height=buttonHight, bg='lightgrey', command={})
        self.button_w.grid(row=0, column=1)
        
        # Bind events, passing the app instance
        self.root.bind('<KeyPress-w>', lambda event: w_down(event, self))
        self.root.bind('<KeyRelease-w>', lambda event: w_up(event, self))

if __name__ == "__main__":
    root = Tk()
    app = RemoteCarApp(root)
    root.mainloop()

Then update your module functions to use the app instance, giving you access to both the button and the car object:

from RemoteCarActions import *

def w_down(event, app):
    app.button_w.config(bg='yellow')
    app.auto.some_car_method()  # Access your car object's methods here
    w_gedrueckt()

def w_up(event, app):
    app.button_w.config(bg='lightgrey')
    # Add w_up logic, including car actions if needed

3. Wrap Module Actions in a Class (Tight Coupling for Focused Logic)

If all your button actions are tied directly to this app, you can turn your module into a class that holds references to the objects it needs:

Modify buttonActions.py:

from RemoteCarActions import *

class ButtonActions:
    def __init__(self, button_w, car_obj):
        self.button_w = button_w
        self.car = car_obj
    
    def w_down(self, event):
        self.button_w.config(bg='yellow')
        self.car.some_car_method()
        w_gedrueckt()
    
    def w_up(self, event):
        self.button_w.config(bg='lightgrey')
        # Add w_up logic

Then in main.py, initialize this class with your objects:

from buttonActions import ButtonActions
# ... (after creating button_w and auto)
button_actions = ButtonActions(button_w, auto)
root.bind('<KeyPress-w>', button_actions.w_down)
root.bind('<KeyRelease-w>', button_actions.w_up)

Key Takeaway #

Avoid trying to access the main script's global variables from modules—it creates tight coupling and makes your code harder to maintain. Using parameter passing or class-based encapsulation will keep your code organized, especially as you add more objects like your car later.

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