Tu función tal y como está escrita no tiene mucho sentido:
- Almacenas el nombre del evento en
d
(¿no podía llamarse mejor nombreEvento
o evento
?)
- Dado que no hay bucles en la función se espera que el programa tenga que llamar varias veces a la función
d
no es estática, luego el valor introducido en la primera llamada no estará disponible en la segunda
- Tampoco habilitas ningún mecanismo para que el valor de dicha variable salga de la función o pueda pasarse desde fuera (ni return ni argumentos).
- ¿Qué es
valorBase
? Su declaracion la has omitido en el código.
- ¿Cómo funciona el ingreso de datos?
- Las cadenas de salida no tienen saltos de línea ¿Has probado el código en algún momento?
- Las opciones del
switch
no están separadas por un break
. ¿Se te han olvidado?
Vamos por partes.
Si la función no se ha de llamar varias veces, entonce te falta un bucle en la función:
void evento()
{
int opcion;
do
{
cout << "BIENVENIDO\n"
<< "1-Crear un evento\n"
<< "2-Ingresar datos a evento nuevo\n"
<< "3-Salir\n";
switch (opcion)
{
// ...
}
} while( opcion != 3 );
}
Si resulta que no, que se ha de llamar varias veces entonces lo lógico es que reciba vía argumentos los datos que necesita para funcionar. En cualquier caso (y esto es aplicable también al ejemplo anterior) tendrías que indicar qué sucede si el usuario introduce un valor fuera del rango (1,2)
void evento(char* nombreEvento, /* ... */)
{
int opcion;
cout << "BIENVENIDO\n"
<< "1-Crear un evento\n"
<< "2-Ingresar datos a evento nuevo\n";
switch (opcion)
{
// ...
case default:
/* ¿Qué hacer aquí? */
}
}
Si resulta que la función ha de ser autocontenida y se va a llamar varias veces (no se me ocurre el motivo, pero viendo el estado del código tampoco se puede adivinar demasiado), entonces d
debería ser estática para conservar su valor entre diferentes llamadas:
void NuevoEvento(char* evento, char* valor)
{
cout << "Usted va a crear un nuevo evento\n"
<< "Por favor,indique el nombre del evento a crear";
cin >> ptr;
cout << "Por favor, digite el valor de la boleta base (Sin comas ni puntos)";
cin >> valor;
cout << "Evento creado con exito";
}
void evento(){
int opcion;
static char d[10000] = '\0';
cout<<"BIENVENIDO";
cout<<"1-Crear un evento";
cout<<"2-Ingresar datos a evento nuevo";
switch (opcion){
case 1:
NuevoEvento(d,valorBase);
break; // ¿Seguro que no es necesario?
case 2:
if( *d == 0 )
cout << "Primero debes crear el evento\n";
else
IntroducirDatosEvento(d);
break;
}
}
Me he tomado la molestia de sacar el código fuera de la función por simple legibilidad. Los switch
ya pueden ser demasiado problemáticos como para ir tentando al diablo.