Introducir un nombre en un vector

[Qorin]

Veréis, necesito leer un nombre e introducirlo a un vector, son las condiciones del proyecto, no nos dejan usar la librería strings. Os muestro mi función leer_alumno y me decís que puede pasar, luego os muestro un ejemplo de ejecución.

Código (c) [Seleccionar] Expandir


void leer_persona (tlista *p)
{
   talumno alumno;
    p->num_pers = 0;
   int i, j = 0, n;
   printf ("\nIntroduce la cantidad de alumnos que quieres guardar. Recuerda que el maximo es 200 alumnos: ");
   scanf("%d", &n);
   if (n < MAXALUMNO) {// Se ha de cumplir que el número de alumnos a introducir sea menor o igual a 200
       
     for (i = 0; i < n; i++)
         {
             printf("\nInformacion del ALUMNO%d\n", i+1);
             printf("Introduce informacion con el siguiente formato(Nombre_apellidos:DNI 00000000X:anyo_ingreso):\n");  
             //ENCONTRAR OTRA MANERA DE LEER EL NOMBRE
     do{
                scanf("%c", &alumno.nombre[j]);
                j++;
             }while (alumno.nombre[j-1] != ':' && j-1 < 20);
             scanf("%8d%1c:%4d", &alumno.num_dni, &alumno.letra_dni, &alumno.fecha_ingreso);
     printf("\n");
             p->lista[i] = alumno;
         } p->num_pers = n;
   }
   else {printf("El máximo de alumnos que puedes introducir es 200.\n");}
}

La variable tlista p está inicializada en el main (), el programa es un menú con diferentes opciones, y en cada case va una función. Un ejemplo de ejecución para que veáis que es lo que puede fallar:

Código [Seleccionar] Expandir


BIENVENIDOS AL PROGRAMA DE GESTIÓN DE EXPEDIENTES      

Escoge una operación a realizar:

1. Introducir los datos de los expedientes.
2. Mostrar todos los datos de los expedientes.
4. Añadir o modificar el expediente de un alumno.
5. Eliminar el expediente de un alumno a partir de su DNI.
6. Mostrar los datos (Nombre, DNI y fecha de ingreso) de los alumnos que más tiempo llevan en la escuela.
7. Mostrar el expediente de un alumno a partir de su DNI.
13. Salir del programa.

Su opción es:1

Introduce la cantidad de alumnos que quieres guardar. Recuerda que el maximo es 200 alumnos: 2

Informacion del ALUMNO1
Introduce informacion con el siguiente formato(Nombre_apellidos:DNI 00000000X:anyo_ingreso):
alejandro:39427559Q:1992


Informacion del ALUMNO2
Introduce informacion con el siguiente formato(Nombre_apellidos:DNI 00000000X:anyo_ingreso):
fernando:11111111Q:1993

Escoge una operación a realizar:

1. Introducir los datos de los expedientes.
2. Mostrar todos los datos de los expedientes.
4. Añadir o modificar el expediente de un alumno.
5. Eliminar el expediente de un alumno a partir de su DNI.
6. Mostrar los datos (Nombre, DNI y fecha de ingreso) de los alumnos que más tiempo llevan en la escuela.
7. Mostrar el expediente de un alumno a partir de su DNI.
13. Salir del programa.

Su opción es:2

alejandro:39427559Q:1992
:11111111Q:1993


Como vereis, cuando uso la función mostrar_datos, que muestra todos los expedientes guardados en la lista, el nombre del segundo no me lo muestra, creo que es porque ya directamente no entra en el while porque se cumple la condición de != ':' . Así que deduzco que el error viene por la función leer_alumnos y en concreto en la parte en la que leo el nombre del alumno y lo introduzco al vector.

Por si es necesario, la función mostrar datos es la siguiente:

Código (c) [Seleccionar] Expandir

talumno mostrar_datos(tlista *p){
                            int i, k=0;
                            for ( i = 0; i < p->num_pers ; i++) {
                                while (p->lista[i].nombre[k] != ':'){
printf("%c", p->lista[i].nombre[k]);
k++;
}
                                printf(":%d%c:%d\n", p->lista[i].num_dni, p->lista[i].letra_dni, p->lista[i].fecha_ingreso); //Mostrar DNI y fecha de ingreso
                               
} // Cerramos el for
printf("\n");
                                } //Cerramos la función

Siento si no me he sabido explicar del todo bien, la asignatura no se me da mal, este es el primer proyecto importante que tengo y me gustaría que me saliera bien, si pudierais ayudarme con esta pequeña duda sería muy importante, porque me permitiría avanzar.

[Littlehorse]

Postea el código completo (en lo posible en pastebin).

A simple vista, los errores podrian provenir del mal uso de scanf.
Al hacer esto:

Código (c) [Seleccionar] Expandir

scanf("%d", &n);


En el buffer de entrada quedan datos (como el salto de linea, por ejemplo) que son leidos en la proxima lectura.

Luego, también hay otras cosas, como por ejemplo esto:

Código (c) [Seleccionar] Expandir

      do{
                 scanf("%c", &alumno.nombre[j]);
                 j++;
              }while (alumno.nombre[j-1] != ':' && j-1 < 20);

Una manera poco eficiente e insegura de leer un nombre. Te recomiendo que utilices fgets con stdin + sscanf.

Por ejemplo:

Código (c) [Seleccionar] Expandir

fgets (buff , size , stdin);
//Chequeas que la cadena este terminada en NULL y lo agregas en caso de ser necesario
//Procesas el buffer, por ejemplo para eliminar datos que no necesitas u caracteres invalidos
strncpy(_struct.nombre, buff, size); //Copias la cadena.


Podes hacer algo similar para otros tipos de datos:

Código (c) [Seleccionar] Expandir

fgets(buff, size, stdin);
//Procesas el buffer, por ejemplo para eliminar datos que no necesitas u caracteres invalidos
sscanf(buff, "%d", &varint);

Por supuesto en todos los casos también podes chequear valores de retorno de cada función para evitar posibles errores.

Saludos!

[Qorin]

Hola Littlehorse, gracias por prestarme un poquito de atención  , pero tengo que decirte que no nos dejan usar fgets ni sscanf...solamente : printf,scanf,for,while,do-while,if-else,vectores,matrices y funciones. Todo esto que te he dicho engloba el temario del cuatrimestre.

Me recomiendas que en todos los scanf añada un %*c para evitar posibles saltos de linea?

[Littlehorse]

Me recomiendas que en todos los scanf añada un %*c para evitar posibles saltos de linea?

Podes hacerlo, funcionar va a funcionar. Recomendar no te lo recomiendo, pero si no te dejan utilizar otras funciones mas que las que mencionaste, muchas alternativas no quedan.

Saludos

[do-while]

¡Buenas!

Si tienes problemas en tiempo de ejecucion, la clave esta en j. La inicializas al principio a cero, pero luego lo unico que haces es incementar su valor, asi que lo mas posible es que te salgas del rango del vector. Tienes que inicializarla a cero antes de la lectura de cada nombre.

¡Saludos!

[Qorin]

Ey chicos! gracias! lo que hice fue, en vez de usar 1 sola función, usar 2, leer_alumno y leer_lista, así que en parte do-while hice lo que has mencionado, inicializar la i a 0 en cada vuelta del for 

Gracias!