[Débutant] Optimisation d'un menu

Optimisation d'un menu [Débutant] - C++ - Programmation

Marsh Posté le 29-11-2004 à 09:52:50    

bonjour,
 
je débute dans le c++ et j'ai un probleme avec un menu.
voici mon code:
 

Code :
  1. #include <iostream>
  2. #include <string>
  3. #include <map>
  5. using namespace std;
  6. int main()
  7. {
  8.   map<string, string> repertoire;
  9.   map<string, string>::iterator it;
  10.   string nom, num;
  11.   int choix;
  12.   do {
  13.     system("cls" );
  14.     cout << "------------------------------" << endl;
  15.     cout << "Répertoire telephonique" << endl;
  16.     cout << "------------------------------" << endl;
  17.     cout << " Nouvelle personne ..........1" << endl;
  18.     cout << " Enlever une personne .......2" << endl;
  19.     cout << " Rechercher un numéro .......3" << endl;
  20.     cout << " Rechercher une personne.....4" << endl;
  21.     cout << " Afficher le repertoire......5" << endl;
  22.     cout << " Quitter l'application ......0" << endl;
  23.     cout << "------------------------------" << endl;
  24.     cout << "Votre choix : ";
  25.     cin >> choix;
  26.     if (choix < 0 || choix > 5) {
  27.     cout << "Choix de la rubrique invalide !" << endl;
  28.     cout << "Saisir un numéro entre 0 et 5 !" << endl;
  29.     system ("pause" );
  30.     }else{
  31.     switch (choix)
  32.     {
  33.       case 1 : cout << "Saisissez le nom de la personne : ";
  34.                cin >> nom;
  35.                cout << "Numéro de téléphone : ";
  36.                cin >> num;
  37.                repertoire[nom] = num;
  38.                break;
  39.       case 2 : cout << "Saisissez le nom de la personne : ";
  40.                cin >> nom;
  41.                repertoire.erase(nom);
  42.                break;
  43.       case 3 : cout << "Saisissez le nom de la personne : ";
  44.                cin >> nom;
  45.                for (it = repertoire.begin(); it != repertoire.end(); it++)
  46.                if (it->first == nom) {
  47.                cout << it->first << " possède le numéro " << num << endl;}
  48.                else
  49.                cout << "cette personne n'existe pas !"<< endl;
  50.                system ("pause" );
  51.                break;
  52.       case 4 : cout << "Saisissez le numéro de téléphone : ";
  53.                cin >> num;
  54.                for (it = repertoire.begin(); it != repertoire.end(); it++)
  55.                if (it->second == num) {
  56.                cout << it->first << " possède le numéro " << num << endl;}
  57.                else
  58.                cout << "pas de personne a ce numero !"<< endl;
  59.                system("pause" );
  60.                break;
  61.       case 5 : cout << "Il y a " << repertoire.size() << " personnes dans le repertoire. " << endl;
  62.                   if (repertoire.size()!=0){
  63.                cout << "Liste des personnes : " << endl;
  64.                for (it = repertoire.begin(); it != repertoire.end(); it++)
  65.                cout << '|' << it->first << ", " << it->second << '|' << endl;
  66.                } else
  67.                cout << "Liste vide" << endl;
  68.                system ("pause" );
  69.       }
  70.     }
  71.   }
  72.   while (choix);
  73.   return 0;
  74. }


 
je voudrai optimiser mon menu.
j'aimerai bien faire en sorte que quand je tape des lettres pour choisir la rubrique, j'ai le message d'erreur comme dans mon code.
 
D'autre part, je voudrai pouvoir gerer les espaces car si par exemple je rajoute une personne de type NOM[espace]NOM, il mémorise le premier nom comme il faut mais le deuxieme nom en tant que numero. Supposons une mauvaise manip et je peux me retrouver dans mon repertoire avec (exemple) jean comme nom et pierre comme numero.
 
Comment gerer ce probleme.
 
Je vous remercie d'avance.

Reply

Marsh Posté le 29-11-2004 à 09:52:50   

Reply

Marsh Posté le 30-11-2004 à 19:38:56    

Salut !  :)  
 
Si tu veut afficher ton message d'erreur lorque tu tapes tout autre chose qu'un chiffre entre 0 et 5, remplace ta condition :
 
 if (choix < 0 || choix > 5) ----> if( !((choix <= 5) && (choix >= 0)) )
 
En faite, il faut prendre le problème dans l'autre sens. (n'oubli pas le "!" ).
 
En effet, je crois que cin ne gère pas les espaces. Utilise la fonction getline (par contre je crois quelle utilise des char donc il faudra réliser un transtypage par la suite).
 
Voilà.
 
@+.  :pt1cable:

Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed