Question

I made this code to load about 8 different textures to a list of objects with a texture property.

I have a folder with textures named “1.png, 2.png, 3.png,…….,46.png” and I want the 8 different objects to be loaded with randomly chosen textures.

        DockedFruitsList = new List<Fruit>(8);
        for (int i = 0; i < 8; i++)
        {
            Fruit temp = new Fruit();
            temp = new Fruit();
            temp.Position = AvailablePositions[i];
            int random=(new Random().Next(0, 4600) % 46);
            temp.Texture = Content.Load<Texture2D>(@"Fruits/" + random);
            DockedFruitsList.Add(temp);          

        }

The thing is, despite the random always generates a different number, the result of draw is always the same texture, it changes from a run to another run, but it’s always the same for all the 8 textures

spriteBatch.Begin();
        for (int i = 0; i < DockedFruitsList.Count; i++)
        {
            spriteBatch.Draw(DockedFruitsList[i].Texture, DockedFruitsList[i].Position, Color.White);
        }
        spriteBatch.End();

Answer 1

Try this out:

for (int i = 0; i < 100; i++)
    Console.WriteLine(new Random().Next(0, 10));

Odds are you’ll get an output like this:

5
5
5
.
.
.

This is because the seed of the Random class is initialized with a time-based value. If you initialize many instances of the Random class near the same point in time, then they will all end up with the same seed, and so the values they will produce will all be the same.

What you really want to do is to create one Random instance and use that for all your random values. The values generated will have no relation to each other:

var r = new Random();
for (int i = 0; i < 100; i++)
    Console.WriteLine(r.Next(0, 10));

Output:

2
8
0
.
.
.