THE HANGED

Commentaire de onacluv le 09/06/2008 14:58:37

Svp, j'aimerai vraiment avoir le conseil de personnes expérimentées pour m'améliorer...

Commentaire de Spoolega le 09/06/2008 23:57:32

absolument pas*

Commentaire de onacluv le 10/06/2008 08:25:49

Bonjour messieurs !  :)

Tout d'abord, merci d'avoir répondu et de m'apporter vos conseils...

Spoolega: Je vais initialiser ça, tu as raison. Pas de warning cependant...
Merci.

JOJO930: C'est vrai, aucune utilisation de la POO... Je me dois de te poser une question... Faut-il utiliser la POO dans le cas où un seul objet sera instancié ?
Simplement pour "ranger les fonctions" ?
Dans un cas tel que ce petit jeu, quelles classes devrai-je créer ? Mot, Joueur, AI ? Merci de m'éclairer un peu sur ce point qui me semble obscur...

Quant à la portabilité de mon code, je suis d'accord et me suis toujours rendu compte de ce défaut. J'utilise en permanence les deux fonctions systemes DOS "CLS" et "PAUSE", car je ne connais pas d'alternative... Si tu as un lien à ce sujet, ou tout simplement la réponse, je suis preneur !
En attendant, je vais modifier mon code pour l'adapter à la POO et je remettrai plus tard en ligne... Merci !  :)

Commentaire de Pistol_Pete le 10/06/2008 11:15:11

Salut

Voila j'apporte aussi quelques remarques:
-Pourquoi inclure vector.h alors que tu ne t'en sert pas?
-Dans ta fonction nombreAleatoire, il est inutile d'appeler srand() a chaque fois que tu appeles cette fonction. srand sert a initialiser le random. Il doit donc etre executer UNE SEULE fois au debut du programme.

Voila pour les remarques. J'ajouterai que c'est bien commenté et que l'organisation du programme est bonne. Il reste maintenant a abandonner les string et a pourquoi pas faire une petite interface graphique.

Bonne continuation
A+

Commentaire de jojo930 le 11/06/2008 05:45:26

Arf c'est sûr qu'il n'y a pas grand chose à mettre en POO ...
Pour effacer la console (Testé sous Linux) : cout << "\033[2J" << endl; ou encore system("clear");
Dans tout les cas, je pense qu'il est préférable d'éviter les system("cls/clear"), je trouve ça brouillon...
Je me demande pourquoi il n'y a pas de fonctions POSIX pour cela, d'ailleurs.

Bye.

Commentaire de onacluv le 11/06/2008 17:24:44

Pffffffffffiouuuuuuuuuuu...........
Enfin, ça compile !
J'ai passé un bon bout de temps à améliorer le code, sa lisibilité... et surtout à l'adapter à la POO !
J'ai suivi vos conseils, reste deux trois trucs à régler au niveau de deux fonctions statiques...
J'ai fait en sorte de ne pas utiliser un seul string pour faire hommage à pistol_Pete.  ;)
J'ai ainsi tout remplacer par des vecteurs< char >...
Ca m'a pas mal compliqué la vie mais maintenant ça fonctionne !
Je règle les deux trois détails et j'édite !

Commentaire de Sodams le 14/06/2008 01:17:11

Encore une chose (oouuuuhhh double post le vilain)

Essaie d'optimiser au maximum ton code.
Dans ta fonction ici tu répètes 2 fois des même lignes :

afficherTitre();
et system("PAUSE");

# void finPartie(string &motTrouve, string &motMystere) {
# if(motTrouve == motMystere) {
# afficherTitre();
# cout << " VOUS AVEZ TROUVE LE MOT MYSTERE !" << endl << endl;
# cout << " Ce dernier etait bien : "<< motMystere <<"" << endl << endl << endl << " ";
# system("PAUSE");
# }
#
# else {
# afficherTitre();
# cout << " VOUS AVEZ PERDU !" << endl << endl;
# cout << " Le mot mystere etait : "<< motMystere <<"" << endl << endl << endl << " ";
# system("PAUSE");
# }
# }

Qui deviendrait donc :-)

# void finPartie(string &motTrouve, string &motMystere) {
# afficherTitre();
# if(motTrouve == motMystere) {
#     cout << " VOUS AVEZ TROUVE LE MOT MYSTERE !" << endl << endl;
#     cout << " Ce dernier etait bien : "<< motMystere <<"" << endl << endl << endl << " ";
# }
# else {
#     cout << " VOUS AVEZ PERDU !" << endl << endl;
#     cout << " Le mot mystere etait : "<< motMystere <<"" << endl << endl << endl << " ";
#
# }
# system("PAUSE");
# }

Commentaire de onacluv le 14/06/2008 17:37:06

Voilà, la nouvelle source est postée.
Lachez vos critiques !  :)

Ps: "cout << "\033[2J" << endl;" ne fonctionne pas sous Vista.
J'ai laissé donc le code comme avant à ce niveau.