Python列表随机选元素并全局移除，重复选取问题求助

需求：从指定列表随机选取人员分配任务，选中后全局移除避免重复，但当前代码仍出现同一人员被重复选中的情况，移除操作未全局生效。

错误原因分析 #

你的代码核心问题出在choice_restul_wkd和choice_toti函数的移除逻辑上：

• 当从restul + wkd（或restul + wkd + cls）中随机选择元素后，尝试同时从两个/三个列表移除该元素，但元素仅存在于其中一个列表，必然触发ValueError。
• 而except块中仅执行nr += 1，未做实际移除操作——导致选中的元素根本没从原列表删除，后续仍会被重复选中。

修复方案 #

修改两个函数的移除逻辑，先判断元素所属列表，再从对应列表中移除；同时优化代码规范与可读性：

from random import choice

wkd = ["Razvan", "Vali", "Stefan", "Dontu","Abel", "David P.", "Beni N."]
cls = ["Angelo", "Dragos", "Beni D.", "Andy", "Vlad T.", "Dani", "Ruben", "George", "Sebi Br."]
restul = ["Claudiu", "Cipri", "David", "Rares N.", "Vlad P.", "Marian", "Seba", "Emi", "Darius", "Stefano", "Iosif", "Radu", "Rares", "Robert", "Alberto B.", "David B.", "Robert T."," James", ]

Duminica = "Duminica"
Luni = "Luni"
Marti = "Marti"
Miercuri = "Miercuri"
Joi = "Joi"
Vineri = "Vineri"

def choice_restul():
    global restul
    selected = choice(restul)
    restul.remove(selected)
    return selected

def choice_restul_wkd():
    global wkd, restul
    combined = restul + wkd
    selected = choice(combined)
    # 判断元素所在列表并移除
    if selected in restul:
        restul.remove(selected)
    elif selected in wkd:
        wkd.remove(selected)
    return selected

def choice_toti():
    global restul, wkd, cls
    combined = restul + wkd + cls
    selected = choice(combined)
    # 判断元素所在列表并移除
    if selected in restul:
        restul.remove(selected)
    elif selected in wkd:
        wkd.remove(selected)
    elif selected in cls:
        cls.remove(selected)
    return selected

class servicii:
    @staticmethod
    def cantina():
        nr_sapt = 1
        while nr_sapt < 5:
            with open(f'cantina{nr_sapt}.txt', 'w') as f:
                f.write("Cantina\n")
                f.write(f"{Duminica}-{choice_restul()}\n")
                f.write(f"{Luni}-{choice_restul_wkd()}\n")
                f.write(f"{Marti}-{choice_restul_wkd()}\n")
                f.write(f"{Miercuri}-{choice_restul_wkd()}\n")
                f.write(f"{Joi}-{choice_restul_wkd()}\n")
                f.write(f"{Vineri}-{choice_restul()}\n")
            nr_sapt += 1
        return "Cantina"

    @staticmethod
    def palier():
        with open('rezultat.txt', 'w') as f:
            f.write("\nPalier\n")
            f.write(f"{Duminica}-{choice_restul()}\n")
            f.write(f"{Luni}-{choice_restul_wkd()}\n")
            f.write(f"{Marti}-{choice_restul_wkd()}\n")
            f.write(f"{Miercuri}-{choice_restul_wkd()}\n")
            f.write(f"{Joi}-{choice_restul_wkd()}\n")
            f.write(f"{Vineri}-{choice_restul()}\n")
        return "Palier"

    @staticmethod
    def curte():
        return "curte"

    @staticmethod
    def poarta():
        return "poarta"

print(servicii.cantina())

关键修改点 #

1. 修复移除逻辑：通过in判断元素所属列表，仅从存在该元素的列表中移除，避免无效的ValueError。
2. 添加@staticmethod装饰器：符合Python类方法规范，避免调用时传入类对象引发错误。
3. 简化字符串拼接：使用f-string提升代码可读性。
4. 清理冗余代码：移除未使用的choices、shuffle导入和无用的nr变量。

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