Odczyt z pliku za pomocą przeładowanego operatora >>

Question

Chcę odczytać dane z pliku za pomocą przeładowanego operatora ">>". Przeczytałem parę tematów na forach i testowałem różne kombinacje ale nic nie daje rezultatu. Niby się kompiluje bez błędów ale zamierzonego wyniku nie ma, gdy w programie wyświetlam magazyn, jest on pusty, a powinien zostać wczytany wraz ze startem programu. Funkcja wyświetlania na pewno działa jak należy, ponieważ pokazuje obiekty utworzone w programie, nie pokazuje tylko tych które powinien wczytać z pliku.

Tak jest zdefiniowana funkcja

istream & operator >> (ifstream & in, Magazyn & obiekt){
    int i = 0;
    while(!EOF){
        return in >> obiekt.magazyn[i]->nazwa_tow >> obiekt.magazyn[i]->ilosc >> obiekt.magazyn[i]->cena_tow >> obiekt.magazyn[i]->indeks_tow;
        i++;
    }
}

A tak ją wywołuję:

void Magazyn::wczytaj_magazyn(Magazyn & obiekt){ //odczyt z pliku
    ifstream plik("magazyn.txt", ios::in);
    plik >> obiekt;
    plik.close();
}

Jeśli potrzeba innego fragmentu kodu wystarczy napisać, nie trzeba od razu krytykować

Comments

Czym jest EOF?

---

miał być warunek dla pętli "zakończ na końcu pliku"

---

Yhm, EOF jest chyba sklandikiem jezyka pascal i oznacza koniec pliku, tzn ze dane zostaly wczytane i nie znajduje sie tam nic więcej.

https://www.freepascal.org/docs-html/rtl/system/eof.html

---

... Ale to C++

Answer 1

Linia 4 pierwszego przykładu, po pierwszym wejściu w pętlę, od razu z niej wychodzisz return'em. Linia 5 już się z całą pewnością nie wykona.

Linia 3 pierwszego przykładu, nie testuj (bliżej nieokreślonego EOF), tylko stan strumienia in.

Linia 2 drugiego przykładu, utworzenie obiektu std::ifstream, jest tożsame z otwarciem pliku w stanie std::ios::in. Ten ostatni argument jest więc niepotrzebny.

Linia 4 drugiego przykładu, zbędne .close(). Wyjście z zakresu w linii 5, powoduje automatyczne zamknięcie pliku.

Comments

A dane dalej się nie wczytują. Nawet jeśli błędna pętla wykonuje się tylko raz to przynajmniej jeden obiekt powinien być wczytany, a tak się nie dzieje

---

Bo test while(...) jest bez sensu. Powtarzam że powinieneś testować stan strumienia in a nie jakieś wydumane EOF.

Answer 2

std::istream & operator >> (std::istream & in, Magazyn & obiekt)
{
	int i = 0;
	while(in >> obiekt.magazyn[i]->nazwa_tow 
			>> obiekt.magazyn[i]->ilosc 
			>> obiekt.magazyn[i]->cena_tow 
			>> obiekt.magazyn[i]->indeks_tow)
	{ ++i; }

	return in;
}

Podejrzany jest ten magazyn, ale nie wnikam...