- 1
- 2
- 3
- 4
- 5
- 6
- 7
public static int getRandomNumber(int from, int to)
{
System.Threading.Thread.Sleep(10);
Random r = new Random();
int var = r.Next(from, to);
return var;
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+110
public static int getRandomNumber(int from, int to)
{
System.Threading.Thread.Sleep(10);
Random r = new Random();
int var = r.Next(from, to);
return var;
}
No comments
3.14159265 20.08.2012 11:52 # +5
Lure Of Chaos 20.08.2012 11:56 # +4
krypt 20.08.2012 12:40 # +5
Но программист настолько суров, что вместо того, чтобы вынести класс за пределы фунции, ждёт пока сменится рандом сид, зависящий от таймера
В общем это труЪговнокод.
Lure Of Chaos 20.08.2012 13:20 # 0
ну здесь же нет стартового сида, поэтому трудно сказать, будут последовательности одинаковы, или нет
phoenixx 20.08.2012 13:37 # +3
vistefan 20.08.2012 19:42 # 0
phoenixx 20.08.2012 19:46 # 0
Либо с помощью Reflector и подобных посмотреть исходный код.
Имхо, чтобы использовать рандомную генерацию, нужно хотя бы знать, что она зависит от начального seed, а не просто тупо скопипастить сэмпл из гугла. Что автор скорее всего и сделал.
krypt 20.08.2012 21:54 # 0
Так вот откуда ноги растут!
Steve_Brown 20.08.2012 15:49 # 0
argos 20.08.2012 11:59 # +5
3.14159265 20.08.2012 12:06 # −1
А Random в шарпах thread-safe?
krypt 20.08.2012 12:42 # 0
3.14159265 20.08.2012 12:49 # 0
>Конкретно с потокобезопасностью тут всё в порядке
Спасибо кеп, мой вопрос о другом.
absolut 20.08.2012 13:01 # +1
Any public static (Shared in Visual Basic) members of this type are thread safe. Any instance members are not guaranteed to be thread safe.
absolut 20.08.2012 13:17 # +1
первый признак конструктора
phoenixx 20.08.2012 13:39 # +1
Теперь нужно подождать нового топика чтобы понять, извлек ли он из этого что-то полезное.
krypt 20.08.2012 13:43 # +2
Думаю - уже не дождёмся