从文件读取并将格式化的行输入到链接列表。 C

Question

I'm trying to read from a file, line by line, a formatted string and storing it in a linked list.

My typedef:

typedef struct s_deck {
    char name[16];
    int atk;
    int def;
    int cost;
    struct s_deck* next;
} card;

Example of a file I'm reading from:

Line 1: Name 4 4 5
Line 2: Name2 5 5 5

My function I'm trying to implement to do such a thing...

card *LoadDeck() {
    FILE* file = fopen("text", "r");
    if(file == NULL) {
        printf("unable to open file...\n");
    }
    char line[16];
    card* start = NULL;
    card* newest = NULL;
    card* previous = NULL;

    fgets(line, 15, file);
    card* newCard = malloc(sizeof(card));
    sscanf(line, "%s %d %d %d\n", newCard->name, &newCard->atk,
                &newCard->def, &newCard->cost);
    printf("Added:%s Attack:%d Defense:%d Cost:%d\n", newCard->name,    newCard->atk,
                newCard->def, newCard->cost);

    if (start == NULL) {
        start = LoadDeck(NULL);
        newest = start;
    } 
    else {
        newest = LoadDeck(newest);
    }

    if (previous != NULL) {
        previous->next = newCard;
    }
    fclose(file);
    return newCard;
}

So my issue I'm having is that it is an infinite loop where it reads the first line and the first line only. The printf I have is to see what its doing and... well its reading the line correctly but it just keeps doing it and never stops.

I don't know if its creating a linked list properly yet. I haven't gotten that far. I think I implemented it right but its so hard going off books only. I tried to find some questions here but their file they are reading from is much more complex than mine so I figured there should be a simpler way.

Answer 1

Well, i know maybe i should put this in comment but sorry i don't have enough reputation to do it. Anyway i think your program loops endlessly because your start variable is always NULL, first call to the function is will lead to function to call itself after finding that start == NULL is true, the subsequent calls will always lead to an endless recursion because start is always NULL. these lines you wrote is the black hole :

   if (start == NULL) {
    start = LoadDeck(NULL);
    newest = start; } 

It doesn't assign anything to start variable, it just causes the endless recursion.