Menu c++

[kireta]

Hola hola!!!
Bueno primero que todo gracias por siempre responder a mis preguntas, lo segundo es que es mi primera vez programando en C++, tengo una duda, soy medio autodidacta y pues me tope con un problema tengo este codigo:
El problema? no se sale del menu tras haberle dado la opcion 3... TT_TT

Código:

#include <stdio.h>
#include <iostream.h>
#include <conio.h>
#include <string.h>

char convoc();
int nac();
int menu();

char *e;
int b,c,ac=2009, ma=5, da=21, an, mn, dn, edad;

char convoc()
{
   clrscr();
   cout<<"Escriba palabra: ";
   cin>>e;
   b = strlen(e);
   for (int i = 0; i < b; i++)
   {
    if ((e[i]=='a')||(e[i]=='e')||(e[i]=='i')||(e[i]=='o')||(e[i]=='u'))
      {
	c = c+1;
      }
   }
  cout<< "Se encontraron "<<c<<" vocales";
  getch();
  void main();
  return 0;
}

int nac()
{
 clrscr();
 cout<< "Digite el dia de su nacimiento: ";
 cin>>dn ;
 cout<< "\nDigite el mes de nacimineto: ";
 cin>>mn;
 cout<< "\nDigite el a¤o de nacimineto: ";
 cin>>an;
 if (mn<=ma)

   {
     if (dn>=da)
     {
       edad = ac-an;
       cout<< "Tiene "<<edad <<" a¤os ";
     }
     else
     {
      edad= ac-1-an;
      cout<< "Tiene "<<edad <<" a¤os ";
     }
   }
   else
   {
    edad = ac-1-an;
    cout<< "Tiene "<<edad <<" a¤os ";
   }
 cout<<"\n";
 getch();
 void main();
 return 0;
}

int menu()
{
 clrscr();
 int opc=0, temp=0;
 cout<< "1.Conteo de Vocal\n";
 cout<< "2.Calculo de edad\n";
 cout<< "3.Salir\n ";
 cout<< "Digite opcion ";
 cin>>opc;

 while ((opc != 3)&&(temp != 3))
 {
  switch (opc)
   {
    case 1: convoc(); menu(); break;
    case 2: nac(); menu(); break;
    case 3: opc = 3; temp = 3; break;
   }
  }
 return 0;
}

void main()
{
 clrscr();
 menu();
}

[fradve]

Hola,

En tu codigo veo que la función menú hace uso de recursividad (se llama a si misma), lo cual veo inncesario. Porque no haces el proceso del bucle en el main o hazlo todo dentro de la función pero que devuelva void, no es necesaria la asignación del tecer case, porque dicho valor ya lo ingresaste por teclado.

Tu función está así:

Código c++:

int menu()
{
 clrscr();
 int opc=0, temp=0;
 cout<< "1.Conteo de Vocal\n";
 cout<< "2.Calculo de edad\n";
 cout<< "3.Salir\n ";
 cout<< "Digite opcion ";
 cin>>opc;
 
 while ((opc != 3)&&(temp != 3))
 {
  switch (opc)
   {
    case 1: convoc(); menu(); break;
    case 2: nac(); menu(); break;
    case 3: opc = 3; temp = 3; break;
   }
  }
 return 0;
}
 

con el primer cambio que te comente sería:

Código c++:

int menu()
{
 clrscr();
 int opc=0;
 cout<< "1.Conteo de Vocal\n";
 cout<< "2.Calculo de edad\n";
 cout<< "3.Salir\n ";
 cout<< "Digite opcion ";
 cin>>opc;
 return opc;
}
 
void main()
{
  int op=0;
 while (op != 3)
 {
  clrscr();
  op=menu();
  switch (op)
   {
    case 1: convoc(); break;
    case 2: nac();
   }
  }
}
 

La segunda forma sería:

Código c++:

void menu()
{
  int opc=0;
 
 while (opc != 3)
 {
    clrscr();
    cout<< "1.Conteo de Vocal\n";
    cout<< "2.Calculo de edad\n";
    cout<< "3.Salir\n ";
    cout<< "Digite opcion ";
    cin>>opc;
 
    switch (opc)
    {
     case 1: convoc(); break;
     case 2: nac();
    }
 }
}
 
void main()
{
   menu();
}
 

Espero que te sea de mucha ayuda.

Saludos

[kireta]

Hola muy agradecida por tu respuesta voy a tratar de hacerlo de la primera forma por que ya habia hecho de la segunda, en realidad le agrege temp como para forzar la salida pero nada mas, gracias nuevamente, te aviso cuando haya provado, gracias de nuevo!

[kireta]

No entiendo por que funciono...