Probleme de logique de boucle dans une boucle

Version imprimable

Bonjour,
Je suis entrain de faire le tp mot mystère d'un cour que je suis sur le c++.

Je vous présente le code
Code:

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 #include <iostream> #include <string> int main() { std::cout << "Veuillez donner un mot : " << std::endl; std::string motUser,motJoueur; std::cin>>motUser; int essaie(5); char choixUser; //do //{ if(essaie>0) { do { std::cout<<"Quel est ce mot ?"<<std::endl; std::cin>>motJoueur; if(motUser==motJoueur) { std::cout<<"Bravo"<<std::endl; std::cout<<"Voulez vous rejouer une partie ? (y/n)"<<std::endl; std::cin>>choixUser; } else (essaie>0) { essaie-=1; std::cout<<"Ce n'est pas le mot il vous reste : "<<essaie<<" essaies."<<std::endl; } }while(motUser!=motJoueur&&essaie>0); } else { std::cout<<"PERDU le mot est : "<<motUser<<std::endl; std::cout<<"Voulez vous rejouer une partie ? (y/n)"<<std::endl; std::cin>>choixUser; } //}while(choixUser!='n'); return 0; }
Mon objectif est de demander à l'utilisateur de deviner le mot.
Il à 5 essaies pour le faire au bout des 5 essaies la partie est perdu et l'on affiche le mot.
Mon bout de programme à l'heure actuelle fonctionne partiellement.
Si la personne trouve la bonne réponse avant les 5 essaies. Le bon message s'affiche
SI la personne use les 5 essaies la boucle s’exécute 5 fois avec le décompte afficher comme voulu.
Seulement une fois le dernier essaie effectuer le programme s’arrête sans afficher le message PERDU.
Et je bute depuis plusieurs heures sur le problème au point de me dire que je ne suis peut être pas en mesure d'apprendre un langage de programmation.
Je suis sur que la solution est toute simple et je passe a cote depuis plusieurs jours.
Si une personne bienveillante voulais bien me donner une direction une lumière ou m'achever en me disant de plutôt faire du tricot.
Merci

Bonjour,

Quand on a du mal avec un problème algorithmique, il faut essayer de décomposer le problème en sous-problèmes plus simples.

Voici une solution de l'exercice qui illustre cette décomposition :
[EDIT : Si tu veux juste un indice et pas la solution entière, lis seulement le contenu de la fonction main() et ignore la suite du code.]
Code:

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 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 #include <iostream> #include <string> // Déclarations en avance pour que ça compile. void jouerUnePartie(); bool demanderSiLeJoueurVeutRejouer(); std::string initialiserMotAChercher(); std::string demanderMotAuJoueur(); void direQuelCeNEstPasLeBonMot(int nbEssaisRestants); void afficherResultat(bool victoire, const std::string& motAChercher); int main() { bool rejouer; do { jouerUnePartie(); rejouer = demanderSiLeJoueurVeutRejouer(); } while(rejouer == true); return 0; } void jouerUnePartie() { const std::string motAChercher = initialiserMotAChercher(); bool motTrouve = false; int nbEssaisRestants = 5; while(nbEssaisRestants > 0 && motTrouve == false) { --nbEssaisRestants; const std::string mot = demanderMotAuJoueur(); motTrouve = (mot == motAChercher); if(nbEssaisRestants > 0 && motTrouve == false) direQuelCeNEstPasLeBonMot(nbEssaisRestants); } afficherResultat(motTrouve, motAChercher); } bool demanderSiLeJoueurVeutRejouer() { std::cout << "Voulez-vous rejouer une partie ? (y/n)\n"; char choixUser; std::cin >> choixUser; return (choixUser == 'y'); } std::string initialiserMotAChercher() { std::cout << "Quel mot le joueur doit-il chercher ?\n"; std::string motAChercher; std::cin >> motAChercher; return motAChercher; } std::string demanderMotAuJoueur() { std::cout << "Quel est ce mot ?\n"; std::string motJoueur; std::cin >> motJoueur; return motJoueur; } void direQuelCeNEstPasLeBonMot(int nbEssaisRestants) { std::cout << "Ce n'est pas le mot. Il vous reste " << nbEssaisRestants << " essai(s).\n"; } void afficherResultat(bool victoire, const std::string& motAChercher) { if(victoire == true) std::cout << "Bravo !\n"; else std::cout << "PERDU. Le mot est : " << motAChercher << '\n'; }
Dans ton algo, tu t'étais emmêlé avec le fait de rejouer une partie et le fait de réessayer à trouver le mot.
Pour construire l'algorithme, il faut les gérer séparément.
Dans le code de ma fonction main(), on voit l'algorithme correspondant à la possibilité de rejouer une partie. Mais tout ce qui correspond à la partie elle-même est séparé dans une fonction à part, jouerUnePartie().

Bonne chance pour la suite.
Avec une décomposition en sous-problèmes, ça devrait aller.
Et avec l'habitude, ça te paraîtra plus simple.

Bonjour

Merci pour l'ensembles des informations que vous m'avez fourni.
j'ai ecrit le code d'une autre manière qui fonctionne et me parraisait un peu plus accessible à mon niveau.
Je vous le présente ici
main.cpp

Code:

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
#include <iostream>
#include "motmelange.hh"
 
int main()
{
    srand(time(0));
    std::string motMystere,motMelange,motUser;
    std::cout << "Veuillez choisir un mot : " << std::endl;
    std::cin>>motMystere;
    for(int i(0);i<45;i++)
    {
        std::cout<<std::endl;
    }
    motMelange=melangeLettres(motMystere);
    char choisUser;
    do
    {
    int essaies(5);
    while(essaies>0)
    {
        std::cout<<"Quel est le mot : "<<motMelange<<std::endl;
        std::cin>>motUser;
        if(motUser==motMystere)
        {
            std::cout<<"Bravo"<<std::endl<<"Voules vous refaire une partie ? (y/n)"<<std::endl;
            std::cin>>choisUser;
            break;
        }
        else
        {
            --essaies;
            std::cout<<"Ce n'est pas le mot il vous reste : "<<essaies<<" essaies"<<std::endl;
        }
    }
    if(essaies==0)
    {
        std::cout<<"PERDU le mot est : "<<motMystere<<std::endl<<"Voules vous refaire une partie ? (y/n)"<<std::endl;
        std::cin>>choisUser;
    }
    }while(choisUser!='n');
    return 0;
}

melangerLettre.cpp

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
#include "motmelange.hh"
 
std::string melangeLettres(std::string mot)
{
    int position;
    std::string motRecu;
    while(mot.size()!=0)
    {
        position=rand()%mot.size();
        motRecu+=mot[position];
        mot.erase(position,1);
    }
    return motRecu;
}

Code:

1
2
3
4
5
6
7
8
9
10
header.hh
#ifndef MOTMELANGE_HH_INCLUDED
#define MOTMELANGE_HH_INCLUDED
#include <string>
#include <ctime>
#include <cstdlib>
 
std::string melangeLettres(std::string mot);
 
#endif // MOTMELANGE_HH_INCLUDED

Mais je garde en tête votre méthode qui me sera peut être nécessaire pour la suite du développement.