C# 避免随机重复颜色

我一直在尝试创建游戏“智囊团”，我尝试当我点击“显示”按钮时，它将从ROYGBIV生成3种颜色。（已经完成）

但是现在我需要做一个声明，它将比较生成的随机颜色。 这是代码的一部分

public partial class Form1 : Form
    {
        Color[] RandomColor = new Color[7] { Color.Red, Color.Orange, Color.Yellow, Color.Green, Color.Blue, Color.Indigo, Color.Violet };
        Random r = new Random();

        public Form1()
        {
            InitializeComponent();

        }

        private void btnShow_Click(object sender, EventArgs e)
        {
            btnShow.Text = "Hide";

            for (int i = 0; i < RandomColor.Length; i++)
            {
                int RandomColorNum = r.Next(0, RandomColor.Length);

                switch (i)
                {
                    case 1: pnlNPC1.BackColor = RandomColor[RandomColorNum];
                        break;
                    case 2: pnlNPC2.BackColor = RandomColor[RandomColorNum];
                        break;
                    case 3: pnlNPC3.BackColor = RandomColor[RandomColorNum];
                        break;
                }

            } 

            pnlNPC1.Visible = true;
            pnlNPC2.Visible = true;
            pnlNPC3.Visible = true;



         }

公共部分类表单1:表单
{
颜色[]随机颜色=新颜色[7]{Color.Red，Color.橙，Color.Yellow，Color.Green，Color.Blue，Color.Indigo，Color.Violet}；
随机r=新随机（）；
公共表格1（）
{
初始化组件（）；
}
私有void b显示\单击（对象发送者，事件参数e）
{
btnShow.Text=“隐藏”；
for（int i=0；i

---

谢谢大家，祝你们一切顺利！

这里有一个函数可以使用：

int[] PickIntegers(int amountToPick, int maxSize)
{
    List<int> numbers = new List<int>(amountToPick);

    for (int x = 0; x < amountToPick; x++)
    {
        int n = 0;
        while (true)
        {
            n = r.Next(0, maxSize);
            if (!numbers.Contains(n))
            {
                numbers.Add(n);
                break;
            }
        }
    }

    return numbers.ToArray();
}

---

注意：此代码尚未测试。它可能包含打字错误。

为什么不将

与RGB

值进行比较？如果您希望避免重复颜色，可以将颜色设置为列表，然后删除所使用的颜色。我建议为此使用单独的函数。顺便说一句，您正在迭代颜色数组，而不是我假定的NPC面板u wanted.fyi-我认为您还需要执行

r.Next（0，RandomColor.Length-1）

否则您将从边界随机中获取索引。下一个是包含下限的，上一个是独占的。他可以按原样调用

r。下一个

调用。如果进行了更改，将永远不会选择最后一个元素。HurpaDerpa，只需使用数组并使用前3种颜色。

private void btnShow_Click(object sender, EventArgs e)
{
    btnShow.Text = "Hide";

    int[] numbers = PickIntegers(3, RandomColor.Length);

    pnlNPC1.BackColor = RandomColor[numbers[0]];
    pnlNPC2.BackColor = RandomColor[numbers[1]];
    pnlNPC3.BackColor = RandomColor[numbers[2]];


    pnlNPC1.Visible = true;
    pnlNPC2.Visible = true;
    pnlNPC3.Visible = true;
 }