problème rand VS() avec pthread-win32

Question

Je viens dans un étrange problème dans la programmation pthread j'ai compilé le code suivant dans VS2005 avec pthread-w32

#include <cstdio> 
#include <cstdlib> 
#include <ctime> 
#include <pthread.h> 
#include <windows.h> 

pthread_mutex_t lock; 

void* thread1(void *) { 
    int r1; 
    while(true) { 
    pthread_mutex_lock(&lock); // rand is maybe a CS 
    r1 = rand() % 1500; 
    pthread_mutex_unlock(&lock); 
    Sleep(r1); printf("1:%d\n", r1); 
    } 
    return NULL; 
} 

void* thread2(void *) { 
    int r2; 
    while(true) { 
    pthread_mutex_lock(&lock); 
    r2 = rand() % 1500; 
    pthread_mutex_unlock(&lock); 
    Sleep(r2); printf("2:%d\n", r2); 
    } 
    return NULL; 
} 

int main() { 
    srand((int)time(NULL)); 
    pthread_mutex_init(&lock, NULL); 

    pthread_t tc_p, tc_v; 
    pthread_create(&tc_p, NULL, thread1, NULL); 
    pthread_create(&tc_v, NULL, thread2, NULL); 

    pthread_join(tc_p, NULL); 
    pthread_join(tc_v, NULL); 

    pthread_mutex_destroy(&lock); 

    return 0; 
} 

et la sortie est comme ce

2:41 
1:41 
1:467 
2:467 
1:334 
2:334 
1:1000 
2:1000 

c'est comme si rand() retournait le même résultat dans tous les deux appels et j'ai srand() mais le résultat ne change pas chaque fois que je lance le programme

Je suis très nouveau à la programmation multi-threads et j'ai entendu parler du rand() n'est pas thread-safe. mais je n'arrive toujours pas à comprendre si le programme ci-dessus est faux ou si la fonction rand() a un problème.

Answer 1

rand est seulement pseudo-aléatoire, et retournera la même séquence à chaque fois. srand fonctionne uniquement sur le thread en cours, donc l'appeler dans votre thread principal n'affectera pas vos threads de travail.

Vous devez appeler srand à l'intérieur de chaque fil, avec une valeur qui est différent pour chaque thread - par exemple, dans votre thread1 et thread2 fonctions:

srand((int)time(NULL)^(int)pthread_getthreadid_np()); 

Answer 2

Essayez d'utiliser rand_s() à la place, il est thread-safe . Voir here. Bien sûr, ce n'est pas portable.