基于Python与Tkinter的世界杯抽奖系统开发问题：解决随机选择后重复选中及撤销功能异常

Hey there! Let's work through the issues in your code to get that sweepstake running smoothly. Here's what's going wrong and how to fix it:

The Core Problems #

• Reinitialized Lists: Every time you click "Select Player" or "Select Team", you're creating a new players/teams list from scratch. That means any items you popped earlier get added back immediately—so duplicates are inevitable.
• Broken Team Selection: In select_team(), you're doing teams = teams.pop(...) which replaces your list with a single string. Next time you click the button, you'll get an error because you can't pop() from a string.
• Incomplete Clear Function: Right now, you're just hiding the labels, but you aren't putting the selected player/team back into their respective lists. Plus, you don't track which items were last selected to undo.

Fixed Code #

from tkinter import *
import random

root = Tk()
root.title('World Cup 2022 - Sweepstake')
root.geometry("600x800")

# Initialize global lists and tracking variables
players = ["Matt", "Steve", "Dave", "Pete"]
teams = ["Qatar", "Ecuador", "Senegal", "Netherlands"]
last_player = None
last_team = None
player_label = None
team_label = None

def select_player():
    global player_label, last_player
    # Clear existing player label if present
    if player_label:
        player_label.pack_forget()
    
    if not players:
        player_label = Label(root, text="No players left!", font=("Arial", 16, 'bold'))
        player_label.pack(pady=30)
        return
    
    chosen_player = players.pop(random.randrange(len(players)))
    last_player = chosen_player
    player_label = Label(root, text=chosen_player, font=("Arial", 16, 'bold'))
    player_label.pack(pady=30)

def select_team():
    global team_label, last_team
    # Clear existing team label if present
    if team_label:
        team_label.pack_forget()
    
    if not teams:
        team_label = Label(root, text="No teams left!", font=("Arial", 22, 'bold'))
        team_label.pack(pady=40)
        return
    
    # Fix: Don't overwrite the teams list! Use a separate variable
    chosen_team = teams.pop(random.randrange(len(teams)))
    last_team = chosen_team
    team_label = Label(root, text=chosen_team, font=("Arial", 22, 'bold'))
    team_label.pack(pady=40)

def clear_label():
    global player_label, team_label, last_player, last_team
    # Hide labels if they exist
    if player_label:
        player_label.pack_forget()
        player_label = None
    if team_label:
        team_label.pack_forget()
        team_label = None
    
    # Put last selected items back into lists
    if last_player:
        players.append(last_player)
        last_player = None
    if last_team:
        teams.append(last_team)
        last_team = None

top_label = Label(root, text="World Cup Sweepstake", font=("Arial", 22, 'bold'))
top_label.pack(pady=20)

select_player_button = Button(root, text="Select Player", font=("Arial", 14), command=select_player)
select_player_button.pack(pady=15)

select_team_button = Button(root, text="Select Team", font=("Arial", 14), command=select_team)
select_team_button.pack(pady=15)

delete_button = Button(root, text="Clear", command=clear_label)
delete_button.pack(pady=20)

root.mainloop()

Key Improvements #

1. Persistent Lists: players and teams are now global variables initialized once, so their state stays consistent between button clicks.
2. Track Last Selections: last_player and last_team store the most recent picks, so we can add them back when clearing.
3. Fixed Team Selection: We now use chosen_team instead of overwriting the teams list—no more string vs. list errors.
4. Empty List Handling: If all players/teams are picked, the app shows a friendly message instead of crashing.
5. Robust Clear Function: It hides labels, resets the label variables, and returns the last selected items to their lists.

Now your app will properly avoid reselecting already picked players/teams, and the clear button will fully undo the last selection!

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