Partage

Tuto (jeu du pendu)

18 août 2007 à 8:10:02

Salut à tous,

Je viens de finir le tuto de Matéo (le jeu du pendu)...et même si la partie 1 était relativement facile, j'ai beaucoup plus galéré sur la partie 2 (le dictionnaire) mais j'y suis finalement arrivé.
Bon j'ai bien conscience que mon code est pas super clean (lol je viens de le finir)...mais ça marche, avec une boucle pour savoir si on veut recommencer une partie.

Hésitez pas à commenter...je cherche toujours à apprendre.

PS1: le code sur le passage d'un string en char est pas de moi mais viens de chez dev.com
PS2: Pour ceux qui cherche un dico, vous pourrez trouver ici un .txt de plus de 22000 mots (avec un passage dans notepad++, vous pouvez facilement remplacer tous les È,É, etc...)

#include <iostream>
#include <stdlib.h>
#include <stdio.h>
#include <time.h>
#include <string>
#include <fstream>
#include <cstring>
using namespace std;
int nombreale(int i);
string motmyst(int nombrealeatoire);
int main(int argc, char *argv[])
{
char y='Y';
char recom='Y';
//Intro
cout<<"\t\t### Le jeu du pendu ###\n\n";
cout<<"Vous demarrez avec 10 chances"<<endl;
cout<<"Trouvez le mot avant d'epuiser vos chances!\n\n"<<endl;
do{
char choixLettre=0;
int coup=10;
int bravo=0;
char n='n';
int nombremot=0;
nombremot=nombreale(nombremot)-1;
//cout<<nombremot<<endl;
//nombrealeatoire
int nombrealeatoire;
srand((int)time(NULL));
nombrealeatoire=rand()%nombremot;
if (nombrealeatoire==0)
nombrealeatoire=1;
else
nombrealeatoire=nombrealeatoire;
//cout<<nombrealeatoire<<endl;
//mot mystere
string motmysta;
motmysta=motmyst(nombrealeatoire);
//cout<<motmysta<<endl;
size_t size = motmysta.size() + 1;
char * motmystsC = new char[ size ];
strncpy( motmystsC, motmysta.c_str(), size );
//cout << motmystsC << '\n';
//cout<<motmystsC<<endl;
int longueurmot=strlen(motmystsC);
//cout<<longueurmot<<endl;
char motcache[26]={0};
int j=0;
do
{
motcache[j]='*';
j++;
}while(j<longueurmot);
//cout<<"Le mot cache est : "<<motcache<<endl;
//
//cout<<"Le mot est: "<<motcache<<endl;
int chance=10;
do
{
//cout<<"Le mot est: "<<motmystsC<<endl;
cout<<"Le mot est: "<<motcache<<endl;
int m=0;
int ok=0;
cout<<"Il vous reste "<<chance<<" chance(s)"<<endl;
if (chance==0)
{
cout<<"Desole vous avez perdu"<<endl;
cout<<"Le mot etait: "<<motmystsC<<endl;
break;
}
cout<<"faites votre choix: ";
cin>>choixLettre;
choixLettre = toupper(choixLettre);
cout<<"\n";
for (m=0;m<longueurmot;m++)
{
if (motmystsC[m]==choixLettre)
{
motcache[m]=choixLettre;
ok=1;
}
}
if (ok==0)
{
chance--;
}
//cout<<motcache<<endl;
if (strcmp(motmystsC,motcache)==0)
{
bravo=1;
cout<<"\n\nbravo vous avez trouve le mot mystere qui etait: "<<motcache<<endl;
}
}while (bravo==0);
do
{
cout<<"Voulez vous recommencer ? Y/N : ";
cin>>y;
cout<<"\n";
y = toupper(y);
if (y!='Y'&& y!='N')
cout<<"Veuillez indiquer votre choix par Y ou N!"<<endl;
}while(y!='Y'&& y!='N');
bravo=0;
delete [] motmystsC;
}while(y==recom);
system ("PAUSE");
return 0;
}
//lecture_fichier_et_nombre_aleatoire
int nombreale(int i)
{
fstream fichier;
fichier.open("dico.txt",ios::in|ios::out|ios::app);
if (fichier)
{
string mot;
i=0;
while(i++,getline(fichier,mot));
/*cout<<mot<<endl;*/
}
fichier.close();
return i;
}
//piochemotmyst
string motmyst(int nombrealeatoire)
{
fstream fichier;
fichier.open("dico.txt",ios::in|ios::out|ios::app);
string piochemotmyst;
int compteur=0;
if(fichier)
{
do
{
compteur++;
getline(fichier,piochemotmyst);
}while(compteur!=nombrealeatoire);
return piochemotmyst;
}
fichier.close();
}

Pingouin chauffé

18 août 2007 à 23:04:53

salut !

Il y a clairement un problème dans le code (qui n'empêche cependant pas son fonctionnement). En effet, C et C++ sont deux langages (très) différents. Tu sembles dire que tu as fait ce code en C++, avec le tuto de M@teo21 qui porte sur... le C.

Je suis parti du principe que tu as voulu coder le pendu du tuto en C++. A ce moment là, cest lignes ont un serieux problème :

#include <iostream>
#include <stdlib.h>
#include <stdio.h>
#include <time.h>
#include <string>
#include <fstream>
#include <cstring>

Tu as inclu trois librairies spécifiques au C !

#include <stdlib.h>
/* Il s'agit d'une partie de la bibliothèque standard C. Rien à voir avec le C++ */
#include <stdio.h>
/* Il s'agit du mécanisme de flux d'entrée et de sortie du C. L'équivalent en C++ et iostream (input output stream, soit flux d'entrée/sortie). Tu as inclu le système de flux du C ET du C++. Cela n'est pas du tout correct ! */
#include <time.h>
/* Encore une fois, cette librairie est une librairie C. Si tu souhaites l'utiliser dans un programme C++, tu dois inclure <ctime> (toutes les librairies C ont un équivalent C++ commençant par 'c'. Pour reprendre les deux erreurs précédents, tu aurais dû inclure <cstdio> et <cstdlib>.) */
#include <string>
#include <cstring>
/* Erreur pus grave, qui montre qu'une notion n'est pas assimilée. Soit on code en C, soit en C++. Mélanger les deux crée un code sale, obsolète, et souvent incompatible. Tu fais croire au lecteur que tu vas utiliser le système de chaîne C++ (<sting>), mais, surprise, tu inclus le système de chaîne C (<string.h>). C'est n'importe quoi !*/

Voici aussi deux remarques :

using namespace std;
/* Même si M@teo21 les utilisent dans son cours, il est déconseillé d'encombrer l'espace de nom global de cette façon. Mais là c'est pas ta faute */
int main(int argc, char* argv[])
{
...
}
/* Sais-tu à quoi servent les arguments argc et **argv ? Non ? Alors pourquoi les utilises-tu ? En C, on avait l'excuse de "c'est l'habitude", mais en C++ cela est intolérable. Tu dois définir "main" sans argument (tu peux ou non spécifier "void"), sauf si bien sûr tu compte les utiliser. */

Je n'ai regarder que les premières lignes de ton code. Tu pensais que les deux langages se mélangeaient, ou bien c'est une (énorme) erreur d'innatention ?

A+. Normalement, j'ai pas dit de bêtise

livai

19 août 2007 à 0:00:41

Salut,

Merci à toi pour ce début de correction. En fait, j'avais commencé à coder en C puis je me suis décidé de le reprendre (la partie 2) après avoir lu sur le c++.

Honnêtement, il y a certes de l'inattention mais aussi des erreurs.
Par exemple je connaissais pas tous les équivalents pour les includes et concernant la fonction main, je savais pas non plus.

Je vais laisser le "mauvais" code en haut tel quel et ajouter les corrections (au fur et à mesure) dans ce post. Comme ça les autres zéros pourront comparer et éviter les même erreurs que moi

#include <iostream>
#include <ctime>
#include <string>
#include <fstream>
using namespace std;
int nombreale(int i);
string motmyst(int nombrealeatoire);
int main()
{
char y='Y';
char recom='Y';
//Intro
cout<<"\t\t### Le jeu du pendu ###\n\n";
cout<<"Vous demarrez avec 10 chances"<<endl;
cout<<"Trouvez le mot avant d'epuiser vos chances!\n\n"<<endl;
do
{
char choixLettre=0;
int coup=10;
int bravo=0;
char n='n';
int nombremot=0;
nombremot=nombreale(nombremot)-1;
//cout<<nombremot<<endl;
//nombrealeatoire
int nombrealeatoire;
srand((int)time(NULL));
nombrealeatoire=rand()%nombremot;
if (nombrealeatoire==0)
nombrealeatoire=1;
else
nombrealeatoire=nombrealeatoire;
//cout<<nombrealeatoire<<endl;
//mot mystere
string motmysta;
motmysta=motmyst(nombrealeatoire);
//cout<<motmysta<<endl;
size_t size = motmysta.size() + 1;
char * motmystsC = new char[ size ];
strncpy( motmystsC, motmysta.c_str(), size );
//cout << motmystsC << '\n';
//cout<<motmystsC<<endl;
int longueurmot=strlen(motmystsC);
//cout<<longueurmot<<endl;
char motcache[26]={0};
int j=0;
do
{
motcache[j]='*';
j++;
}while(j<longueurmot);
//cout<<"Le mot cache est : "<<motcache<<endl;
//
//cout<<"Le mot est: "<<motcache<<endl;
int chance=10;
do
{
//cout<<"Le mot est: "<<motmystsC<<endl;
cout<<"Le mot est: "<<motcache<<endl;
int m=0;
int ok=0;
cout<<"Il vous reste "<<chance<<" chance(s)"<<endl;
if (chance==0)
{
cout<<"Desole vous avez perdu"<<endl;
cout<<"Le mot etait: "<<motmystsC<<endl;
break;
}
cout<<"faites votre choix: ";
cin>>choixLettre;
choixLettre = toupper(choixLettre);
cout<<"\n";
for (m=0;m<longueurmot;m++)
{
if (motmystsC[m]==choixLettre)
{
motcache[m]=choixLettre;
ok=1;
}
if (ok==0)
chance--;
//cout<<motcache<<endl;
if (strcmp(motmystsC,motcache)==0)
{
bravo=1;
cout<<"\n\nbravo vous avez trouve le mot mystere qui etait: "<<motcache<<endl;
}
}while (bravo==0);
do
{
cout<<"Voulez vous recommencer ? Y/N : ";
cin>>y;
cout<<"\n";
y = toupper(y);
if (y!='Y'&& y!='N')
cout<<"Veuillez indiquer votre choix par Y ou N!"<<endl;
}while(y!='Y'&& y!='N');
bravo=0;
delete [] motmystsC;
}while(y==recom);
system ("PAUSE");
return 0;
}
//lecture_fichier_et_nombre_aleatoire
int nombreale(int i)
{
fstream fichier;
fichier.open("dico.txt",ios::in|ios::out|ios::app);
if (fichier)
{
string mot;
i=0;
while(i++,getline(fichier,mot));
/*cout<<mot<<endl;*/
}
fichier.close();
return i;
}
//piochemotmyst
string motmyst(int nombrealeatoire)
{
fstream fichier;
fichier.open("dico.txt",ios::in|ios::out|ios::app);
string piochemotmyst;
int compteur=0;
if(fichier)
{
do
{
compteur++;
getline(fichier,piochemotmyst);
}while(compteur!=nombrealeatoire);
return piochemotmyst;
}
fichier.close();
}

Pingouin chauffé

19 août 2007 à 0:31:07

C'est déjà plus clair, mais c'est encore un exemple parfait de code mélangeant le C et le C++

J'ai cru comprendre que tu te basais sur le cours de M@teo21 pour faire ce jeu de pendu en C++, mais ce tuto n'est pas encore assez avancé pour que l'on puisse s'en servir comme cela.

En C++, la bibliothèque standard,nommée STL (Standard Template Library), apporte beaucoup de nouvelles notions (que M@teo21 n'a pas encore détaillé), notions qui devraient apparaître plusieurs fois dans un code comme celui-ci. Par exemple, en C++, l'utilisation d'un tableau de char pour représenter une chaîne de caractères est tout à fait obsolète (sauf dans des cas bien spécifiques). On utilise à la place la classe String (si tu n'as pas de notion en Programmation Orientée Objet, le terme de classe doit te paraître très flou). Dans ton code :

#include <string>

signifie que tu vas utiliser la classe String (ce que tu ne fais pas, cette directive est donc inutile).

Aussi, en C la valeur VRAI (TRUE) et representée par l'entier 1, et la valeur FAUX (FALSE) est représentée par l'entier 0. On nomme cela le système booléen, du mathématicien Boole qui l'a créé.
En C++, on a un nouveau type de donnée : le type bool. Une variable booléenne (de type bool) peut prendre les valeurs true ou false, représentant l'entier 1 et l'entier 0 du C respectivement.

Tu utilises bon nombre de variables booléennes dans ce code, je pense que tu devrais les convertir. Ta variable bravo, par exemple, devrait être de type bool (et initialisée à false).

Dernière chose. L'inclusion de la librairie ctime ets parfaitement légale, mais la plupart du temps avoir recours à des fonctions C est une mauvaise idée. Normalement, le C++ a déjà ce qu'il faut. Je ne me souviens plus de l'équivalent C++ de <time.h> (il y a de grande chance que ce soit <time>

)...

Apprends le C++ plus en profondeur avant de vouloir faire de bon programme avec

.

Essaye de t'initier à la programmation objet avec un bon bouquin, puis lit un ouvrage de référence sur C++ (si tu veux vraiment l'apprendre, car l'apprentissage de C++ est très difficile). Ils proposent pleins de bons ouvrages sur www.developpez.com.

livai

19 août 2007 à 8:03:47

Salut,
Merci pour ton aide et tes conseils.

En fait, je vais essayé "d'oublier" le C pour éviter les mélanges à l'avenir

Sinon, je pense que je vais acheter "Le langage C++" de Jesse Liberty.
En attendant, il y a aussi Thinking In C++ (dont on peut en plus trouver la traduction sur dev.com).

Voilà, là je pense que j'ai de la lecture pour l'hiver

A+

Livai

Anonyme

19 août 2007 à 12:00:24

Citation : &ltZer0&gt

salut !

Il y a clairement un problème dans le code (qui n'empêche cependant pas son fonctionnement). En effet, C et C++ sont deux langages (très) différents. Tu sembles dire que tu as fait ce code en C++, avec le tuto de M@teo21 qui porte sur... le C.

Je suis parti du principe que tu as voulu coder le pendu du tuto en C++. A ce moment là, cest lignes ont un serieux problème :

#include <iostream>

#include <stdlib.h>

#include <stdio.h>

#include <time.h>

#include <string>

#include <fstream>

#include <cstring>

Tu as inclu trois librairies spécifiques au C !

#include <stdlib.h>

/* Il s'agit d'une partie de la bibliothèque standard C. Rien à voir avec le C++ */

#include <stdio.h>

/* Il s'agit du mécanisme de flux d'entrée et de sortie du C. L'équivalent en C++ et iostream (input output stream, soit flux d'entrée/sortie). Tu as inclu le système de flux du C ET du C++. Cela n'est pas du tout correct ! */

#include <time.h>

/* Encore une fois, cette librairie est une librairie C. Si tu souhaites l'utiliser dans un programme C++, tu dois inclure <ctime> (toutes les librairies C ont un équivalent C++ commençant par 'c'. Pour reprendre les deux erreurs précédents, tu aurais dû inclure <cstdio> et <cstdlib>.) */

#include <string>

#include <cstring>

/* Erreur pus grave, qui montre qu'une notion n'est pas assimilée. Soit on code en C, soit en C++. Mélanger les deux crée un code sale, obsolète, et souvent incompatible. Tu fais croire au lecteur que tu vas utiliser le système de chaîne C++ (<sting>), mais, surprise, tu inclus le système de chaîne C (<string.h>). C'est n'importe quoi !*/

Voici aussi deux remarques :

using namespace std;

/* Même si M@teo21 les utilisent dans son cours, il est déconseillé d'encombrer l'espace de nom global de cette façon. Mais là c'est pas ta faute */

int main(int argc, char* argv[])

{

...

}

/* Sais-tu à quoi servent les arguments argc et **argv ? Non ? Alors pourquoi les utilises-tu ? En C, on avait l'excuse de "c'est l'habitude", mais en C++ cela est intolérable. Tu dois définir "main" sans argument (tu peux ou non spécifier "void"), sauf si bien sûr tu compte les utiliser. */

Je n'ai regarder que les premières lignes de ton code. Tu pensais que les deux langages se mélangeaient, ou bien c'est une (énorme) erreur d'innatention ?

A+. Normalement, j'ai pas dit de bêtise .

Il y a des points sur lesquels je suis d'accord, et d'autres non. Déjà, les headers ne sont ni des librairies, ni des bibliothèques ; comme je viens de l'écrire, ce sont des headers. D'autre part, inclure cstdlib est correct, même en C++, beaucoup de programmes écrits par des gros barbus le font. J'ai aussi remarqué que tu disais qu'il existait un équivalent C++ des lib(raries) du C, ce qui est bien évidemment faux, puisque ce sont les mêmes. Même les headers le sont, le 'c', justement, indique au préprocesseur de rechercher le header dans le dossier include du C. Enfin, dans le cas général, inclure <cstring> avec <string> est correct, dans la mesure où l'on veut peut être utiliser des fonctions C comme memset(), ou bien memcpy(), beaucoup plus rapides. Sur ce point, je me trompe peut être, mais je ne connaît pas d'équivalent C++.

PS : Le bouquin de Jesse est de la grosse merde, tu n'y vois même pas les grands points du C++ (je parle de celui où l'on apprend à coder un jeu nommé Decryptix, si je me souviens bien).

Pingouin chauffé

19 août 2007 à 20:07:00

Pour ce qui est des librairies, c'est exact j'aurais du parler d'en-têtes...

Sinon je pensais que les en-têtes débutants par un 'c' étaient une alternative C++ un peu remaniée, je ne me doutais pas que c'était les même

.

Pour <string.h> et <string>, je ne suis pas d'accord. Mélanger le C et le C++, cay mal :colere2:

. Je doute que les fonctions C soient plus rapides que celles C++ de façon significative...

Anonyme

19 août 2007 à 20:38:13

Bah, je connais pas d'alternative à memcpy() en C++, et beaucoup sont dans mon cas, car il n'est pas rare de voir des codes C++ contenant l'usage de cette fonction. Sinon, l'usage de fonction C n'est pas à totalement prohiber. Si tu utilises des fonctions C avec parcimonie, aucune raison de bipper, d'ailleurs la lib standard C++ utilise l'I/O du C hein, sauf que toi, tu t'en rends pas compte.

Cela dit, l'algorithme de la STL std::fill_n() inclut dans <algorithm> permet de remplacer aisément memset() (d'ailleurs utilisé dans l'implémentation).

livai

20 août 2007 à 0:58:48

Salut,

Citation : Kreeg

PS : Le bouquin de Jesse est de la grosse merde, tu n'y vois même pas les grands points du C++ (je parle de celui où l'on apprend à coder un jeu nommé Decryptix, si je me souviens bien).

