【问题标题】:Take a random item from one list and add it to another list从一个列表中取出一个随机项目并将其添加到另一个列表中
【发布时间】:2014-06-25 22:12:41
【问题描述】:

所以...

我有两个列表。一个是通过在其中添加数字 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 列表中。

任何帮助表示赞赏!

【问题讨论】:

  • 使用调试器单步执行应该会显示您的错误;不是在第一次通过循环时,而是可能在任何后续迭代中......
  • @CloseVoter 这一次,这不是问题。

标签: c# list random


【解决方案1】:

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 开始。

【讨论】:

  • 虽然这确实回答了这个问题,但不清楚它是如何回答这个问题的。请以清晰的方式解释答案,而不仅仅是试图快速发布有效的内容。如果 OP 实际上 learns 会更好。
【解决方案2】:

首先,您用于填充allNumbers 列表的所有方法都需要付出很多努力,这可以使用数组上的ToList() 方法完成并将其分配给allNumbers,如下所示:

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

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

int index;

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

【讨论】:

    【解决方案3】:

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

    将您的代码替换为:

    newNumbers.Add(allnumbers[now]);
    

    这应该可以解决问题。

    【讨论】:

      【解决方案4】:

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

      【讨论】:

        【解决方案5】:

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

        使用这个 sn-p 应该可以修复它:

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

        【讨论】:

        • 是的,我很困惑,我以为我是在生成一个数字(那是我将随机方法的 minValue 设置为 1)而不是生成索引...谢谢!
        猜你喜欢
        • 2021-08-25
        • 1970-01-01
        • 1970-01-01
        • 2013-01-23
        • 2021-08-18
        • 2016-09-23
        • 2018-05-01
        • 2021-12-22
        • 1970-01-01
        相关资源
        最近更新 更多