Question

I’m aware that the random() function in C# uses the processor’s clock to return its values, soon I had to do some tricks to make it return different values in a short timespan. Now something new has ocurred: I can return multiple different values, but it is not quite “random”.

May I explain myself with this picture. I let this piece of code running for a few seconds…

spawnMsCounter += gt.ElapsedGameTime.Milliseconds;
        if (spawnMsCounter > 1)
        {
            spawnMsCounter -= 1;
            random = new Random();
            float x = random.Next(-1, game.Window.ClientBounds.Width);
            int y = random.Next(-1,random.Next(1,game.Window.ClientBounds.Height));
            Coins.Add(new Coin(coinTexture,x,y));
        }

And I got this:
http://img62.imageshack.us/img62/4783/khrogbeta20110916154637.png

As you can see, most points spawn in the top part of the screen; why? And how to fix – or create a better code out of – it?

Answer 1

Do not create a new Random object every time. Make that a private static or something in the class that you are using this in.