Discussion :

[malus56]

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]

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]

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]

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]

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]

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?