[решено] qrand() внутри потока

[serg_hd]

Если ипользовать метод qrand() (или просто rand()) внутри потока, то он почему-то всё время возвращает одно и то же число:

Код

C++ (Qt)
run()
{
 qDebug() << qrand();
}
 

Если за его пределами, то всё нормально, числа разные. В чём причина, может кто-нибудь в курсе?
Сами потоки запускаются в разное время.

[serg_hd]

Не совсем понял с чем связано (именно касательно потока), но помогло:
run()
{
 qsrand(QTime(0,0,0).msecsTo(QTime::currentTime()));
 int iRand = (int)qrand();
 qDebug() << iRand;
}

[Alex Custov]

доку надо читать

The sequence of random numbers generated is deterministic per thread. For example, if two threads call qsrand(1) and subsequently calls qrand(), the threads will get the same random number sequence.

qsrand/qrand - thread safe функции, они работают per-thread, и хранят свои промежуточные значения в TLS.

[garryHotDog]

сталкивался с такой проблемой, что в разных потоках были одинаковые случайные числа....твой код(функция run) имеет туже ошибку что была у меня:

Код

C++ (Qt)
 [b]qsrand(QTime(0,0,0).msecsTo(QTime::currentTime()));[/b]
 int iRand = (int)qrand();
 qDebug() << iRand;
 

ты инициализируешь каждый раз рандомизатор с определенным значеним (кол-во секунд) - ОШИБКА: если 2 потока будут созданы в один и тот же интервал времени у них будут одни и те же случайные числа....
Решение
выполняй qsrand(QTime(0,0,0).msecsTo(QTime::currentTime())); в констукторе главного (основного) класса ТОЛЬКО 1 РАЗ (например в конструкторе), а в потоках (run()) используй просто qrand()....и все будет ок

[serg_hd]

... если 2 потока будут созданы в один и тот же интервал времени

дело в том, что не будут  . Это контролирует другой класс.

Решение
выполняй qsrand(QTime(0,0,0).msecsTo(QTime::currentTime())); в констукторе главного (основного) класса ТОЛЬКО 1 РАЗ (например в конструкторе), а в потоках (run()) используй просто qrand()....и все будет ок

Спасибо, попробовал. Числа одинаковые.

главного (основного) класса

Имеется ввиду ведь класс-наследник QThread'a у которого этот метод run()? Если да, то особой роли этот факт (в конструкторе или нет) играть не будет, т.к. поток запускается сразу же после создания объекта этого класса. Да и числа, как выше написал, будут одинаковые.

[SimpleSunny]

... если 2 потока будут созданы в один и тот же интервал времени

дело в том, что не будут  . Это контролирует другой класс.

Мало ли что может случится потом, поэтому лучше немного обезопасить себя.

qsrand(QDateTime::currentDateTime().toTime_t() + int(this));

[serg_hd]

Мало ли что может случится потом, поэтому лучше немного обезопасить себя.

qsrand(QDateTime::currentDateTime().toTime_t() + int(this));

да, в ходе экспериментов это оказался наилучший вариант, блародарствую

[Alex Custov]

QDateTime::currentDateTime().toTime_t()

блин, ну напишите вы ::time(0), зачем так сложно-то

[garryHotDog]

Цитата: garryHotDog от Сегодня в 03:41
главного (основного) класса
Имеется ввиду ведь класс-наследник QThread'a у которого этот метод run()? Если да, то особой роли этот факт (в конструкторе или нет) играть не будет, т.к. поток запускается сразу же после создания объекта этого класса. Да и числа, как выше написал, будут одинаковые.

я имел ввиду выполнить инициализацию qsrand() в каком то одном классе(не потоке), а qrand использовать уже в потоках без повторного использования qsrand()!!!!

[serg_hd]

Цитата: garryHotDog от Сегодня в 03:41
главного (основного) класса
Имеется ввиду ведь класс-наследник QThread'a у которого этот метод run()? Если да, то особой роли этот факт (в конструкторе или нет) играть не будет, т.к. поток запускается сразу же после создания объекта этого класса. Да и числа, как выше написал, будут одинаковые.

я имел ввиду выполнить инициализацию qsrand() в каком то одном классе(не потоке), а qrand использовать уже в потоках без повторного использования qsrand()!!!!

и так пробовал, числа одинаковые

[Alex Custov]

я имел ввиду выполнить инициализацию qsrand() в каком то одном классе(не потоке), а qrand использовать уже в потоках без повторного использования qsrand()!!!!

qsrand() надо делать для каждого потока, желательно разными значениями. Выше уже всё написано, а в документации и подавно.

[garryHotDog]

qsrand() надо делать для каждого потока, желательно разными значениями. Выше уже всё написано, а в документации и подавно.

я так сначала и пробовал: в каждом потоке делал qsrand(), в качестве параметра передавал время в мс....но получалось так что несколько потоков создавалось в один временной интервал и "случайные" числа после qrand() оказывались одинаковыми...попробуй сам!

[Alex Custov]

qsrand() надо делать для каждого потока, желательно разными значениями. Выше уже всё написано, а в документации и подавно.

я так сначала и пробовал: в каждом потоке делал qsrand(), в качестве параметра передавал время в мс....но получалось так что несколько потоков создавалось в один временной интервал и "случайные" числа после qrand() оказывались одинаковыми...попробуй сам!

Я это знаю, по-другому и быть не может - псеводослучайные числа считаются по формуле. Делать qsrand() один раз вне потоков - НЕ правильно, почему - уже написано несколько раз. Чтобы избежать одинаковых рядов можно к зерну добавлять какое-то случайное число, как уже показали - например int(this), или значение неинициализированной переменной (я об этом и написал в предыдущем сообщении).

[garryHotDog]

хмм...что то я все равно не понял почему один раз нельзя!? можно еще раз?

[Alex Custov]

хмм...что то я все равно не понял почему один раз нельзя!? можно еще раз?

qrand() и qsrand() работают с одним конкретным, текущим, TLS, сколько раз ещё написать?! Именно поэтому они потокобезопасны.