c - fopen 导致段错误

Question

如标题所示。简单练习，尝试学习结构和其他基本的 c 功能。这是一个包含 char[] 和数字的结构，我尝试将其值保存在文件中并读回。我通过 fopen() 阅读了很多关于 Seg.fault 的主题，但找不到我的错误!这里有人知道为什么 fopen() 在这种情况下会崩溃吗？欢迎对此事提出任何建议和批评!

发生段错误的函数是load():

void load(struct Telephon *structure, int *counter)
{
    char filename[255];
    char puffer[255], puffercpy[255];
    int i, c, newline_count;
    size_t strlaen;
    FILE *datei=NULL;
    char *token=NULL;

    printf("\033[0;35mWelche Datei oeffnen?\033[0m\n");
    scanf("%s", filename);
    //emptystdin();
    //strlaen=strlen(filename);
    //printf("%d", strlaen);
    //filename[strlaen+1] = '\0';

    //printf("\n%s", filename);

    datei = fopen(filename, "r");

    if(datei==NULL)
    {
        printf("\033[0;31mKonnte Datei %s nicht oeffnen.\033[0m\n", filename);
    }
    else
    {
        while ( (c=fgetc(datei)) != EOF ) //count lines of file
        {
            if ( c == '\n' )
            {
                newline_count++;
            }

        }

        for(i=0; i<=newline_count; i++) //get values in between ";" 
        {
            fgets(puffer, 254, datei);
            strcpy(puffercpy, puffer);

            token = strtok(puffercpy, ";");
            *counter = atoi(token);

            token = strtok(NULL, ";");
            strcpy(structure[i].name, token);

            token = strtok(NULL, ";");
            structure[i].nummer = atoi(token);

        }

    fclose(datei);
    }
    return;

}   

完整代码，main在最后:

telephonListen.c

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <time.h>
#include <ctype.h>
#define MAX 100

struct Telephon
{
    char         name[210];
    unsigned int nummer;
}TELE[MAX];

char* gotTime(char *timestrg)
{
    time_t now;
    now = time(NULL);
    strftime (timestrg, 19, "%d.%m.%Y, %H:%M", localtime (&now));
    return timestrg;
}

void printT(struct Telephon *structarray, int addcount)
{   int i;
    if(addcount==0)
    {
        printf("\033[0;31mEs sind noch keine Eintraege vorhanden!\033[0m\n");
        return;
    }
    else
    {
        for(i=0; i<addcount; i++)
        {
            printf("\nEintrag Nr.%d\n%s:\t%d\n",i+1, structarray[i].name, structarray[i].nummer);
        }
        return;
    }

}

void eingabe(int num, struct Telephon *structarray)
{
    size_t inputlen;
    int check;
    if(num>MAX)
    {
        printf("\033[0;31mMaximale Anzahl von Eintraegen erreicht!\033[0m\n");
        return;
    }
    else
    {
        printf("\n\033[0;35mNamen eingeben:\t\033[0m");
        //fgets(structarray[num].name, MAX, stdin);
        fgets(structarray[num].name, 209, stdin);
        inputlen=strlen(structarray[num].name);
        structarray[num].name[inputlen-1]='\0';
        printf("\n\033[0;35mNummer eingeben:\t \033[0m");
        do
        {
            check = scanf("%10u", &structarray[num].nummer);
        }while( getchar()!='\n');
        fflush(stdin);

        if(check==1)
        {
            printf("Ihr Kontakt wurde angelegt!\n%s:\t%u\n", structarray[num].name ,structarray[num].nummer);
        }
        else
        {
            printf("Fehler bei der Eingabe. Kontakt wurde nicht angelegt!");
            return;
        }

        return;
    }

}
void writeFile(struct Telephon *structure, char *zeitf, int counter)
{
    char filename[255];
    int i;
    FILE *datei;

    if(counter>0)
    {
        printf("\033[0;35mIn welche Datei soll das Telephonbuch geschrieben werden?\n(Achtung: Vorhandene Dateien werden ueberschrieben!)\t\033[0m\n");
        scanf("%s", filename);
        getchar();
        datei = fopen(filename, "w");
        if(NULL == datei)
        {
            printf("\033[0;31mKonnte Datei %s nicht öffnen.\033[0m\n", filename);
        }

        fprintf(datei, "Telephonverzeichnis vom %s\nNAME\t\t|NUMMER\n\n", zeitf);
        for(i=0; i<counter; i++)
        {
            fprintf(datei, "%s\t\t|%d\n", structure[i].name, structure[i].nummer);
        }
        printf("\033[0;32mDatei gespeichert.\033[0m\n");
        fclose(datei);
    }
    else
    {
        printf("\033[0;31mEs sind noch keine Eintraege vorhanden!\033[0m\n");
        return;
    }
    return;

}

