Error #1 Los strings no necesitan referencia
char nombre[24];
scanf("%s", & nombre);
scanf
necesita que se le pasen punteros a las variables, eso le permite modificar el valor de tus variables. Lo que sucede es que un array char[]
ya es un puntero en si mismo, luego no necesita que uses referencias en este caso. El uso correcto sería:
scanf("%s", nombre);
Error #2 Usa los modificadores adecuados a cada tipo de variable.
char sexo;
scanf("%d",&sexo);
En este caso sexo
es de tipo char
pero tu le dices a scanf
que lea un entero. Aquí tenemos un problema bastante gordo, ya que sexo
ocupa únicamente un byte, mientras que el tipo int
ocupa 4. Esto quiere decir que el programa sobreescribirá 3 bytes que no pertenecen a tu variable. Puedes estar modificando otros valores sin darte cuenta.
Si sexo
es char
, entonces lee un char
:
scanf("%c", &sexo);
Ahora bien, esta línea que te acabo de poner introducirá un nuevo error, y es que el programa aparentará hacer caso omiso a esta línea y avanzará sin esperar a que introduzcas ningún valor. Esto se debe a que la lectura anterior, la del string, deja un salto de línea en el buffer de entrada ... y ese mismo salto de línea es el que se va a almacenar en sexo
. Para evitar esto hay que dejar un espacio antes de la lectura:
scanf(" %c", &sexo);
De esta forma, scanf
descartará cuantos saltos de línea, tabuladores o espacios haya antes del dato que tu vas a introducir.
Esto no sucede cuando lees otros tipos de datos porque la función sobreentiende que, por ejemplo, un entero no va a empezar por un espacio. Sin embargo cuando quieres leer un caracter, la función no puede decidir por ti si te interesa o no el salto de línea que puede encontrarse ahí
%d
no es el más adecuado para obtener y almacenar un dato de tipochar
, más bien es para números enteros.