Verknüpfte Liste in C mit fgets()

Question

Ich habe Probleme mit meinem Code überspringe die erste Frage in der zweiten Datenstruktur. Ich bin mir ziemlich sicher, es ist, weil das(), aber nicht sicher ist. Ich glaube, ich habe versucht, fgets(), aber es gab mir immer noch Probleme. Warum?

#include <stdio.h> 
#include <stdlib.h> 
#include <string.h> 

#define NumberOfActresses 5 

typedef struct Actress 
{ 
char *name, *placeofbirth, *haircolor; 
int age; 
float networth; 
struct Actress *next; 

} Actress; 

void PopulateStruct(Actress *node) 
{ 
node->name = (char *) malloc(sizeof(char) * 50); 
node->placeofbirth = (char *) malloc(sizeof(char) * 50); 
node->haircolor = (char *) malloc(sizeof(char) * 50); 

printf("Please enter the name of the actress/actor: "); 
gets(node->name); 
printf("Please enter the actress/actor place of birth: "); 
gets(node->placeofbirth); 
printf("Please enter the actress/actor hair color: "); 
gets(node->haircolor); 
printf("Please enter the actress/actor age: "); 
scanf("%d", &node->age); 
printf("Please enter the actress/actor networth: "); 
scanf("%f", &node->networth); 
} 

void DisplayStruct(Actress *head) 
{ 
Actress *crawler; 

crawler = head; 

while(crawler != NULL) 
{ 
    printf("The name of the actress/actor is: %s\n", crawler->name); 
    printf("The place of birth for the actress/actor is: %s\n",  crawler->placeofbirth); 
    printf("The hair color of the actress/actor is: %s\n", crawler->haircolor); 
    printf("The actress/actor age is: %d\n", crawler->age); 
    printf("The networth for the actress/actor is %f\n", crawler->networth); 
    crawler = crawler->next; 
} 
} 

int main() 
{ 
int i; 
Actress *head = (Actress *) malloc(sizeof(Actress)), *crawler; 

crawler = head; 

for (i = 0; i < NumberOfActresses; i++) 
    { 
    PopulateStruct(crawler); 
    if (i == 2) 
     crawler->next = NULL; 
    else 
     crawler->next = malloc(sizeof(Actress)); 

    crawler = crawler->next; 
} 

crawler = NULL; 

DisplayStruct(head); 

return 0; 
} 

Answer 1

Misch fgets und scanf dreht sich immer schlecht aus. Der Grund dafür ist, dass scanf das Zeilenumbruchzeichen im Eingabestream belässt und das folgende fgets daher eine leere Zeile liest. Und unter Verwendung gets ist just plain wrong.

Die Lösung besteht darin, die Eingabe immer mit fgets zu lesen und dann den Eingang mit sscanf nach Bedarf zu analysieren. Für die Fälle, in denen sscanf nicht benötigt wird, d. H. Die Eingabe ist eine Zeichenkette, können Sie strcspn verwenden, um die neue Zeile aus dem Puffer zu entfernen.

int PopulateStruct(Actress *node) 
{ 
    char buffer[100]; 

    printf("Please enter the name of the actress/actor: "); 
    if (fgets(buffer, sizeof buffer, stdin) == NULL) 
     return 0; 
    buffer[strcspn(buffer,"\n")] = '\0'; 
    if ((node->name = malloc(strlen(buffer) + 1)) == NULL) 
     return 0; 
    strcpy(node->name, buffer); 

    // ditto for place of birth and hair color 

    printf("Please enter the actress/actor age: "); 
    if (fgets(buffer, sizeof buffer, stdin) == NULL) 
     return 0; 
    if (sscanf(buffer, "%d", &node->age) != 1) 
     return 0; 

    printf("Please enter the actress/actor networth: "); 
    if (fgets(buffer, sizeof buffer, stdin) == NULL) 
     return 0; 
    if (sscanf(buffer, "%lf", &node->networth) != 1) 
     return 0; 

    return 1; 
} 

Oh, und ich änderte die networth von float zu double. A float hat nur 6 oder 7 Stellen Genauigkeit, und das ist nicht annähernd genug für das Vermögen einer Schauspielerin/Schauspieler.