Chyba při čtení z vytvořeného souboru – C / C++ – Fórum – Programujte.com

Zdravím, nejdříve sem si vytvořil program, který čte z již vytvořeného souboru čísla, která jsou napsaná vždy jedno na řádku a potom do druhého souboru pro zápis je napíše postupně nejdříve sudá potom lichá tedy vyberu sudá a zapíši je a pak skočím na začátek souboru vyberu lichá a zapíši je. 
Potom sem si chtěl program upravit tak abych si soubor, z kterého potom čtu vytvořil sám a sám do něj nastrkal hodnoty. Ale problém je v to, že poslední číslo, které zadám se načte dvakrát. 
Například mám čísla 1 2 3 4 5 (mezeru berte jako řádek) a ve výstupním souboru se mi napíše 2 4 1 3 5 5 tedy dvakrát ta 5. Jak tuto chybu opravit stále nemůžu zjistit, když sem si do souboru pro čtení sám nezapisoval v programu tak to bylo funkční. 
Zde je zdrojový kód:  

#include <fstream>
#include <iostream>

using namespace std;


int evenOdd ( const char * srcFileName, const char * dstFileName )
{
    int cislo;


    ifstream soubor1;
    soubor1.open(srcFileName,ios::in);

    ofstream soubor2;
    soubor2.open(dstFileName,ios::out);

    if(soubor1==0 || soubor2==0){
            cout<<"soubor se neotevrel"<<endl;
            return 1;
    }

    while(!soubor1.eof()) {
        soubor1>>cislo;
        if (cislo%2==0){
            soubor2<<cislo<<endl;
        }
    }

soubor1.clear();
soubor1.seekg(0, ios::beg);


      while(!soubor1.eof()) {
        soubor1>>cislo;
        if (cislo%2!=0){
            soubor2<<cislo<<endl;
        }
        }



    soubor1.close();
    soubor2.close();

      return 0;
}


int main ( int argc, char * argv [] )
 {
    int zapis;

     ofstream soubor;
      soubor.open("input.txt",ios::out);
      if(!soubor.is_open()){
            cout<<"soubor se neotevrel"<<endl;
            return 1;
      }
    for(int i=0;i<5;i++){
            cout<<"Napis cislo:"<<endl;
            cin>>zapis;
        soubor << zapis<<endl;
    }

soubor.close();

   evenOdd("input.txt", "output.txt");
   return 0;
 }

chyba je v tom, ze nekontrolujes, jestli se neco nacetlo

#2 KIIV
U této části:
 

for(int i=0;i<5;i++){
            cout<<"Napis cislo:"<<endl;
            cin>>zapis;
        soubor << zapis<<endl;
    }

A jak bych to provedl normálně pomocí cin.fail()?

#3 Martin

o cin tam nejde.. zkus zadat jako pate cislo misto 5 nejake sude, najednou tam bude napriklad  2 4 6 6 1 3  pro cisla  1 2 3 4 6

proste chyba je az dal.. pri tom nacitani ze souboru, ne z klavesnice

Aha a ošetřit by se to dalo jak? Nic sem nenašel. A jakto, že mi funguje když tam ty čísla nepíši takto ale tak, že si vytvořím texťák pro (input.txt) do něj dám čísla a funguje to.

#5 Martin
kdyz vytvaris soubor za behu, tak tam mas pokazdy cislo a ukonceni radku

kdyz vytvoris, tak nejspis za poslednim cislem zadny znak nemas (ani odradkovani), takze kdyz >> dojede na konec cisla, nastavi EOF.. pokud tam nejaky znak je, musi provest jeste jedno nacitani, kde mas predchozi hodnotu a jelikoz selze, tak se ta hodnota nezmeni a provede zapis

#6 KIIV
Takže chyba je to, že za posledním číslem v input.txt mám ještě endl a tím pádem se provede ještě jednou cyklus while s předchozí načtenou hodnotou, říkám to správně? Ajestli jo tak jak se dá zjistit zda to znovu načetlo číslo nebo ne? 

kdyz se nenacetlo, tak se da pouzit  souborx.fail()  nebo rovnou   if (souborx >> zapsat) .... (operator >> vraci referenci na ten vystupni stream a ma pretizeny operator pretypovani na bool, ktery vraci true jen v pripade, kdyz vsechno fungovalo)

#8 KIIV
Děkuji mockrát :) Ještě  dotaz do budoucna. Tohle budu muset použít vždy, když si vytvořím vlastní soubor, který nebude končit znakem ale odřádkováním, že ano? Kdyby tam to odřádkování nebylo nemusel bych toto řešit,že? 

No spise vzdy.. proste libovolny znak za cislem pred koncem souboru znamena, ze se jeste nedorazilo ke konci souboru - tudiz neni nastaven EOF flag a jeste se o to pokusi znova.

Taktez kdyz ti nekdo zada misto cisla nejaky nesmysl - kdyz to nekontrolujes, tak se zacyklis v nekonecne smycce - pokus o nacteni cisla selze, nastavi fail bit a nikam se neposune. Kdyz je nastaven fail bit, tak uz >> nikdy nic nenacte, dokud ho nevynulujes.

Proste MUSIS kontrolovat stavy a v pripade fail bitu ho taky resetovat (pripadne jeste nacist jedno slovo, abys nestal na miste a nezkousel furt to same)