Tienes unos cuantos problemas: no usas bien los argumentos, scanf( )
necesita punteros a los datos, las cadenas han de ir siempre encerradas entre comillas dobles, los cambios en tus variables de entrada son locales a la función y no son visibles desde fuera, ...
Para lo que pretendes hacer, no hay ninguna necesidad de usar un switch( )
:
const char *pedir_signo( ) {
static const char *SIGNOS[] = {
"aries",
"tauro",
"geminis",
"cancer",
"leo",
"virgo",
"libra",
"escorpio",
"sagitario",
"capricornio",
"acuario",
"piscis"
};
int num;
printf( "Ingresa el numero de orden de tu signo zodiacal:\n" );
scanf( "%i", &num );
return SIGNOS[num - 1];
}
Si has de usarlo por fuerza, y no puedes devolver nada desde la función, pues tienes que realizar varios cambios también:
#include <stdio.h>
void pedir_signo( const char **signo ) {
int num;
printf( "Ingresa el numero de orden de tu signo zodiacal:\n" );
scanf( "%i", &num );
switch( num ) {
case 1:
*signo = "aries";
break;
case 2:
*signo = "tauro";
break;
case 3:
*signo = "geminis";
break;
case 4:
*signo = "cancer";
break;
case 5:
*signo = "leo";
break;
case 6:
*signo = "virgo";
break;
case 7:
*signo = "libra";
break;
case 8:
*signo = "escorpio";
break;
case 9:
*signo = "sagitario";
break;
case 10:
*signo = "capricornio";
break;
case 11:
*signo = "acuario";
break;
case 12:
*signo = "piscis";
break;
}
}
int main( void ) {
const char *signo;
pedir_signo( &signo );
printf( "Signo introducido: %s\n", signo );
return 0;
}
Fíjate en el uso de variables const
, para indicar que no vas a modificar los datos apuntados por el puntero. De no hacerlo así, obtendrías errores de compilación, y para evitarlos sería necesario usar memoria dinámica y realizar copias de las cadenas con los signos.
Además, el argumento de la función pasa a ser un puntero-a-puntero.