随机对象未在C#中处理

Question

我正在使用C#中的平滑地形生成算法并使用XNA来显示数据.

我正在制作它,因此它在每次迭代的每个点之间创建一个新点,在两者之间的随机高度.这工作正常,我设置它,以便在第二次迭代时它选择一个像幻灯片2中的随机点,而不是尝试在同一轴上的点之间创建一个新点.

发生的事情是循环使用上一次迭代中的相同随机值:http://i.stack.imgur.com/UmWr7.png

这显然不是理想的,因为它不是一个适当的随机生成.

如果我Thread.Sleep(20)在每个点生成后使用它,它可以正常工作:http://i.stack.imgur.com/KziOg.png

我不想在可能的情况下使用Sleep变通方法,因为它非常慢,我想实时使用它.我很确定这与C#垃圾收集器有关.

这是我的获取点代码

Random r = new Random();
int x = (p1.X + p2.X) / 2;
int y;
if (!initial)
       y = r.Next(Math.Min(p1.Y, p2.Y), Math.Max(p1.Y, p2.Y));
else
       y = r.Next(Math.Min(p1.Y, p2.Y) - Game1.screenHeight / 2, Math.Max(p1.Y, p2.Y) + Game1.screenHeight / 2);
return new Point(x, y);

垃圾收集是问题的一部分吗？

解决这个问题的任何建议或解决方案

Answer 1

可能你是在循环中创建一个新的Random对象.

for (/* ... */) {
    int x = new Random().Next(); // Don't do this!
    // ...
}

尝试在程序启动时只创建一个Random实例,然后重新使用它.如果你有多个线程,那么你可以为每个线程使用一个随机对象.

Answer 2

听起来你可能Random在每次迭代¹上创建一个新实例.这将从当前时间开始播种 - 所以没有睡眠,你会反复得到相同的值; 睡着了,你最终会得到一个不同的种子.

答案是在整个程序中重用一个 Random实例 - 但要注意Random不是线程安全的事实.如果你所有的工作都是在UI线程中完成的,你会没事的 - 但是你可能想要使用我在文章中Random给出的一种方法(更多地讨论这个问题).

¹是的,既然您已经发布了确实如此的代码.