Question

因此...

我有两个清单。一个是通过在其中添加数字1-48来创建的。另一个在开头是空的：

int max = 35;
int[] Numbers = {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};
List<int> allNumbers = new List<int>();
List<int> newNumbers = new List<int>();

我使用此方法填充allNumbers列表：

for (int i = 0; i < Numbers.Length - 1; i++)
        {
            allNumbers.Add(Numbers[i]);
        }

我的应用程序应该在allNumbers列表中选择一个随机项并将其添加到newNumbers列表中：

Random r = new Random();
int now;

for (int i = 1; i < max; i++)
{
now = r.Next (1, allNumbers.Count);
allNumbers.Remove(now);
newNumbers.Add(now);

这是有效的（因为它不会崩溃或任何事情）但它不会删除随机数，即我通常在我的newNumbers列表中多次获得相同的数字（例如，我可以获得11 ，4,5,5,11,27,33,4等。）。

这应该是一个有点类似于彩票的游戏，所以不能有任何双数。如何从allNumbers列表中选择一个随机列表项，将其添加到newNumbers列表中，然后将其从allNumbers列表中删除？因此，当for循环再次通过它时，allNumbers.Count将小1，而我之前选择的数字不再在allNumbers列表中。

任何帮助表示赞赏！

Answer 1

这是因为您正在选择一个随机索引，删除该索引处的值，然后将该索引作为值添加到newNumbers列表中。

使用此代码段应该修复它：

for (int i = 0; i < max; i++)
{
   now = r.Next(0, allNumbers.Count);
   newNumbers.Add(allNumbers[now]);  
   allNumbers.RemoveAt(now);
}

Answer 2

问题在于您要添加Random.Next方法生成的数字，而不是随机索引上的列表内容。

用以下代码替换您的代码：

newNumbers.Add(allnumbers[now]);

这应该可以解决问题。

Answer 3

首先，您用来填充allNumbers列表的方法是为了可以使用阵列上的ToList()方法完成并将其分配给allNumbers，像这样：

List<int> allNumbers = Numbers.ToList();

使用random从列表中获取索引并在该索引处添加项的值并将其插入newNumbers列表。然后，您可以使用索引将其从allNumbers列表中删除。

int index;

for (int i = 0; i < max; i++)
{
    index = r.Next (0, allNumbers.Count);
    newNumbers.Add(allNumbers[index]);
    allNumbers.RemoveAt(index);
}

Answer 4

您正在使用now = r.Next(allNumbers.Count);随机化索引，然后尝试将此索引作为值删除。更改上面的行（索引从0开始！）并使用RemoveAt方法而不是Remove。

Answer 5

now变量表示数字列表中的索引（位置）。因此，您需要使用[]运算符检索与此索引对应的数字。

然后，您可以将此数字添加到newNumbers列表，并在使用RemoveAt方法从allNumbers中删除相同的数字后，将索引作为参数传递。

for (int i = 0; i < max; i++)
{
    now = r.Next(0, allNumbers.Count);

    newNumbers.Add(allNumbers[now]);
    allNumbers.RemoveAt(now);

}

如果您愿意，还可以使用Remove方法，将要删除的数字作为参数传递，而不是索引。

allNumbers.Remove(allNumbers[now]);

注意索引，在C＃中从0开始。

从一个列表中取一个随机项并将其添加到另一个列表中

5 个答案: