quelqu'un qui si connait en code c pourrait m'aider svp Résolu/Fermé

Question

Bonjour, j'ai un problème avec mon programme qui exécute un jeu du pendu, je n'est pas beaucoup de temps devant moi, si quelqu'un peut montrer le bout de son nez, je partage le programme sinon tan pis. Merci d'avance  



Configuration: Windows / Chrome 43.0.2357.81

-L0Lock- · Answer

Bonjour. Ça irait tellement plus vite si tu expliquais dès le début ton problème, code à l'appui.
Là tu perds du temps à attendre une première réponse, et nous à attendre que tu expliques après t'avoir répondu, et...
Tout ce temps perdu pendant lequel on aurait pu analyser le problème et te donner tout de suite des pistes.

aloudi12 · Answer

Désolé, c'est mon premier post et je ne savais pas si j'allais avoir une réponse rapide. Alors pour commencer voici mon programme, c'est un jeu du pendu: #include #include #include int main()//fonction principale { char mot[5]; char moti[5]; char lettre; int success = 0; int erreur = 0; mot[0]=80;//P mot[1]=79;//O mot[2]=77;//M mot[3]=77;//M mot[4]=69;//E moti[0]='P'; moti[1]='-'; moti[2]='-'; moti[3]='-'; moti[4]='-'; printf("Bienvenue dans le jeu du pendu! "); printf("Entrez une lettre en majuscule "); printf("Ne rentrez qu'une fois chaque lettre "); printf("Tant que vous n'avez pas trouve le mot "); printf("ou que vous n'etes pas mort, entrez des lettres "); printf("Le mot est P - - - - "); printf("A vous de jouer: "); while(success !=3 || erreur !=3 ) { scanf("%c", &lettre); if(lettre == 79 || lettre == 77 || lettre == 69) { if(lettre==79) { moti[1]=79; printf("Bien joue "); int i; for (i = 0 ; i < 5 ; i++) { printf("%c ",moti[i]); } success = success+1; } if(lettre==77) { moti[2]=77; moti[3]=77; printf("Bien joue "); int i; for (i = 0 ; i < 5 ; i++) { printf("%c ",moti[i]); } success = success+1; } if(lettre==69) { moti[4]=69; printf("Bien joue "); int i; for (i = 0 ; i < 5 ; i++) { printf("%c",moti[i]); } success = success+1; } if(success==3) { break; } } else { erreur = erreur+1; { if(erreur==1) { printf("rate "); printf("---------- "); } if(erreur==2) { printf("encore rate "); printf("______________ |/ | | | | | | _|__________ "); } if(erreur==3) { printf("perdu "); printf("______________ |/ | | @ | -|- | | | |^| | - - _|__________ "); break; } } } } } Le joueur doit deviner le mot pomme au joueur, si il ce trompe 3 fois on affiche différente image du pendu et si il donne les bonnes lettres, il gagne. 79, 77 et 69 représente les lettres O, M et E en ascii Il y a différent problème, quand on rentre 1 mauvaise réponse, le programme affiche les printf des if(erreur=1) et if(erreur=2), quand on rentre les 2 premieres reponses quelque soit l'ordre, ca affiche le mot avec la lettre en plus et aussi le raté. Je vois pas ce que j'ai mal fait, je vous remercie de votre spontanéité.

chris79 · Answer

Salut,

Ton problème vient du fait que lorsque tu rentres un caractère, tu rentres également un retour chariot de valeur 0x0A pour valider: c'est ce dernier caractère qui explique pourquoi tu as une boucle supplémentaire et donc pourquoi tu affiches une erreur. Ce comportement est classique à la fonction scanf et beaucoup se font avoir ;)

Une solution parmi d'autres serait de remplacer :
scanf("%c", &lettre); 
Par :
scanf(" %c", &lettre); 
(Ajout d'un espace avant %c).

Tu peux aussi te pencher vers d'autres fonctions fgets, getchar ou autres...mais il faudra tout de même gérer ce dernier caractère ;)

Sinon comme dit LOlock, un switch case serait tout de même plus joli ^^

fiddy · Answer

Bonjour, Différents conseils : scanf("%c", &lettre); Attention à l'utilisation du scanf("%c",...). Cela engendrera un retour chariot dans le buffer. Pour le vider, tu peux faire un simple getchar() : scanf("%c",&lettre); getchar(); mot[0]=80;//P mot[1]=79;//O mot[2]=77;//M mot[3]=77;//M mot[4]=69;//E Il vaut mieux faire : mot[0]='P'; etc. C'est plus lisible la manipulation du caractère que le code ascii... Sinon encore mieux :

char mot[]="POMME";

if(lettre==79)
{
moti[1]=79; 
[...]

Fonctionnellement c'est bon, mais difficilement maintenable. Il vaut mieux faire une boucle for sur chacune des lettres. Ainsi tu n'as plus qu'à changer le mot sans rien changer d'autres.

Je te conseille l'utilisation de fonctions pour rendre ton code plus lisible.

Enfin, n'oublie pas le return 0; à la fin du main().

Note : pour la prochaine fois, n'oublie pas d'utiliser la balise "code" pour poster ton code. Exemple :
ici tu mets ton code

Cdlt,

Quelqu'un qui si connait en code c pourrait m'aider svp

4 réponses

Discussions similaires