Plik wczytywanie śmieci C

Question

Witam, powie mi ktoś jak zrobić, żeby gdy plik jest pusty nie wczytywało mi śmieci ?

 

struct album* wczytaj_z_pliku()
{
    FILE *baza=fopen(PLIK_Z_BAZA,"r");
    struct album* list_z_pliku=NULL;

        while(!feof(baza))
        {
            struct album* wczytany = alokuj();
            fscanf(baza,"%d,",&wczytany->id);
            char tytul[64];
            fscanf(baza,"%[^,],",tytul);
            wczytany->tytul=malloc(strlen(tytul)*sizeof(char));
            strcpy(wczytany->tytul,tytul);

            char artysta[64];
            fscanf(baza,"%[^,],",artysta);
            wczytany->artysta=malloc(strlen(artysta)*sizeof(char));
            strcpy(wczytany->artysta,artysta);

            fscanf(baza,"%d,",&wczytany->rok);

            char gatunek[64];
            fscanf(baza,"%[^,],",gatunek);
            wczytany->gatunek=malloc(strlen(gatunek)*sizeof(char));
            strcpy(wczytany->gatunek,gatunek);

            char stan[64];
            fscanf(baza,"%[^,],",stan);
            wczytany->stan=malloc(strlen(stan)*sizeof(char));
            strcpy(wczytany->stan,stan);

            char przesluchany[64];
            fscanf(baza,"%s\n\r",przesluchany);
            wczytany->przesluchany=malloc(strlen(przesluchany)*sizeof(char));
            strcpy(wczytany->przesluchany,przesluchany);

            wczytany->next=NULL;
            list_z_pliku=dodaj_element(list_z_pliku,wczytany);
        }

    fclose(baza);
    return list_z_pliku;
}

Dziękuje :)

Answer 1

W liniach 12, 17, 24, 29 i 34 masz błąd (chyba w innym wątku zwróciłem na to uwagę). Przydzielasz o jeden bajt za mało.

 

Możesz zrobić tak:

int id;

while(fscanf(baza,"%d,", &id) == 1) {
	struct album* wczytany = alokuj();
	wczytany->id = id;

	char tytul[64];
	fscanf(baza,"%[^,],", tytul);
	wczytany->tytul = malloc(strlen(tytul) * sizeof(char) + 1);
	strcpy(wczytany->tytul, tytul);
	...
}

Choć prawdę mówiąc powinieneś testować wszystkie wywołania fscanf.

Answer 2

Możesz to zrobić na dwa sposoby.

1. Sprawdzaj wszędzie wartość zwracaną funkcji fscanf - funkcje z tej rodziny zwracają liczbę poprawnie wypełnionych pól (gdzie każde %costam to jedno pole). Jeśli dostaniesz wartość która nie jest spodziewana, to nie dodawaj nic wczytanego do bazy i wyjdź z pętli.

2. Ew. sprawdź na początku wielkość pliku (zazwyczaj robi się to sekwencją instrukcji fseek(f, 0, SEEK_END); size_t wielkosc = ftell(f); fseek(f, 0, SEEK_SET);)