void change(struct Telephon *structure, int count)
{
    int eintragnum;
    if (count == 0)
    {
        printf("\033[0;31mEs sind noch keine Eintraege vorhanden!\033[0m\n");
        return;
    }
    else
    {
        printT(structure, count);
        printf("\033[0;31mWelcher Eintrag soll geaendert werden?\033[0m\n");
        do
        {
            scanf("%d", &eintragnum);
        }while(getchar()!='\n');

        if(eintragnum<1||eintragnum>count)
        {
            printf("\033[0;31mBitte die Nummer [zwischen %d und %d]\n des zu aendernden Eintrags eingeben!\033[0m\n", 1, count);
        }
        else
        {
            eingabe(eintragnum-1, structure);
        }

    }

    return;
}
void emptystdin()
{
  int c;
  while( ((c = getchar()) != EOF) && (c != '\n') );
}
void searchContact(struct Telephon *structure, int count)
{
    char searchC;
    int inputlen;
    int i=0;
    int countcompare;
    if (count == 0)
    {
        printf("\033[0;31mEs sind noch keine Eintraege vorhanden!\033[0m\n");
        return;
    }
    else
    {
        printf("Anfangsbuchstabe:\t");
        scanf("%c", &searchC);
        emptystdin();

        for(i=0; i<count; i++)
            {
                if(structure[i].name[0]==searchC)
                {
                    printf("Eintrag gefunden:\n");
                    printf("Nr. %d\nName:\t%s\nNummer:\t%u\n", i+1, structure[i].name, structure[i].nummer);
                }

            }
        return;
    }

}


void save(struct Telephon *structure, int counter)
{
    char filename[255];
    int i;
    FILE *datei;

    if(counter>0)
    {
        printf("\033[0;35mUnter welchem Dateinamen speichern?\n(Achtung: Vorhandene Dateien werden ueberschrieben!)\t\033[0m\n");
        scanf("%s", filename);
        emptystdin();
        datei = fopen(filename, "w");
        if(NULL == datei)
        {
            printf("\033[0;31mKonnte Datei %s nicht anlegen.\033[0m\n", filename);
        }

        for(i=0; i<counter; i++)
        {
            fprintf(datei, "%d;%s;%d\n", i+1, structure[i].name, structure[i].nummer);
        }
        printf("\033[0;32mDatei gespeichert.\033[0m\n");
        fclose(datei);
    }
    else
    {
        printf("\033[0;31mEs sind noch keine Eintraege vorhanden!\033[0m\n");
    }
    return;
}




