Wersja do druku
Dziwne rzeczy się dzieją; albo jestem ślepy i nie widzę jakiegoś oczywistego błędu, albo zgłupiałem do końca.
Program ma robić kodowanie rle, ale wrzucam tu tylko ten powycinany fragment:
Jeśli zawartością pliku będą dla przykładu dwa znaki: "ef", to po program wypluje "eff".Kod:char znak2 = NULL;
while(!plik.eof())
{
plik.get(znak2);
cout << znak2;
}
Czemu ;o
daj cały kod (z obsługą pliku) bo jak wklejam twój fragment to pętla się nie kończy, być może zapomniałem o czymś banalnym ;p
@
nvm przez pomyłkę zapisywałem egzeka w złym folderze xD
Cały, nieskomentowany kod, który kompiluję:
Zawartość pliku "dane.txt": eKod:#include <iostream>
#include <fstream>
using namespace std;
int main(){
fstream plik;
plik.open("dane.txt", ios::in);
if(!plik.good())
return 0;
char znak2 = NULL;
while(!plik.eof())
{
plik.get(znak2);
cout << znak2;
}
plik.close();
getchar();
return 0;
}
Wynik na ekranie: ee
...eee? ;d
A nie jest tak, że eof zostanie ustawione dopiero gdy będziesz na końcu pliku i wtedy spróbujesz odczytać kolejny znak, a nie po przeczytaniu ostatniego znaku?
Znaczy, masz kursor ustawiony za ostatnim znakiem, ale program jeszcze nie wie, że to ostatni znak, więc dopiero gdy będziesz chciał jeszcze raz przesunąć kursor w prawo, to skapnie się, że to był już koniec pliku. Dlatego do pętli wchodzi ci o jeden raz za dużo?
śmieszna sprawa, u mnie to samo jest xd najlepsze jest to, że
daje taki sam efekt, coKod:while(true))
{
if(plik.eof()){break;}
plik.get(znak2);
cout << znak2;
}
@upKod:while(true))
{
plik.get(znak2);
cout << znak2;
if(plik.eof()){break;}
}
do while - tak samo
Ale patrzcie (drug kod nienormalnego):
Wchodzi do pętli, czyta to "e" (zakładamy, że plik to po prostu znak e). Potem sprawdza czy został już osiągnięty koniec pliku, a teoretycznie jeszcze nie został, kursor jest ustawiony za ostatnim znakiem i dopiero teraz gdy będziesz chciał coś przeczytać, to osiągniesz koniec pliku. Więc znowu próbuje wczytać znak, ale to się nie udaje, bo tam już nic nie ma, jednocześnie ustawia znacznik eof na true, potem wypisuje znowu tamtą zmienną i dopiero potem warunek nie jest spełniony i wychodzi z pętli.
Wniosek:
zamienić naKod:while(true))
{
plik.get(znak2);
cout << znak2;
if(plik.eof()){break;}
}
Kod:while(true))
{
plik.get(znak2);
if(plik.eof()){break;}
cout << znak2;
}
Nie mam zainstalowanego srodowiska, ale
sprobuj tak :pKod:while (!plik.eof())
{
znak2 = plik.get();
cout << znak2;
}
@edit
Nie no, chyba nie prawil dobrze. Funkcja get przesuwa wskaznik za "e" i potem plik.eof juz widzi ze wskaznik wskazuje na koniec pliku i przerywa petle. Po prostu zle uzyles funkcji .get().
O wygugliłem
Czyli tibia77 chyba dobrze prawił ;o
Ale rozwiązanie Frozelixa załatwiło sprawę, wszystko hula, dzięki <3
Nie. W twoim rozwiązaniu jeśli używasz tam podstawienia, a nie udało się nic przeczytać, to za znak2 jest wstawiany '\0', dlatego cout nie wypisuje tej ostatniej literki powtórnie. Co nie zmienia faktu, że nadal program dochodzi do tego miejsca (po przeczytaniu ostatniego znaku i tak włazi w pętlę jeszcze raz), bo nie masz racji co do tego, kiedy eof ustawiane jest na true.Cytuj:
Frozelix napisał
Dowód:
Moje rozwiązanie
http://oi44.tinypic.com/nxkefb.jpg
Twoje rozwiązanie
http://oi44.tinypic.com/34pjnes.jpg
On użył funkcji get dobrze, a jedyna różnica między twoim kodem a jego jest taka, że u niego gdy nie uda się nic przeczytać, to znak2 pozostaje bez zmian, a u ciebie zmienia się na '\0'. Ale jak widzisz, tak czy siak mamy wypisany jeden nadprogramowy znak.
Zadzialalo mi dobrze :)Kod:#include <iostream>
#include <fstream>
using namespace std;
int main(){
fstream plik;
plik.open("dane.txt", ios::in);
if(!plik.good())
return 0;
char znak2 = NULL;
while( plik.get(znak2) && !plik.eof())
{
cout << znak2;
}
plik.close();
getchar();
return 0;
}
@up
Powinno wystarczyć, bo raczej nie zaistnieje taka sytuacja, żebyś przeczytał kolejny znak, mimo osiągnięcia końca pliku, lub na odwrót (eof będzie true, tylko jeśli nie uda ci się przeczytać następnego znaku).Kod:while(plik.get(znak2))
{
cout << znak2;
}