[c++] Nadprogramowy znak z pliku

**Celestian hunter** OP · 12-11-2013 21:03

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:

Kod:

char znak2 = NULL;

while(!plik.eof())
{
		
		plik.get(znak2);
		cout << znak2;
		
}

Jeśli zawartością pliku będą dla przykładu dwa znaki: "ef", to po program wypluje "eff".
Czemu ;o

niemoralny · 12-11-2013 21:15

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

**Celestian hunter** OP · 12-11-2013 21:21

Cały, nieskomentowany kod, który kompiluję:

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.eof())
{
		
		plik.get(znak2);
		cout << znak2;
}
plik.close();
getchar();
return 0;
}

Zawartość pliku "dane.txt": e
Wynik na ekranie: ee

...eee? ;d

**Tibiarz** · 12-11-2013 21:25

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?

niemoralny · 12-11-2013 21:26

śmieszna sprawa, u mnie to samo jest xd najlepsze jest to, że

Kod:

while(true))
    {
                if(plik.eof()){break;}
		plik.get(znak2);
		cout << znak2;
   }

daje taki sam efekt, co

Kod:

while(true))
    {

		plik.get(znak2);
		cout << znak2;
                if(plik.eof()){break;}
   }

@up
do while - tak samo

**Tibiarz** · 12-11-2013 21:27

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:

Kod:

while(true))
    {

		plik.get(znak2);
		cout << znak2;
                if(plik.eof()){break;}
   }

zamienić na

Kod:

while(true))
    {

		plik.get(znak2);
                if(plik.eof()){break;}
		cout << znak2;
   }

Frozelix · 12-11-2013 21:31

Nie mam zainstalowanego srodowiska, ale

Kod:

while (!plik.eof())    
   {
      znak2 = plik.get();   
      cout << znak2;
   }

sprobuj tak :p
@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().

**Celestian hunter** OP · 12-11-2013 21:35

O wygugliłem
Czyli tibia77 chyba dobrze prawił ;o

Ale rozwiązanie Frozelixa załatwiło sprawę, wszystko hula, dzięki <3

**Tibiarz** · 12-11-2013 21:42

Frozelix napisał

Cytat został ukryty, ponieważ ignorujesz tego użytkownika. Pokaż cytat.

Nie mam zainstalowanego srodowiska, ale

Kod:

while (!plik.eof())    
   {
      znak2 = plik.get();   
      cout << znak2;
   }

sprobuj tak :p
@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().

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.

Dowód:

Moje rozwiązanie

Twoje rozwiązanie

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.

Bazan · 12-11-2013 23:54

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;
}

Zadzialalo mi dobrze :)

**Tibiarz** · 13-11-2013 00:07

@up

Kod:

while(plik.get(znak2))
{
   cout << znak2;
}

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).

Temat: [c++] Nadprogramowy znak z pliku

Narzędzia tematu

Oceń ten temat

[c++] Nadprogramowy znak z pliku

Informacje o temacie

Użytkownicy przeglądający temat

Podobne tematy

Znak wodny

C++ Wczytywanie z pliku + tworzenie pliku

Znak w hotkeyach

Odszumianie, podgłaśnianie i podbijanie wyraźności pliku głosowego AMR

[Poradnik] Jak zrobić clienta Tibii w jednym pliku!

Zakładki

Zakładki

Zasady postowania