Salut à tous !

Pour m'entrainer sur les classes, j'ai voulu refaire le TP Plus ou moins de M@teo mais en utilisant cette fois-ci les classes.

Mais j'ai un petit problème , à chaque partie que je recommence le nombre mystère reste le même .

Voici mon code :

Nombre.h:

Nombre.cpp:

main.cpp:

Merci de m'aider à résoudre ce problème .

dans le tuto de M@theo, si tu relance le programme juste après l'avoir fermé, c'est également le même nombre.
Regarde dans les tutos, je coris qu'il y en a un pour faire des nombres un peu plus aléatoire

Pourtant quand j'ai fait le tp de m@téo ca changeait avec chaque fois .
J'ai regardé dans le tuto sur ces fameux nombres aléatoires et j'ai essayé la fonction sur ce projet et sur un autre à part mais le problème persiste .

Citation : Geoffroy

dans le tuto de M@theo, si tu relance le programme juste après l'avoir fermé, c'est également le même nombre.
Regarde dans les tutos, je coris qu'il y en a un pour faire des nombres un peu plus aléatoire

Je pense que geoffoy a raison le nombre est seulement pseudo aléatoire dans ce cas la ... il faut chercher un peu (dsl mais j'ai oublié comment faire)

inclus <ctime> et enlève <stdlib.h>. Le premier point résoudra peut-être ton problème

J'ai mis <ctime> à la place mais ca ne marche toujours pas .

Salut,

le problème n'as aucun rapport avec les nombre aléatoire.
c'est simplement une mauvaise conception du code

je vais mettre en rouge ce que j'ai changé, en espérant que ça t'aidera :

edit : désolé, il semble avoir un problème quand on met des balises dans une balise de code
edit2: je vais les enlever, rien que pour etre capable de lire convenablement

Nombre.h:

#include <iostream>

class Nombre
{
public:

Nombre();
void genererNombre(int nombreAleatoire, int MAX, int MIN);
void comparerNombre(int nombreTape, int nombreMystere);

private:

int m_nombreMystere;
int m_nombreTape;

};

Nombre.cpp:

#include <iostream>
#include <stdlib.h>

using namespace std;

#include "Nombre.h"

Nombre::Nombre()
{
m_nombreMystere = 0;
m_nombreTape = 0;
}

void Nombre::genererNombre(int nombreAleatoire, int MAX, int MIN)
{
nombreAleatoire = (rand() % (MAX - MIN + 1)) + MIN;

m_nombreMystere = (rand() % (MAX - MIN + 1)) + MIN;
}

void Nombre::comparerNombre(int nombreTape, int nombreMystere)
{
int nombreTape = 0;
do
{
cout << "Choisissez un nombre entre 1 et 100" << endl;
cin >> nombreTape;

m_nombreMystere = nombreMystere;
m_nombreTape = nombreTape;

if (nombreTape < m_nombreMystere)
{
cout << "C'est plus." << endl << endl;
}

else if (nombreTape > m_nombreMystere)
{
cout << "C'est moins." << endl << endl;
}

else
{
cout << "Bravo, vous avez trouve le nombre mystere !" << endl << endl;
}
}
while(m_nombreMystere != nombreTape);
}

main.cpp:

#include <iostream>

#include "Nombre.h"

using namespace std;

int main()
{
int nombreMystere, nombreTape;
Nombre Nombre;
srand(time(NULL));

Nombre.genererNombre(nombreMystere, 100, 0);

cout << "<<<<<< Plus ou moins avec les classes >>>>>>" << endl << "Un jeu cree par Gymnopaul" << endl << endl;

Nombre.comparerNombre(nombreTape, nombreMystere);

return 0;
}




Voilà !

shadosan, mon sauveur !!!
Merci beaucoup de cette réponse très précise, j'ai bien compris mon erreur, et tout marche impec maintenant.

Encore merci à toi et à tous !!! .

EDIT : C'est quoi le problème de balise de code ?

bah tiens je vais le faire juste là (si tu vois rien d'étrange chez toi, c'est bizarre) :




edit : testé sur FFv2.0.0.7 et IE7, pareil.

-> c'est normal qu'on ne puisse pas faire comme tu as fait, c'est le but d'une balise code après tous!