Zapis danych z pliku do listy jednokierunkowej

Question

void odczyt(struct dane **head)
{
    struct dane *nowy = (struct dane*)malloc(sizeof(struct dane));
    struct dane *point = *head;
    char bufor[30];
    printf("zmienne\n");
    FILE *plik;
    plik = fopen("test.txt", "r");

    if(plik == NULL)
        printf("Błąd podczas otwierania pliku!");
    else
    {
        while(fscanf(plik, "%d", bufor) != EOF)
        {
            struct dane *nowy = (struct dane*)malloc(sizeof(struct dane));
            printf("wejscie w while\n");

            nowy->next = NULL;

            nowy->id = (int*)malloc(sizeof(int));
            strcpy(nowy->id, bufor);

            fscanf(plik, "%s", bufor);
            strcpy(nowy->tytul, bufor);

            fscanf(plik, "%d", bufor);
            nowy->rok = (int*)malloc(sizeof(int));
            strcpy(nowy->rok, bufor);

            fscanf(plik, "%d", bufor);
            nowy->stan = (int*)malloc(sizeof(int));
            strcpy(nowy->stan, bufor);

            fscanf(plik, "%d", bufor);
            nowy->przesluchany = (int*)malloc(sizeof(int));
            strcpy(nowy->przesluchany, bufor);

            printf("po wszystkich strcpy\n");

            if(*head == NULL)
            {
                *head = nowy;
                printf("head = nowy\n");
            }
            else
            {
                while(point->next != NULL)
                {
                    point = point->next;
                    printf("point = point->next\n");
                }
                point->next = nowy;
                printf("point->next=point\n");
            }
        }
    }
    printf("odczyt udany");
    fclose(plik);
}

 

Witam

 

Mam problem z wczytywaniem danych do listy jednokierunkowej. Co należy zmienić w funkcji aby działała? 

Z góry dziękuje!

Plik z którego czyta wygląda tak: 

2
roko
1992
0
1

1
enej
2001
0
1

 

Answer 1

Tak na szybko:

if(*head == NULL)
{
        *head = point = nowy;
        printf("head = nowy\n");
}

