Dos observaciones:
scanf("%s", ...)
lee cualquier número de caracteres deteniéndose en el primer carácter en blanco que encuentre. Al terminar agrega un carácter nulo \0
al final de la secuencia.
De esto extraemos un par de problemas:
scanf
dejará de leer cuando encuentre un espacio, dejando el resto de la cadena en el buffer. Si ingresas una cadena con un espacio, tu primer scanf
lee hasta encontrarlo y almacena eso en nombre
. Tu segundo scanf
continua leyendo lo que se dejó el primero en el buffer
(hasta encontrar un carácter en blanco) y lo almacena en n_ord
.
scanf
es inseguro por naturaleza. Primero debes tener en cuenta que si tienes un array de 6 caracteres no puedes almacenar una cadena de 6 caracteres, debes reservar uno para el carácter nulo. Es más, si el usuario ingresa más caracteres de los que el array puede contener a scanf
le da exactamente lo mismo, por lo que termina escribiendo (o intentándolo) en memoria que no le pertenece (overflow).
Aunque ambos problemas se pueden paliar (por ejemplo, podemos limitar el número de caracteres que debe leer: scanf("%5s", n_ord);
), te aconsejo usar fgets
si quieres poder ingresar cadenas con espacios y protegerte del desbordamiento de memoria.
fputs((nombre), archivo),("\n");
no hace lo que esperas, que es agregar el carácter de nueva línea al final de la cadena. Date cuenta que ("\n")
está fuera de la llamada a la función, por lo que no tiene efecto alguno en cuanto al archivo. Por otro lado los paréntesis alrededor de nombre
son totalmente superfluos en este caso.
Para agregar una nueva línea tienes varias opciones:
- Llamar a
fputs("\n", archivo)
después de agregar la linea.
- Llamar a
fputc('\n', archivo)
después de agregar la línea.
El código podría quedar así:
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
int main() {
char nombre[41];
char n_ord[7];
printf( "Introduzca su nombre (40 letras máximo): " );
fgets(nombre, 41, stdin);
if ((strlen(nombre) > 0) && (nombre[strlen(nombre) - 1] == '\n'))
nombre[strlen(nombre) - 1] = '\0';
// Si el usuario ingresó más de 40 caracteres debemos limpiar el buffer.
char t;
while ((t=getchar()) != '\n' && t != EOF){
}
printf( "Introduzca el numero de orden (6 digitos máximo): " );
fgets(n_ord, 7, stdin);
if ((strlen(n_ord) > 0) && (n_ord[strlen(n_ord) - 1] == '\n'))
n_ord[strlen(n_ord) - 1] = '\0';
FILE* archivo;
archivo = fopen("ordennum.txt", "wt");
fputs(nombre, archivo);
fputc('\n', archivo);
fputs(n_ord, archivo);
fputc('\n', archivo);
fclose(archivo);
printf("Proceso completado");
return 0;
}
fgets
, si no se ha superado el tamaño asignado, deja de leer ante el primer carácter de nueva línea que encuentre, pero lo consume y lo incluye en la cadena. Es decir, ante:
char num[6];
fget(num, 6, stdin);
si ingresamos:
123456789
num
queda como:
12345\0
si ingresamos:
123
num
queda como:
123\n\0
Para eliminarlo hago uso de:
if ((strlen(n_ord) > 0) && (n_ord[strlen(n_ord) - 1] == '\n'))
n_ord[strlen(n_ord) - 1] = '\0';
cierto es que se puede recurrir a:
strtok(n_ord,"\n");
el problema es que falla (mejor dicho, no elimina nada) si la cadena solo contiene \n
, por ejemplo si el usuario ingresa una cadena vacía.
Lo ideal es meter el código encargado de leer cada entrada y de limpiar el buffer en una función para evitar repetir código.