c# - random string generation - two generated one after another give same results

Question

I have a simple piece of code:

public string GenerateRandomString()
        {
            string randomString = string.Empty;
            Random r = new Random();
            for (int i = 0; i < length; i++)
                randomString += chars[r.Next(chars.Length)];

            return randomString;
        }

If i call this function to generate two strings, one after another, they are identical... but if i debug through the two lines where the strings are generated - the results are different. does anyone know why is it happening?

Answer 1

This is happening, because the calls happen very close to each other (during the same milli-second), then the Random constructor will seed the Random object with the same value (it uses date & time by default).

So, there are two solutions, actually.

1. Provide your own seed value, that would be unique each time you construct the Random object.

2. Always use the same Random object - only construct it once.

Personally, I would use the second approach. It can be done by making the Random object static, or making it a member of the class.