Si tu regardes ici il y a le bouquin dont je parlais. J'ai eu le temps d'un peu le consulter en bibliothèque et je le trouvais pas mal. Tu confirmes que c'est le même que tu trouvais nul?

Voilà la table des matières:

Bien débuter en C++
Anatomie d'un programme C++
Variables et constantes
Expressions et instructions
Fonctions
La programmation orientée objet
Déroulement d'un programme
Pointeurs
Références
Fonctions avancées
Analyse et conception orientée objet
Héritage
Tableaux et chaînes
Polymorphisme
Classes et fonctions spéciales
Héritage avancé
Les flux
Espaces de nommage
Les modèles
Gestion des erreurs et exceptions
Et maintenant ?
Hexadécimal et binaire
Priorité des opérateurs

A+

Livai

Anonyme

20 août 2007 à 14:14:40

Ce n'est donc à priori pas le bouquin que je pensais. De toutes façons, ce bouquin n'a pas l'air conséquent. Prends un meilleur bouquin.

MatteX

20 août 2007 à 17:42:34

@livai : travail sur ton indentation : dans chaque bloc (entre { et } ) tu indente tout, apres un if sans bloc tu indente une ligne, etc. sinon ça fait des gros bloc lourd à lire pour nous. lorsque le code est bien indenté la lecture est très rapide et les erreurs ressortent comme par magie

