[C++] Rozdzial 34 Zad. 2

Siemanko.
Problem z  tą funkcją mam taki, że działa jak należy jednak na ostatniej linijce w pliku się zapętla, niestety nie wiem czemu...

plik do odczytu:

W drugiej pętli while brakuje negacji przy plik.eof()

Niestety, nic to nie daje a nawet nie wczytuje nawet tych 4 linii, zapętla się na samym początku. :/

Jak wyżej ktoś pisał brakuje negacji plik.eof() i w tej samej linijce zamiast znaku '||' musi być znak '&&'.

Co może być przyczyną, że otrzymuję wynik w takiej formie?:
0x6afdc00x6afdc00x6afdc06
0x6afdc00x6afdc00x6afdc00x6afdc0Bledne dane w wierszu: 2
0x6afdc00x6afdc05
0x6afdc00x6afdc00x6afdc00x6afdc00x6afdc013
0x6afdc00x6afdc00x6afdc0Bledne dane w wierszu: 5

Kod:

EDIT: Już doszedłem gdzie popełniłem błąd. W funkcji czyNapotkanoZnakNowegoWiersza wpisałem .

Cześć,

słuchajcie, kombinuje z zadaniem 2 z rozdziału 34 i mam taki problem - poniższy kod przedstawia moje wypociny i jeśli uruchomię go tak jak jest to wyświetla mi wszystko jak oczekuję (czyli 5 linii jak w pliku txt), ale pętla się nie zakańcza i muszę wychodzić poprzez ctrl+c. Natomiast jeśli usunę komentarz z argumentów drugiej funkcji WHILE z funkcji czyNapotkanoZnakNowegoWiersza pętla się zakańcza i wszystko jest PRAWIE ok lecz wyświetla mi informację o sumie z linii 6... a w pliku mam linii tylko 5. Zatem coś mi umyka i tu prośba do Was - oświećcie mnie co przeoczyłem.

MOJ KOD:

#include <iostream>
#include <string>
#include <fstream>


bool czyNapotkanoZnakNowegoWiersza( std::ifstream & plik )
{
    char cZnak;
    for(;; ) //nieskończona pętla
    {
        plik.clear();
        cZnak = plik.peek(); //sprawdzamy jaki kolejny znak zostanie zwrócony przez operację odczytu
        if( plik.fail() || plik.bad() )
             return false; //wystąpił błąd odczytu danych

        if( !isspace( cZnak ) )
             return false; //pobrany znak nie jest białym znakiem

        plik.get( cZnak ); //odczytujemy biały znak z pliku
        if( plik.fail() || plik.bad() )
             return false; //wystąpił błąd odczytu danych

        if( cZnak == '\n' )
             return true;

    } //for
}


void wczytaj(std::ifstream &plik, int iLiczba, char cZnak)
{
    plik.clear();
    int i=1;
    bool ok;
    int suma;

    while(!plik.eof())
    {
        ok=true;
        suma=0;
        while(!czyNapotkanoZnakNowegoWiersza(plik)/*&&!plik.eof()*/)
        {
            plik.clear();
            plik>>iLiczba;
            if(plik.fail())
            {
                plik.clear();
                plik>>cZnak;
                ok=false;
            }else
            suma+=iLiczba;
        }

        if(ok==false)
            std::cout<<"Bledne dane w linii "<<i<<std::endl;
        else
            std::cout<<"Suma w linii "<<i<<" = "<<suma<<std::endl;
        ++i;
    }
}


bool wczytajPlik(std::string sNazwaPliku)
{
    int iLiczba;
    char cZnak;
    //std::string linia[10];
    std::ifstream plik;
    plik.open(sNazwaPliku.c_str());

    if(!plik.good())
    {
        std::cout<<"Otwarcie pliku zakonczone niepowodzeniem";
        return false;
    }else
    std::cout<<"Plik OK"<<std::endl;

    wczytaj(plik, iLiczba, cZnak);


    plik.close();
    std::cout<<"Plik zamkniety";
    return true;
}

int main()
{
    int iLiczba;
    char cZnak;

    wczytajPlik("31 Roz 34.txt");

    return 0;
}

while( !plik.eof() )

To jest powód. EOF ustawia flagę dopiero gdy próbujesz odczytać z końca pliku.
Po wczytaniu piątej linii nadal EOF == false, następnie pętla robi kolejną rundę
i jest błąd odczytu (EOF), ale pętla i tak kończy obieg i dolicza kolejną inkrementację.

ps: oczywiście zależy to jak kończy się ostatnia linia.

karambaHZP to nie jest prawda co piszesz.
Program który napisał alloyZ zapętla się w funkcji "czyNapotkanoZnakNowegoWiersza"

Znak nowej linii kończący wiersz 5-ty spowoduje wykonanie instrukcji:

i powrót nie do pętli while(!plik.eof())
ale do pętli  while(!czyNapotkanoZnakNowegoWiersza(plik)
z funkcji "czyNapotkanoZnakNowegoWiersza" nigdy nie wyjdzie (zapętli się) bo znak EOF nie jest badany.
Niech ktoś poprawi jak źle myślę.