Как починить рандом? [дубликат]

Всё дело в том что Random при инициализации конструктора без параметров инициализируется в зависимости от текущего времени, если сделать сразу несколько запросов подряд, то за это время время не изменяется и Random генерирует одинаковые значения.

Сам Random при инициализации одинаковым числом генерирует одну и туже последовательность. В вашей функции он каждый раз инициализируется заново и как раз одинаковым параметром.

Решение: Вынести переменную Random за пределы функции, и передавать в функцию как параметр.

Random r1 = new Random();
var a1 = r1.Next();
r1 = new Random();
var a2 = r1.Next();

Высока вероятность что a1 == a2 == a3 итп.

Random r1 = new Random();
var a1 = r1.Next();
var a2 = r1.Next();
var a3 = r1.Next();

a1, a2, a3 - разные числа, если случайно не совпадут.

Все «внутренности» работы класса Random полностью детерминистичны. Это значит, что если вы возьмёте несколько экземпляров класса Random с одинаковым начальным состоянием, которое задаётся через параметр конструктора seed, и для каждого экземпляра вызовите определённые методы в одинаковом порядке и с одинаковыми параметрами, то в конце вы получите одинаковые результаты.

Как починить, и статья раскрывающая тему по ссылке https://habrahabr.ru/post/165459/

З.Ы.Ж. Простейший вариант заплатки:

Random rnd = new Random(DateTime.Now.Millisecond);