void load(struct Telephon *structure, int *counter)
{
    char filename[255];
    char puffer[255], puffercpy[255];
    int i, c, newline_count;
    size_t strlaen;
    FILE *datei=NULL;
    char *token=NULL;

    printf("\033[0;35mWelche Datei oeffnen?\033[0m\n");
    scanf("%s", filename);
    //emptystdin();
    //strlaen=strlen(filename);
    //printf("%d", strlaen);
    //filename[strlaen+1] = '\0';

    //printf("\n%s", filename);

    datei = fopen(filename, "r");

    if(datei==NULL)
    {
        printf("\033[0;31mKonnte Datei %s nicht oeffnen.\033[0m\n", filename);
    }
    else
    {
        while ( (c=fgetc(datei)) != EOF ) //Zeilen in Datei zählen
        {
            if ( c == '\n' )
            {
                newline_count++;
            }

        }

        for(i=0; i<=newline_count; i++) //CVS parsen/auslesen
        {
            fgets(puffer, 254, datei);
            strcpy(puffercpy, puffer);

            token = strtok(puffercpy, ";");
            *counter = atoi(token);

            token = strtok(NULL, ";");
            strcpy(structure[i].name, token);

            token = strtok(NULL, ";");
            structure[i].nummer = atoi(token);

        }

    fclose(datei);
    }
    return;

}

int main(void)
{
    int auswahl;
    int count = 0;
    char zeit[20];
    char buffer[2];
    struct Telephon *structptr; //malloc(MAX*(sizeof(TELE)));
    structptr = TELE;

    gotTime(zeit);
    system("clear");
    printf("Telephonkontaktverwaltung\t%s\n", zeit);


    do
    {

        printf("\033[30;47m1: Kontakt hinzufuegen\t2: Kontakte anzeigen\n3: Kontakt aendern\t4: Als Datei speichern\n5. Kontakt suchen\n6. Als CVS sichern\t7. Aus CVS laden\n8. Beenden\nEine der Ziffern eingeben, mit Enter bestaetigen\033[0m\n");

        /*
        scanf("%d", &auswahl);
        scanf("%c", &buffer);

        fgets(buffer, 2, stdin);
        if(isdigit(buffer[1]))
        {
            auswahl=atoi(buffer);
        }
        else
        {
            printf("Eine der Nummern eingeben um Aktion auszufuehren!\n");
        }


        do
        {
           scanf("%d", &auswahl);
        }while(getchar()!='\n');

        fgets(buffer, 2, stdin);
        sscanf(buffer, "%d", &auswahl);
        */

        scanf("%d", &auswahl);
        emptystdin();




        switch (auswahl)
        {
            case 1  :   eingabe(count++, structptr);
                        break;
            case 2  :   printT(structptr, count);
                        break;
            case 3  :   change(structptr, count);
                        break;
            case 4  :   writeFile(structptr, gotTime(zeit), count);
                        break;
            case 5  :   searchContact(structptr, count);
                        break;
            case 6  :   save(structptr, count);
                        break;
            case 7  :   load(structptr, &count);
                        break;
            case 8  :   printf("ENDE\n");
                        break;
            default :   printf("Eine der Nummern eingeben um Aktion auszufuehren!\n");
                        break;
        }



    }while(auswahl!=8);

    return EXIT_SUCCESS;
}

Answer 1

使用rewind()。

在while()和for()循环之间，文件需要重新从头开始。

while ( (c=fgetc(datei)) != EOF ) //count lines of file
  { ...}
rewind(datei);
for(i=0; i<=newline_count; i++) //get values in between ";"
   { ...}

---

强烈建议避免使用scanf() 和fgets(,,stdin)。推荐一个。首选 fgets()。

您可能需要 scanf("%c", 而不是 scanf("%c",。(添加空格)。

---

小想法:
并不是说英语是万能的语言，但 OP 可能想考虑像

// Bold Red "Could not create file" EOL
const char *Err_FileCreation_format = "\033[0;31mKonnte Datei %s nicht anlegen.\033[0m\n"
printf(Err_FileCreation_format, filename);
// or 
#define Err_FileCreation_fmt1 "\033[0;31mKonnte Datei "
#define Err_FileCreation_fmt2 " nicht anlegen.\033[0m\n"
printf(Err_FileCreation_fmt1 "%s" Err_FileCreation_fmt2, filename);