C# WinForm随机分配唯一分类至标签重复问题求助

你的代码里的Fisher-Yates洗牌逻辑存在错误——每次循环只是从unshuffledCategories里随机挑一个元素加到新列表，但从未把选中的元素从原列表中移除，导致同一个元素可能被反复选中，这就是重复值出现的核心原因。

下面是两种可行的修复方式：

方式1：补全你的洗牌逻辑（移除已选中元素） #

在每次选中元素后，把它从原列表里删除，确保不会被再次选中：

public void button2_Click(object sender, EventArgs e)
{
    List<String> unshuffledCategories = new List<String> { "History", "Geography", "Music", "Animals", "Logos", "Flags", "Food", "games", "Sports" };
    List<String> shuffledCategories = new List<string>();

    Random r = new Random();

    var labels = new List<Label> { lblField1, lblField2, lblField3, lblField4, lblField5, lblField6, lblField7, lblField8, lblField9 };

    for (int n = unshuffledCategories.Count; n > 0; --n)
    {
        int k = r.Next(n);
        String temp = unshuffledCategories[k];
        shuffledCategories.Add(temp);
        unshuffledCategories.RemoveAt(k); // 关键步骤：移除已选中的元素
    }

    for (int i = 0; i < labels.Count; i++)
    {
        labels[i].Text = shuffledCategories[i];
    }
}

方式2：原地Fisher-Yates洗牌（更高效） #

标准的Fisher-Yates算法采用原地打乱列表的方式，不需要额外创建新列表，效率更高：

public void button2_Click(object sender, EventArgs e)
{
    List<String> categories = new List<String> { "History", "Geography", "Music", "Animals", "Logos", "Flags", "Food", "games", "Sports" };
    Random r = new Random();
    var labels = new List<Label> { lblField1, lblField2, lblField3, lblField4, lblField5, lblField6, lblField7, lblField8, lblField9 };

    // 原地执行Fisher-Yates洗牌
    for (int n = categories.Count - 1; n > 0; n--)
    {
        int k = r.Next(n + 1);
        // 交换当前元素与随机选中的元素
        var temp = categories[n];
        categories[n] = categories[k];
        categories[k] = temp;
    }

    // 将打乱后的分类分配到对应标签
    for (int i = 0; i < labels.Count; i++)
    {
        labels[i].Text = categories[i];
    }
}

额外优化点 #

如果快速点击按钮，每次在方法内新建Random实例可能因为种子相同导致随机序列重复。建议把Random声明为类的成员变量：

private Random r = new Random(); // 类级别变量，只初始化一次

public void button2_Click(object sender, EventArgs e)
{
    // 直接使用已初始化的r即可，无需再次新建
    // ... 其余代码不变
}

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