随机产品列表被覆盖

Question

我正在尝试使用以下方法生成随机产品列表，但我多次获得相同的产品实例。

Output for count 5:
Name: Qmlcloboa Price: 3.608848
Name: Qmlcloboa Price: 3.608848
Name: Qmlcloboa Price: 3.608848
Name: Qmlcloboa Price: 3.608848
Name: Qmlcloboa Price: 3.608848

我读到列表类型是一个引用类型，它覆盖了东西。下面是我的代码。我是否缺少一些可以为我提供独特产品实例的东西？谢谢，我将感谢您的帮助。

public List<Product> ProductGroupGenerator(int count)
    {
        List<Product> pList = new List<Product>();
        for (int i = 0; i < count; i++)
        {
            Random r = new Random();
            string alphabet = "abcdefghijklmnopqrstuvwyxzeeeiouea";
            Func<char> randomLetter = () => alphabet[r.Next(alphabet.Length)];
            Func<int, string> makeName =
              (length) => new string(Enumerable.Range(0, length)
                 .Select(x => x == 0 ? char.ToUpper(randomLetter()) : randomLetter())
                 .ToArray());
            //string last = makeName(r.Next(7) + 7);
            //string company = makeName(r.Next(7) + 7) + " Inc.";
            string prodName = makeName(r.Next(5) + 5);
            int unitsInStock = r.Next(100);
            float unitPrice = (float)(r.NextDouble() * 10);
            Product p = new Product();
            p.Name = prodName;
            p.UnitsInStock = unitsInStock;
            p.UnitPrice = unitPrice;
            pList.Add(p);
            p = null;
        }
        return pList;
    }

Answer 1

当调用太快时，多个Random r = new Random()将产生具有相同种子的随机数。

在 for 循环之外声明一次，你应该有更好的值。

    Random r = new Random();
    for (int i = 0; i < count; i++)
    {

Answer 2

将

Random 变量的创建移动到顶部两行：

Random r = new Random();
for (int i = 0; i < count; i++)
{

否则，您总是对随机使用相同的种子，因为 for 循环执行得太快了。Random构造函数使用当前时间。相当于新Random(Environment.TickCount)。

默认种子值派生自系统时钟，并且具有有限的 分辨率。因此，在 中创建的不同随机对象 通过调用默认构造函数的紧密继承将具有 相同的默认种子值，因此将产生相同的 随机数集。此问题可以通过使用单个 随机对象生成所有随机数。

Answer 3

尝试创建 Random 的单个实例，并在for循环期间使用它。每次创建 Random 的实例时，它都会重置种子，因此每次调用都会获得相同的值。

Answer 4

随机

播种

Random r = new Random(DateTime.Now.Millisecond);

此外，在循环之外声明它。