使用exec()创建tkinter按钮时触发NameError: name 'self' is not defined

问题场景 #

为克里比奇牌程序制作GUI时，需要开局展开52张牌供玩家选择切牌位置。为避免手动编写52个Frame和Button的重复代码，采用for循环结合exec()动态创建组件，但点击按钮时触发NameError: name 'self' is not defined，而手动创建少量按钮时可正常运行。

原代码：

def cut(self, i):
    self.select_card_frame.place_forget()
    for k in range(52):
        exec(f'self.cut_frame_{k}.place_forget()')
    return i


def start_cut(self):
    global cut_image
    cut_image = self.resize_card('cards/back_of_card.png')
    self.select_card_frame = Frame(self.window)
    self.select_card_label = Label(self.select_card_frame, text='Select Card to Cut', font=20)
    self.select_card_label.pack()
    self.select_card_frame.place(x=450, y=50)
    for i in range(52):
        exec(f'self.cut_frame_{i} = Frame(self.window)')
        exec(f'self.cut_label_{i} = Button(self.cut_frame_{i}, image=cut_image, command=lambda: self.cut({i}))')
        exec(f"self.cut_label_{i}.pack(side='left')")
        exec(f'self.cut_frame_{i}.place(x={i*20}, y=100)')

错误原因 #

exec()执行字符串代码时，会在当前局部作用域动态生成逻辑，但lambda表达式中的self并未被正确绑定到当前实例的上下文。更核心的问题是：用exec()动态创建带编号的实例属性是完全没必要的反模式——这种做法不仅容易引发作用域问题，还会让代码变得难以调试和维护。

优化方案：抛弃exec，用容器管理组件 #

Python中可以用列表或字典批量存储Frame和Button，既规避exec的坑，又能简化后续组件操作，完全不用写冗余的重复代码。

修改后的代码示例：

def cut(self, i):
    self.select_card_frame.place_forget()
    # 遍历存储的所有frame，直接隐藏
    for frame in self.cut_frames:
        frame.place_forget()
    return i


def start_cut(self):
    # 初始化容器存储所有frame和button
    self.cut_frames = []
    self.cut_buttons = []
    
    # 改用实例属性存储图片，避免全局变量
    self.cut_image = self.resize_card('cards/back_of_card.png')
    self.select_card_frame = Frame(self.window)
    self.select_card_label = Label(self.select_card_frame, text='Select Card to Cut', font=20)
    self.select_card_label.pack()
    self.select_card_frame.place(x=450, y=50)
    
    for i in range(52):
        # 创建frame并添加到列表
        frame = Frame(self.window)
        self.cut_frames.append(frame)
        # 创建button，用lambda i=i捕获当前循环的i值（避免延迟绑定问题）
        btn = Button(frame, image=self.cut_image, command=lambda i=i: self.cut(i))
        btn.pack(side='left')
        self.cut_buttons.append(btn)
        # 布局frame
        frame.place(x=i*20, y=100)

关键改进点： #

• 用self.cut_frames列表存储所有动态创建的Frame，后续操作直接遍历列表即可，无需再用exec拼接属性名。
• lambda表达式中用i=i显式捕获当前循环的i值，避免循环变量延迟绑定导致所有按钮都使用最后一个i的问题（原代码即使没有self的错误，也会存在这个隐患）。
• 去掉全局变量cut_image，改用实例属性self.cut_image，符合面向对象的代码规范。

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