Question

I know this is such an old thing and I have searched like every website for an answer before I post this but I couldn’t find a solution

I am using C# – Windows Forms

and I have this peice of code :

char[] options = { 'a', 'l', 'w' };

and I also have this function :

static Random rand = new Random();
    static char OptionPicker(char[] options)
    {
        rand = new Random();
        return options[rand.Next(0, 2)];
    }

my problem is that when I want to navigate through an array of chars
and execute the previous function on each of this array

I get the same option picked and it doesn’t get randomed ?

can you tell me how to do it ?
Best Regards.

Answer 1

Remove the rand = new Random(); line, you’re creating the same object twice. rand.Next(0, 3) is enough to generate new random numbers. Also make sure that your upper bound is ‘3’ not ‘2’ since it is exclusive.

static Random rand = new Random();
static char OptionPicker(char[] options)
{
    return options[rand.Next(0, 3)]; // 3 is exclusive so results will be 0,1,2 only.
}