Question

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

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

我在List类型上读到了一个引用类型＆amp;它会覆盖东西。以下是我的代码。我错过了可以给我独特的产品实例的东西吗？谢谢＆amp;我很感激你的帮助。

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()会生成具有相同种子的Randoms。

在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)。

默认种子值是从系统时钟派生的，并且是有限的解析度。结果，创建了不同的Random对象通过调用默认构造函数来关闭连续相同的默认种子值，因此，将产生相同的随机数集。使用单个可以避免此问题随机对象生成所有随机数。

Answer 3

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

Answer 4

随机播种

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

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

随机产品列表被覆盖

4 个答案: