pb avec srand

**malus56** · 05/07/2011, 11h18

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

**skeud** · 05/07/2011, 11h28

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

Fais plutot comme ça:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[0-1]
fois 6
[0-6]
modulo 6
[0-6] en entier sans virgule

**dragonjoker59** · 05/07/2011, 11h38

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 :

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)

**skeud** · 05/07/2011, 13h07

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)

**malus56** · 05/07/2011, 13h43

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.

**skeud** · 05/07/2011, 13h47

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?

**malus56** · 05/07/2011, 13h58

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

**skeud** · 05/07/2011, 14h01

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

**malus56** · 05/07/2011, 14h40

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

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

**skeud** · 05/07/2011, 14h43

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

**malus56** · 05/07/2011, 14h53

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

. 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.