Discussion :

[Astraya]

Bonjour à tous!
J'ai un petit problème dont je ne comprend pas le "fonctionnement":
Je me trouve présentement avec un classe Devinette qui réalise le simple jeux trouve le nombre.. trop petit... trop grand, je génère donc un chiffre aléatoire avec une classe Randomize, or il me génère bien des nombres aléatoire mais dans l'ordre croissant... Voici mes classes:
Devinette

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
 
class Devinette
{
public :
    static enum {TROP_GRAND = 2, TROP_PETIT = -1, SUCCES = 1, AUCUN_TEST = 0};
    static enum {VALEUR_MINIMALE = 1, VALEUR_MAXIMALE = 100000};
    int iNombreAleatoire; 
private :
    static Randomize GestionnaireAleatoire;
    unsigned int iNombreEssai;
    bool bReussite;
 
 
 
public :
    Devinette();
    static bool EstNombreValide(int _nombre);
    int Essayer(int _nombre) ;
    bool EstReussi() const;
    unsigned int GetNombreEssais() const ;
 
};

Dans mon Devinette.cpp

Code :

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

Randomize Devinette::GestionnaireAleatoire;

Randomize

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
 
class Randomize
{
 
public:
 
    Randomize()
    {
       srand(static_cast<unsigned int>(time(0)));
    }
 
    static int GetRandomNumber(const int _max, const int _min)
    {
        return ((rand() % (_max - _min + 1)) + _min);
    }
};

Ayant passer pas mal de temps sur java, j'ai oublié un peu mes classiques de C++ .
Ce que je ne comprend pas c'est pourquoi... Je ne pense pas qu'il s'agit d'un problème énorme... mais je ne vois pas le problème!
Merci

[Médinoc]

Tu devrais utiliser la formule passant par des floats, plutôt qu'un modulo.

[jabbounet]

il ne faut pas changer tout le temps le int en paramètre sinon tu réinitialises a chaque fois le générateur.

The srand() function sets its argument as the seed for a new sequence of pseudo-random integers to be returned by rand(). These sequences are repeatable by calling srand() with the same seed value.

http://linux.die.net/man/3/srand

[Médinoc]

il ne faut pas changer tout le temps le int en paramètre sinon tu reinitiliser a chaque fois le générateur.

Sauf qu'il s'agit d'une variable statique: Il n'est initialisé qu'une seule fois, ce qui est correct.

[3DArchi]

Bonjour,
Il y a une discussion sur srand assez intéressante.

[Invité]

A vue de nez, tu dois réinstancier ton générateur à chaque appel... (c'est la seule explication pour tes nombres croissants) Pour en etre sur, essaye de remplacer, dans le constructeur de randomize, time(0) par une valeur constante, et regarde si ton programme te renvoie toujours cette valeur...

Si la valeur est toujours la même c'est forcément que GetRandomNumber() n'est pas appelée...

Tu l'appelles comment GetRandomNumber()?

Aussi, je ne suis pas certain de comprendre le static int devant sa déclaration... Ca en fait une fonction statique, je crois, mais sur une classe instanciée, dont on définit dans une autre classe une instance statique...

Ca sent un peu le soufre, quoi...

J'ai l'impression qu'un appel à srand() au début de l'application, puis des appels à rand(), sans cette classe Randomize qui semble être la cause de tous tes malheurs (et qui n'apporte rien) te mènerait assez loin.

Francois

[jabbounet]

Je confirme un utilisation incorrecte de la classe Randomize peux donner de mauvais résultats.

Quand je recrée a chaque fois Randomize, j'ai un résultat constant (probablement due à la granularité de la fonction time).

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
 
#include <iostream>
 
class Randomize
{
 
public:
 
    Randomize()
    {
       srand(static_cast<unsigned int>(time(0)));
    }
 
  static int GetRandomNumber(const int _max, const int _min)
  {
    return ((rand() % (_max - _min + 1)) + _min);
  }
};
 
 
int getBadRand() {
  Randomize *r = new Randomize();
  //  sleep(1);
  int random = r ->  GetRandomNumber(1000000,0);
  delete r;
  return random;
}
 
 
int main(int argc,char* *argv)
{
  Randomize *r = new Randomize();
 
  std::cout << "Correct random generation :"  << std::endl
            << r ->  GetRandomNumber(1000000,0) << std::endl
            << r ->  GetRandomNumber(1000000,0) << std::endl
            << r ->  GetRandomNumber(1000000,0) << std::endl
            << r ->  GetRandomNumber(1000000,0) << std::endl
            << r ->  GetRandomNumber(1000000,0) << std::endl
            << r ->  GetRandomNumber(1000000,0) << std::endl
            << r ->  GetRandomNumber(1000000,0) << std::endl
            << r ->  GetRandomNumber(1000000,0) << std::endl
            << r ->  GetRandomNumber(1000000,0) << std::endl;
  delete r;
 
  std::cout << "Bad random generation :"  << std::endl
            << getBadRand() << std::endl
            << getBadRand() << std::endl
            << getBadRand() << std::endl
            << getBadRand() << std::endl
            << getBadRand() << std::endl
            << getBadRand() << std::endl
            << getBadRand() << std::endl
            << getBadRand() << std::endl
            << getBadRand() << std::endl;
 
  return 0;
}

g++ randomtest.cpp -o randomtest && ./randomtest
Correct random generation :
215328
170112
138454
184037
39074
346258
110469
271663
856304
Bad random generation :
856304
856304
856304
856304
856304
856304
856304
856304
856304

[Médinoc]

En effet, pour ce genre de choses, il vaut mieux rendre la classe entièrement statique:

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
#include <iostream>
 
class Randomize
{ 
private:
    Randomize()
    {
       srand(static_cast<unsigned int>(time(0)));
    }
 
    static Randomize instance;
public:
  static int GetRandomNumber(const int _max, const int _min)
  {
    //je ne sais plus si c'est nécessaire,
    //mais je veux être sûr que l'instance soit créée.
    (void)instance;
    //Retourne une nouvelle valeur.
    //La version float serait préférable.
    return ((rand() % (_max - _min + 1)) + _min);
  }
};

Code :

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

Randomize Randomize::instance;

[Invité]

En effet, pour ce genre de choses, il vaut mieux rendre la classe entièrement statique:

Oui, mais alors, il va falloir qu'on m'explique l'intéret de la classe, par rapport à un bête srand() au début du programme et une fonction "libre" GetRand(min,max) qu'on appelle quand on a besoin...

Une classe entièrement statique, sans données membres, et n'ayant qu'une fonction, j'avoue que je ne vois pas l'utilité.

Francois

[Médinoc]

Le seul avantage est que srand() est automatiquement appelé, ce qui en vaut pas forcément grand-chose.

Les vraies classes Random sont supposées maintenir leur propre statut, ce qui permet d'avoir des données reproductibles sans déranger les autres parties du programme qui ont besoin de valeurs pseudo-aléatoires (et potentiellement reproductibles).

[Goten]

Ne jamais utilisé le modulo avec le résultat de rand(). (y'a bien une entrée de la faq là dessus non?)