• # mes remarques

    Posté par . En réponse au message Relancement de mon jeu en c++ avec while. Évalué à 2.

    Les magic constant (ou constante magique c'est mal, si plus tard tu veux passer à 42 essais, faut changer tout les 10 qui trainent

    • avoir une constante const int NB_ESSAIS=10; et initialiser essai comm suit essai=NB_ESSAI
    • le premier paramètre du for ne sert à rien, ça devrait être int essai=NB_essai (et pour le coup ne plus l'avoir déclaré avant
    • avoir des else après les if (si il est == il ne sera pas > ni < )
    • faire une fonction, le code dans le main c'est mal, avoir la gestion du jeu et des nouvelle partie pourquoi pas, mais le reste devrait être une fonction.

    typiquement un 

    do{
 if( !rechercheMystere(mysterieuCaractere(), NB_ESSAI) )
 {
 std::cout << "vous avez lamentablement échoué; voulez vous retenter votre chance afin de tenter de sauver"
 << "l'ego ridicule qui vous reste? o/n"
 std::cin >> rejoue
 }
 else 
 {
 rejoue='n';
 }
}while(rejoue == 'o');

    devrait faire l'affaire

    Il ne faut pas décorner les boeufs avant d'avoir semé le vent