pb avec srand

Version imprimable

pb avec srand

Bonjour,
j'ai un problème avec un génération de nombre aléatoire à cause je pense de srand((unsigned)(time(NULL))) sous visual c++ 6.
En fait j'ai un lien de composition entre une classe filtre et une matrice. Le contenu de ma classe matrice est généré aléatoirement grâce a un rand()%6 puis je créé un filtre qui est un tableau de matrice grâce à une boucle.
Ensuite pour que mes nombre soit vraiment tiré aléatoirement je dois initialise srand((unsigned)(time(NULL))); mais si je le mais dans mon constructeur de matrice celui-ci aura toujours la même valeur et si je le mets dans OnInitDialog() mon programme crash je ne sait donc pas ou le mettre.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
 
matrice::matrice()
{
 
	tab= new cases**[2];
 
    for(int i=0;i<2;i++)
    {
        tab[i]=new cases*[2];
        for (int j=0;j<2;j++)
        {
			tab[i][j]=NULL;
 
 
 
		}
	}
 
 
srand((unsigned)(time(NULL)));
int var = rand()%6;
 
			switch (var)
                            {
                                case 1:
									tab[0][0]=new cases(true);
									tab[0][1]=new cases(true);
									tab[1][0]=new cases(false);
									tab[1][1]=new cases(false);
                                    break;
                                case 2:
                                    tab[0][0]=new cases(true);
									tab[0][1]=new cases(false);
									tab[1][0]=new cases(true);
									tab[1][1]=new cases(false);
                                    break;
                                case 3:
                                    tab[0][0]=new cases(true);
									tab[0][1]=new cases(false);
									tab[1][0]=new cases(false);
									tab[1][1]=new cases(true);
                                    break;
                                case 4:
                                    tab[0][0]=new cases(false);
									tab[0][1]=new cases(true);
									tab[1][0]=new cases(true);
									tab[1][1]=new cases(false);
                                    break;
								case 5:
                                     tab[0][0]=new cases(false);
									tab[0][1]=new cases(true);
									tab[1][0]=new cases(false);
									tab[1][1]=new cases(true);
                                    break;
								case 6:
                                    tab[0][0]=new cases(false);
									tab[0][1]=new cases(false);
									tab[1][0]=new cases(true);
									tab[1][1]=new cases(true);
 
 
 
                            }
 
 
}
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
nb=n;
 
	tab = new matrice*[n];
 
	for (int i=0;i<n;i++)
	{
		tab[i] = new matrice;
	}

Merci à ceux qui voudront bien m'aider

05/07/2011, 11h28
skeud

rand te renvoi un nombre entre 0 et 1 donc forément tu auras toujours 1 dans ton cas.

Fais plutot comme ça:

Code:

int var=(rand()*6)%6

[0-1]
fois 6
[0-6]
modulo 6
[0-6] en entier sans virgule ;)
05/07/2011, 11h38
dragonjoker59

Citation:

rand te renvoi un nombre entre 0 et 1 donc forément tu auras toujours 1 dans ton cas.

Tu es sur de ça ? Moi je voyais plutôt ça :

Citation:

The rand function returns a pseudorandom integer in the range 0 to RAND_MAX (32767). Use the srand function to seed the pseudorandom-number generator before calling rand.

Par contre pour le crash dans OnInitDialog,, ça me laisse perplexe. Es-tu bien sur que ça vienne de là ? Sinon, tu peux toujours le mettre à la création de ton appli (dans le WinMain)
05/07/2011, 13h07
skeud

Citation:

Si vous désirez engendrer un entier aléatoire entre 1 et 10, vous devez toujours procéder en utilisant les bits de poids forts, comme dans :
j=1+(int) (10.0*rand()/(RAND_MAX+1.0));

et jamais ainsi :
j=1+(rand() % 10);

(car cette derniere version utilise les bits de poids faibles)

;)
05/07/2011, 13h43
malus56

Citation:

