Jeu des allumettes : Code bon ou pas

Tu peux déjà reposter un code avec les commentaires qui devraient y figurer : mets-toi à la place d'un dev qui lit ton code(ce n'est pas toujours aussi simple, il faut expliquer ce que ça fait et aussi pourquoi cette méthode).
C'est le b-a-ba pour faire "bien et propre".

donc ça donnerais à peut près ça?

#include <iostream>
#include <string>

int main();               /*declaration des variables */
int menu(); 
int configjeu();
int jeu();
int regle();
int pCondition1 = 1, pCondition2 = 3, x, y, nombreAllumettes=21;
std::string joueur1; std::string joueur2;
int main(){
 return menu();
}

int menu(){
 system ("cls");         /*menu  du jeu*/
 int selecteur;  nombreAllumettes=21;
  std::cout<<"\t ******************"<<std::endl;
 std::cout<<"\t Jeu des Allumettes"<<std::endl;
 std::cout<<"\t ******************"<<std::endl;
 std::cout<<std::endl;
 std::cout<<"0.sortir du programme."<<std::endl;
 std::cout<<"1.jouer en mode classic."<<std::endl;
 std::cout<<"2.configurer les parametres."<<std::endl;
 std::cout<<"3.lire les regles de jeu."<<std::endl;
 std::cout<<"choix :"; std::cin>>selecteur;
 switch(selecteur){
  case 0: exit(1); break; 
  case 1: return jeu(); break; 
  case 2: return configjeu(); break;
  case 3: return regle(); break;
  default: std::cout<<"votre doigt à peut-etre glisser"<<std::endl ;return menu();
 }}

int jeu(){
 std::cout<<"nom du joueur 1"<<std::endl; std::cin>>joueur1;    /*jeu*/ 
 std::cout<<"nom du joueur 2"<<std::endl; std::cin>>joueur2;
 system ("cls"); 
 std::cout<<"\t ******************"<<std::endl;
 std::cout<<"\t jeu des allumettes"<<std::endl;
 std::cout<<"\t ******************"<<std::endl;
 std::cout<<std::endl;std::cout<<std::endl;
 std::cout<<"le jeu commence avec "<<nombreAllumettes<<std::endl;
  while(nombreAllumettes>0)                
  {
   std::cout<<joueur1<<" saisir un nombre"<<std::endl; std::cin>>x;

if(x>pCondition2 || x<pCondition1)
    {
     std::cout<<"nombre saisi invalide"<<std::endl;
    }
   else nombreAllumettes=nombreAllumettes-x;
    std::cout<<"le nombre d'allumettes est de :"<<  nombreAllumettes<<std::endl;

std::cout<<joueur2<<" saisir un nombre"<<std::endl; std::cin>>x;

if(x>pCondition2 || x<pCondition1)
    {
     std::cout<<"nombre saisi invalide"<<std::endl;
    }
   else nombreAllumettes=nombreAllumettes-x;
    std::cout<<"le nombre d'allumettes est de :"<<  nombreAllumettes<<std::endl; 
   if (nombreAllumettes<=0)
   {
     std::cout<<std::endl;
     std::cout<<"Il n'y a plus d'allumettes, c'est fini !!!";
     std::cout<<" Il n'y a plus d'allumettes, c'est fini !!! ";
     std::cout<<" il n'y a plus d'allumettes, c'est fini !!!"<<std::endl;;
   }
   }
      std::cout<<std::endl;
  system ("pause");return menu();

}
int configjeu(){
 system ("cls");              /*menu de configuration*/
 std::cout<<"\t ****************************"<<std::endl;   
 std::cout<<"\t configuration des parametres"<<std::endl;
 std::cout<<"\t ****************************"<<std::endl;
 std::cout<<std::endl;
 std::cout<<"entrer le nombre d'allumettes :"; std::cin>>nombreAllumettes;
 std::cout<<"entrer la condition minimum :"; std::cin>>pCondition1;
 std::cout<<"entrer la condition maximum :"; std::cin>>pCondition2;
 system ("pause");
 return jeu();
 }
int regle(){
 std::cout<<"\t ************"<<std::endl;        /*regle du jeu..*/
 std::cout<<"\t regle du jeu"<<std::endl;
 std::cout<<"\t ************"<<std::endl;
 std::cout<<std::endl;
 std::cout<<"les regles sont (en mode classic) :"<<std::endl;
 std::cout<<"\t -le but du jeu est de NE PAS retirer la derniere allumette"<<std::endl;
 std::cout<<"\t -vous n'avez le droit de ne retirer au minimum 1 et au maximum 3 allumettes"<<std::endl;
 std::cout<<std::endl; 
 system ("pause");
 std::cout<<"les conditions designent le nombre d'allumettes que l'on peut retirer à chaque tour"<<std::endl;
 std::cout<<std::endl;
 system ("pause");
 return menu();
 }

Oui,

goto

est même impossible ici.
La solution consiste à créer une fonction

menu()

sous la forme d'une boucle. Les autres fonctions n'auront qu'à faire un simple

return

, qui nous ramènera dans la boucle de la fonction

menu()

. Le

return

de la fonction

menu()

sera lui écrit à la place de l'appel à

exit()

.

heu je crois que tu m'as perdu la...
j'ai pas trop compris comment on crée la fonction menu sous la forme d'une boucle.

Comme cela:

int menu(){
   for(;;) {
      system ("cls");
      int selecteur;
      nombreAllumettes=21;
      ......
      std::cout<<"choix :"; std::cin>>selecteur;
      switch ( selecteur ) {
         case 0: return 0;
         case 1: jeu(); break; 
         case 2: configjeu(); break;
         case 3: regle(); break;
         default: std::cout<<"votre doigt à peut-etre glisser"<<std::endl;
      }
   }
}

ok merci, je vais tacher de corriger ça :]