Warum ein Segmentierungsfehler in feof()?

Question

Ich habe folgende Enum und Struktur:

enum Destination { unknown = 0, hosok, parlament, var }; 

struct Client 
{ 
    char name[30]; 
    char email[30]; 
    char phone_num[11]; 
    int client_num; 
    enum Destination destination; 
    struct tm registration_date; 
}; 

Wenn ich die folgende Methode aufrufen, liest sie die erste Struktur und ausdrucken Name ist, dann bekomme ich einen Segmentation Fault.

void list_clientss() 
{ 
    FILE *f = fopen(filename, "r"); 
    if(f == NULL) 
    { 
    perror("Error"); 
    } 
    struct Client client; 
    while(!feof(f)) 
    { 
    fread(&client, sizeof(struct Client), sizeof(struct Client), f); 
    printf("Name: %s\n", client.name); 
    } 
    fclose(f); 
} 

Was habe ich falsch gemacht?

Answer 1

Erstens Ihre fread Anrufe sollte wie folgt lauten:

fread(&client, sizeof(struct Client), 1, f); 

Zweitens statt feof verwenden, können Sie den Rückgabewert von fread verwenden. fread gibt die Anzahl der zu lesenden Elemente zurück, die Sie übergeben haben. Sie können überprüfen, ob diese Nummer von einer Nummer abweicht. Zum Beispiel

while (fread(&client, sizeof(struct Client), 1, f) == 1) { 
    printf("Name: %s\n", client.name); 
} 

Edit 1: Aktualisieren Sie die while-Schleife, um mehr idiomatische und elegante Version als Wetterfahne vorgeschlagen.