Par contre pour le crash dans OnInitDialog,, ça me laisse perplexe. Es-tu bien sur que ça vienne de là ? Sinon, tu peux toujours le mettre à la création de ton appli (dans le WinMain)

A priori oui si je mets mon srand((unsigned)(time(NULL))) dans mon constructeur ça compile et fonctionne correctement hormis que le tirage n'est pas aléatoire mais si je l'enlève ou le met dans OnInitDialog ça compile mais dès que j'instancie mon filtre j'ai un crash.
Par contre j'ai pas de main je suis sous visual c++ donc je sais toujours pas où mettre mon srand.
05/07/2011, 13h47
skeud

T'as essayé ce que j'ai mis?

Tu as un message d'erreur pour le crash?

Pourquoi un cast en unsigned? ça compile pas sans?
05/07/2011, 13h58
malus56

Citation:

T'as essayé ce que j'ai mis?

Tu as un message d'erreur pour le crash?

Pourquoi un cast en unsigned? ça compile pas sans?

oui j'ai essayer ce que tu as mis.
Je mets le unsigned je le mets parceque j'ai vu que c’était conseille pour la gestion de l'aléatoire lors de l'initialisation. Si je le mets ailleurs ou que je l'enlève ça me fait un rapport d'erreur lors de l’événement qui déclenche le constructeur
05/07/2011, 14h01
skeud

Essaye d'afficher la variable générer pour voir ça valeur, ça me parait bizarre que ça te génere pas une valeur aléatoire.

Edit: c'est bon j'ai compris.
Tu utilise srand dans ton constructeur, donc tu réinitialise la graine a chaque construction, comme la construction prend moins de 1 sec, la graine est toujours la meme.met le srand au dessus de ta boucle de création.Ou alors garde un flag en statique dans ta classe pour savoir si ton srand a été fait ;)
05/07/2011, 14h40
malus56

Citation:

Envoyé par skeud

Essaye d'afficher la variable générer pour voir ça valeur, ça me parait bizarre que ça te génere pas une valeur aléatoire.

Edit: c'est bon j'ai compris.
Tu utilise srand dans ton constructeur, donc tu réinitialise la graine a chaque construction, comme la construction prend moins de 1 sec, la graine est toujours la meme.met le srand au dessus de ta boucle de création.Ou alors garde un flag en statique dans ta classe pour savoir si ton srand a été fait ;)

J'ai declaré un static flag que j'initialise a false dans mon constructeur je fait

Citation:

if(!flag)
{
srand((unsigned)(time(NULL)));
flag=true;
}
int j=1+(int) (10.0*rand()/(RAND_MAX+1.0));

si je laisse mon if j'ai un crash a l'instanciation si je je l'enleve ca plante pas mais comme avant pas de génération aléatoire:cry:
05/07/2011, 14h43
skeud

Je pense que ça vient du fait que le srand est fait dans un bloc qui se termine, du coup la graine est valable que pour cette partie la, au changement d'entité, la graine n'est pus valide.

J'ai une méthode un peu porcasse mais qui pourrait fonctionné:

Plutot qu'un flag, utilise un int, que tu incremente à chaque srand.
Au moment du srand, utilise l'int plutot que le time(NULL).

Je sais pas si ça fonctionnera mais on peut toujours essayer ;)
05/07/2011, 14h53
malus56

Citation:

srand((unsigned)(time(&flag)));;
flag++;

int j=1+(int) (10.0*rand()/(RAND_MAX+1.0));

il y a du progrès mes c'est pas encore ca :roll:. Ca ne crash plus et les valeurs changent à chaque exécution mais quand je construit mon tableau de matrice toute les matrice à l’intérieur sont les même.
05/07/2011, 15h18
malus56

C'est bon ça marche merci pour ton aide voila

Citation:

if(flag==0)
{
srand((unsigned)(time(&flag)));;
flag++;
}
int j=1+(int) (6*rand()/(RAND_MAX+1.0));
05/07/2011, 15h59
skeud

Ah oui utilisé l'addresse, j'y avais pas penser ^^.

Bon beh super si ça marche comme ça ;)