C++ - ładowanie danych z pliku do listy

Question

Cześć. Za zadanie mam ułożenie danych z pliku tekstowego do listy jednokierunkowej. Wszystko okej, tworzę strukturę i próbuje załadować pliki po kolei do odpowiednich kategorii.

int main()
{
	
	struct node* head = NULL, * tmp = NULL;
	
	ifstream myFile;
	myFile.open("eurovision.txt");
	if (!myFile.good()){
		cout << "Something went wrong" << endl;
		return 0;
	}
	
	tmp = new struct node;
	head = tmp;

	while (myFile >> tmp->year >> tmp->country >> tmp->song >> tmp->name >> tmp->language) {
	
		tmp->next = NULL;
		tmp = tmp->next;
		tmp = new struct node;
		
	}

	for (tmp = head; tmp; tmp = tmp->next) {
		cout << tmp->year << " " << tmp->country << " " << tmp->song << " " << tmp->name << " " << tmp->language << endl;
	}

	return 0;
}

Problem jest taki, że poprawnie wyświetla mi się pierwsza linijka, wszystko tk jak powinno, ale na tym się kończy. Kolejne linijki już nie są wyświetlane. Rozumiem, że problem jest w tej pętli while? Zdaję się, że na końcu tworzę jeden nowy pusty element, ale głowie się już kilka godzin i nadal nie wiem jak mogę to zmienić

Comments

tmp = new struct node;

To nie C, wystarczy tmp = new node;

---

Racja, dziękuje bardzo za podpowiedź

Answer 1

Wewnątrz pętli while ustawiasz następny element na NULL zamiast nadać mu wartość przydzielonej pamięci.

I oczywiście trzeba też usuwać nadmiarowy element na końcu. Alternatywą jest użycie zmiennych lokalnych zamiast przypisywania danych od razu do struktury.

Comments

Okej, w takim razie usuwam nadmiarowy element na końcu, ale nadal nie rozumiem jak mam nadać jakąś wartość w takiej sytuacji, co ma być tą wartością :(

---

Zobacz, co się dzieje linia po linii w następującym fragmencie:

tmp->next = NULL;
tmp = tmp->next;
tmp = new struct node;

Co jest na początku i co zostaje na końcu w strukturze tmp i w której, we wcześniejszej i w nowo utworzonej.

---

@letmestay, głównym problemem jest to, że w prawdzie tworzysz obiekty node dla każdej pozycji w pliku, ale nie łączysz ich w listę, dlatego masz tylko jeden wpis, bo ten przypisujesz na początku do head.

---

Udało się i usunąć i ładnie załadować :) Dziękuje za pomoc