避免随机重复颜色答案

【问题标题】：Avoiding Random Duplicate Colours避免随机重复颜色
【发布时间】：2014-12-27 02:19:42
【问题描述】：

我一直在尝试创建游戏“Mastermind”，当我点击“显示”按钮时，它会从 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;



         }

谢谢大家，祝一切顺利！

【问题讨论】：

为什么不比较ToARGB 的值？
如果要避免重复颜色，可以将颜色设置为列表，然后删除使用的颜色。我为此推荐一个单独的功能。顺便说一句，您正在遍历颜色数组，而不是我认为您想要的 NPC 面板。
fyi-我认为你还需要做r.Next(0, RandomColor.Length -1) 否则你会得到一个超出范围的索引
Random.Next 是包含下限和不包含上限。他按原样拨打r.Next 是安全的。 msdn.microsoft.com/en-us/library/2dx6wyd4.aspx 如果进行了更改，则永远不会选择最后一个元素。
HurpaDerpa，只需 suffle 您的阵列并采用前 3 种颜色。

标签： c#

【解决方案1】：

这是一个你可以使用的函数：

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();
}

然后你可以这样调用：

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;
 }

注意：此代码未经测试。它可能包含拼写错误。

【讨论】：