Select 单击按钮时来自文件夹的随机、独特的图像

Question

我正在开发一个 C# 纸牌游戏，我希望在单击按钮时随机选择图像。一旦选择了一张卡片，它就必须显示给用户，并且必须发生一些事情才能再次选择它。我记下了第一部分，选择了一张随机图像并显示给用户，但我做不到，所以无法再次选择。有时一张卡片被多次挑选，有时根本没有选择图像，我得到错误图像。这是到目前为止的代码。

public partial class Form1 : Form
    {
        private List<int> useableNumbers;

        public Form1()
        {
            InitializeComponent();
            // Creates a list of numbers (card names) that can be chosen from
            useableNumbers = new List<int>{1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 19, 20, 21, 22, 23, 24, 25, 26, 27, 28, 29, 30, 31, 32, 33, 34,
                                          35, 36, 37, 38, 39, 40, 41, 42, 43, 44, 45, 46, 47, 48, 49, 50, 51, 52, 53, 54};
            settings = new Settings();
        }

        private void btnDrawCard_Click(object sender, EventArgs e)
        {
            this.setImage();
        }

        private void setImage()
        {
            // Checks if there are still numbers left in the List
            if (useableNumbers.Count() == 0)
            {
                MessageBox.Show("The game has ended");
                Application.Exit();
            }
            else
            {
                Random r = new Random();
                int i = r.Next(useableNumbers.Count());
                // Looks for the path the executable is in
                string path = System.IO.Path.GetDirectoryName(Application.ExecutablePath) + @"\images\";
                // Looks up the image in the images folder, with the name picked by the Random class and with the extension .png
                string image = path + i + ".png";
                // Sets the image in the pictureBox as the image looked up
                pictureBox1.ImageLocation = image;
                // Removes the selected image from the List so it can't be used again
                useableNumbers.RemoveAt(i);
            }
        }

        private void quitToolStripMenuItem_Click(object sender, EventArgs e)
        {
            Application.Exit();
        }

        private void settingsToolStripMenuItem_Click(object sender, EventArgs e)
        {
            settings.Show();
        }

澄清一下；我在与可执行文件相同的文件夹中有一个名为 'images' 的文件夹。在该文件夹内，有 54 张名为“1”到“54”的图像（52 张普通牌和两张王牌）。 useableNumbers 列表中的数字代表该文件夹中的图像名称。选择图像后，我想使用 useableNumbers.RemoveAt(i); 从列表中删除该图像的名称。虽然我确实收到消息 'The game has ended'，但我也遇到了上述问题。

我觉得 useableNumbers.RemoveAt(i); 不会更改列表的索引，所以当删除“10”时，它会保留索引 11，而不是将所有值都向下移动一个，如果你明白我的意思。

我也试过将图像存储在一个列表中，但也无法让它工作，所以这就是我这样做的原因。对 C# 还是新手，所以也许有更好的方法。

如何修复从列表中删除的问题，这样我就不会两次或多次获得相同的图像，或者根本没有图像？

Answer 1

当您使用创建 "i" 整数时，该值是从剩余值的数量中选择的，而不是值本身。我建议在 int i = r.Next(useableNumbers.Count()); 下面放一行，使整数等于列表中包含的值之一： int j = useablenumbers(i); 之后，所有 "i" 整数的使用都将被 "j" 代替，除了最后一行 useablenumbers.RemoveAt(i); 将保持不变。

Answer 2

取而代之的是获取介于 0 和可用数字的当前大小之间的随机数，

int i = r.Next(useableNumbers.Count());

我会这样做，

int i = useableNumbers[r.Next(useableNumbers.Count())];

所以这是从剩余的可用数字之一中选择而不是当前大小。目前你可以有重复项，因为，你只是减少了要选择的数字范围 - 一开始它是零到 53，但在一半之后你减少到该范围的一半并且重复项变得更有可能。最后你肯定会得到 1。范围不应该改变，你总是想要一个 1 - 54 的随机数，而不是已经选择的那个。因此，通过这种新方法，您可以在列表中获取随机索引以检索值，而不仅仅是获取随机索引并将其作为您的值。

与此类似，

useableNumbers.RemoveAt(i);

这样做，

useableNumbers.RemoveAt(useableNumbers.IndexOf(i));

然后将从列表中删除那个数字，而不是列表中该位置的值。

Select 单击按钮时来自文件夹的随机、独特的图像

Select random, unique, images from folder on button click

c#

random

image