livai

20 août 2007 à 18:51:44

Citation : Mattex

@livai : travail sur ton indentation : dans chaque bloc (entre { et } ) tu indente tout, apres un if sans bloc tu indente une ligne, etc. sinon ça fait des gros bloc lourd à lire pour nous. lorsque le code est bien indenté la lecture est très rapide et les erreurs ressortent comme par magie

Salut Mattex, je vais essayer de corriger tout ça prochainement. Merci pour ton aide.

Livai

PS: je vois que l'on est dans le même fuseau horaire...moi aussi je suis de Montréal

MatteX

20 août 2007 à 20:19:28

Effectivement, je suis un montréalais

enfin bonne chance l'indentation fait parti des bonnes pratiques, une fois acquisent c'est un réflexe. Les bonnes pratiques aident à améliorer la rapidité et l'efficacité d'un(e) programmeur(e)

Tuto (jeu du pendu)

× Après avoir cliqué sur "Répondre" vous serez invité à vous connecter pour que votre message soit publié.

× Attention, ce sujet est très ancien. Le déterrer n'est pas forcément approprié. Nous te conseillons de créer un nouveau sujet pour poser ta question.

Tuto (jeu du pendu)

Tuto (jeu du pendu)

Supprimer

Modérer

Signaler le message aux modérateurs