void odczyt(struct dane **head)
{
	struct dane *nowy = (struct dane*)malloc(sizeof(struct dane)); //<--- po co ten 'nowy'...
	...
	
	 while(fscanf(plik, "%d", bufor) != EOF)
     {
		struct dane *nowy = (struct dane*)malloc(sizeof(struct dane)); // <--- ...jeśli jest ten?
		...

 

Comments

Jest już lepiej, odczytuje lecz gdy wyświetlam te dane ukazuje się takie coś:

ID: 6950224
Tytul: roko
Artysta: =Windows_NT
Gatunek: \CodeBlocks\MinGW\bin;Çj
Rok: 6950528
Kupiony: Nie
Przesluchany: Nie

 

ID: 6950680
Tytul: enej
Artysta: em32\WindowsPowerShell\v1.0;C:\Program Files (x86)\N(j
Gatunek: \Program Files (x86)\N(j
Rok: 6950696
Kupiony: Nie
Przesluchany: Nie

---

 (int*)malloc(sizeof(int));

Dlaczego dla inta alokujesz pamięć? Nie możesz po prostu w strukturze dać int id; itd?

nowy->id = (int*)malloc(sizeof(int));
strcpy(nowy->id, bufor);

strcpy do kopiowania wartości typu int? To nie może się udać. generalnie zamiast cyrkować z tym buforem, od razu czytaj do odpowiednich pól:

fscanf(plik, "%s", nowy->tytul);
fscanf(plik, "%d", &nowy->rok);
fscanf(plik, "%d", &nowy->stan);
fscanf(plik, "%d", &nowy->przesluchany);

Oczywiście rok, stan i przesluchany muszą być intami

 

---

Niestety nadal mam problem Poprawiłem kod, lecz po wykonaniu:

void odczyt(struct dane **head)
{

    struct dane *point;
    printf("zmienne\n");
    FILE *plik;
    plik = fopen("dane.txt", "r");

    if(plik == NULL)
        printf("Błąd podczas otwierania pliku!");
    else
    {
        while(fgetc(plik) != EOF)
        {
            struct dane *nowy = (struct dane*)malloc(sizeof(struct dane));
            printf("wejscie w while\n");

            nowy->next = NULL;
            fscanf(plik, "%d", &nowy->id);
            fscanf(plik, "%s", nowy->tytul);
            fscanf(plik, "%s", nowy->artysta);
            fscanf(plik, "%s", nowy->gatunek);
            fscanf(plik, "%d", &nowy->rok);
            fscanf(plik, "%d", &nowy->stan);
            fscanf(plik, "%d", &nowy->przesluchany);

            printf("po wszystkich strcpy\n");

            if(*head == NULL)
            {
                *head = point = nowy;
                printf("head = nowy\n");
            }
            else
            {
                while(point->next != NULL)
                {
                    point = point->next;
                    printf("point = point->next\n");
                }
                point->next = nowy;
                printf("point->next=point\n");
            }
        }
    }
    printf("odczyt udany");
    fclose(plik);
}

ukazuje się:

ID: 11617264
Tytul: enej
Artysta: aea
Gatunek: reage
Rok: 2222
Kupiony: Nie
Przesluchany: Nie

ID: 11617264
Tytul: ie
Artysta: Nie
Gatunek: 2
Rok: 1348221509
Kupiony: Nie
Przesluchany: Nie

ID: 11617264
Tytul: ihanna
Artysta: rrr
Gatunek: rock
Rok: 2001
Kupiony: Nie
Przesluchany: Nie

ID: 11617264
Tytul: ak
Artysta: Tak
Gatunek: 1
Rok: 1633906508
Kupiony: Nie
Przesluchany: Nie

ID: 11617264
Tytul: nej
Artysta: asd
Gatunek: pop
Rok: 1992
Kupiony: Nie
Przesluchany: Nie

ID: 11617264
Tytul: ie
Artysta: Tak
Gatunek: ly 6 Model 94 Stepping 3, GenuineI
Rok: 539767584
Kupiony: Nie
Przesluchany: Nie

z pliku: 

3
enej
aea
reage
2222
0
0
2
rihanna
rrr
rock
2001
1
1
1
enej
asd
pop
1992
0
1

Jest dwa razy więcej węzłów niż być powinno po odczytaniu z pliku. Oraz w niektórych polach są błędne dane. Podejrzewam że jest zły warunek w pętli while, lecz nie mam pojęcia jaki byłby poprawny.

---

Pokaż definicję struktury dane.

while(fgetc(plik) != EOF) { ... }

Pewny jesteś tego fgetc? Co rekord czytasz dodatkowo jeden znak, co może zaburzyć odczyt następnych rekordów. Tak bym to zrobił:

int id;

while(fscanf(plik, "%d", &id) == 1)
{
	struct dane *nowy = (struct dane*)malloc(sizeof(struct dane));
	nowy->next = NULL;
	
	nowy->id = id;
	
	fscanf(plik, "%s", nowy->tytul);
	fscanf(plik, "%s", nowy->artysta);
	...
	

 

---

Moja struktura:
 

struct dane
{
    int id;
    char tytul[20];
    char artysta[30];
    char gatunek[20];
    int rok;
    int stan;
    int przesluchany;
    struct lista *next;
};

 

---

OK. Popraw tę pętlę.

 

To na zaliczenie, czy coś bardziej użytkowego ma być?

---

Gdy zmienię na kod podany przez Pana, w ogóle nie kompiluje się ponieważ nie jest zadeklarowane id. nowy->id także nie działa ponieważ przydział pamięci do nowy wykonuje się już w pęti while.

Tak, to jest praca do szkoły.

---

nie kompiluje się ponieważ nie jest zadeklarowane id.

Jak nie jest, jak jest? Źle przepisałeś.

---

Nie zauważyłem pierwszej linijki. Wczytuje już poprawną liczbę węzłów, lecz pole id nadal ma jakieś kosmiczne wartości.

---

Sprawdź debugerem, czy fscanf poprawnie wczytuje tą wartość.

---

Jeśli dobrze to wykonałem to w oknie "watch" debugera jest poprawna wartości pola id.

---

To może coś z wyświetlaniem tej wartości jest nie tak. Trudno powiedzieć...

---

Jakoś już sobie z tym poradzę, dziękuje za pomoc.