vector!! push_back bug in c++ prog.

[It3rat0r]

Hello there. Im working with a bank program in console c++. I have faced a problem i can't solve now. When im shuting down the program with my exit function, It is supose to load all the data into a file.txt.
It does this, But it also adds an account with no name, No accountnumber and no balance.

This is my complete code for the moment.

#include <iostream>
#include <fstream>
#include <string>
#include <windows.h>
#include <vector>
#include <time.h>

using namespace std;


struct banken //struct som innehåller det som ett "konto" skall innehålla. Dvs ett namn ett kontonummer och ett saldo.
{
    string Namn;
    int KontoNr;
    double Saldo;
};
void Meny(vector<banken>& BankVekt);//Säger till kompilatorn att jag kommer att kalla på dessa funktioner senare i programmet.
void DpCash(vector<banken>& BankVekt);//--||--
void VisaKonto(vector<banken>& BankVekt);//--||--
void VisaSaldo(vector<banken>& BankVekt);//--||--
void WthCash(vector<banken>& BankVekt);//--||--
void Avsluta(vector<banken>& BankVekt);//--||--
void DeleteAcc(vector<banken>& BankVekt);//--||--

banken SkapaKonto(string Namn,int KontoNr,double Saldo)
{
    banken b;
    b.Namn=Namn;
    b.KontoNr=KontoNr;
    b.Saldo=Saldo;
    return b;
}
void CreateAcc(vector <banken>& BankVekt) // Funktion för att skapa ett konto
{
    system("cls");
    srand(time(NULL)); // Skapar srand som skall använda sig av tiden som behållare för ett slumpat tal
    int RandTal=rand()%100; // Ger slumptalet namnet RandTal och säger åt den att slumpa tall upp till 100
    string namn;

    cout<<"Skriv '<' for att komma till huvudmenyn\n";
    cout<<"Skriv ditt namn tack: ";
    getline(cin,namn);
    if(namn=="<")
    {
        system("cls");
        Meny(BankVekt);
    }
    else
    {
    BankVekt.push_back(SkapaKonto(namn,RandTal,0)); // Lägger SkapaKontons värden dvs Namn Kontonummer och Saldo längst bak i min vector.
    system("cls");
    cout<<"Du har aktiverat ett konto med namnet: " <<namn<<endl<< "Och har tilldelats kontonummer "<<RandTal<<endl
    <<"ditt saldo ar for tilfallet 0 Kronor\n";
    Sleep(3000);
    system("cls");
    Meny(BankVekt);
    }
    return;
}

void Meny(vector<banken>& BankVekt) // Min Meny.
{
string val;
cout<<"1) Visa saldo\n2) Ta ut pengar\n3) Sätt in pengar\n4) Visa alla konton\n5) Skapa nytt konto\n6) Ta bort ett befintligt konto\n7) Avsluta programmet\nVar god valj ett alternativ\n";
getline(cin,val);
if(val=="1")
{
    VisaSaldo(BankVekt);//visa saldo funk
}
else if(val=="2")
{
   WthCash(BankVekt);// ta ut pengar funk
}
else if(val=="3")
{
    DpCash(BankVekt);//Sätt in pengar funk
}
else if(val=="4")
{
        VisaKonto(BankVekt);//Visa alla konton funk
}
else if(val=="5")
{
    CreateAcc(BankVekt);// skapa konto funk
}
else if(val=="6")
{
        DeleteAcc(BankVekt);//ta bort konto funk
}
else if(val=="7")
{
    Avsluta(BankVekt); // Avsluta program funk.
}

else if(val=="<")
    {
        system("cls");
        Meny(BankVekt);
}
else

{
    cout<<"Felaktigt val, var god prova igen\n";
    Sleep(2000);
    system("cls");
    Meny(BankVekt);
}


return;
}

void DpCash(vector<banken>& BankVekt) // sätta in pengar funktion
{
system("cls");

int KontoNr,Okas;

    for(vector<banken>::iterator it = BankVekt.begin(); it != BankVekt.end(); ++it){ // Itererar igenom vectorns konton.
    cout << endl << "NAMN: " << it->Namn << endl << "NUMMER: " << it->KontoNr << endl // Skriver ut Namn Nummer och saldo.
    << "SALDO: " << it->Saldo << endl << endl;
    }

cout<<"Ange kontonummer"<<endl;
cin>>KontoNr;

cout<<"Hur mycket vill du satta in?"<<endl;
cin>>Okas;

for(vector<banken>::iterator it = BankVekt.begin(); it != BankVekt.end(); ++it){ // Itererar igenom vectors konton.
if(it->KontoNr==KontoNr) // Om iteratorns KontoNr är lika med KontoNr så ökas saldot med det värde som användaren matat in.
{
it->Saldo+=Okas;
}
}
Meny(BankVekt);
return;
}

void VisaKonto(vector<banken>& BankVekt) // Visa konton funktion
{

    system("cls");
    for(vector<banken>::iterator it = BankVekt.begin(); it != BankVekt.end(); ++it){ // Itererar igenom vector och skriver sedan ut både namn nummer och saldo på skärmen.
    cout << endl << "NAMN: " << it->Namn << endl << "NUMMER: " << it->KontoNr << endl
    << "SALDO: " << it->Saldo << endl << endl;
    }
    Meny(BankVekt);
}

void VisaSaldo(vector<banken>& BankVekt) // Visa saldo funktionen.
{
    system("cls");
    for(vector<banken>::iterator it = BankVekt.begin(); it != BankVekt.end(); ++it){ // Itererar igenom min vector efter konton.
    cout << endl << "NAMN: " << it->Namn << endl << "NUMMER: " << it->KontoNr<< endl << endl;// Och skriver sedan ut Namn och kontonummer på skärmen ifall man har glömt bort sitt nummer
    }

    int KontoNr;
    cout<<"Ange kontonummer"<<endl;
    cin>>KontoNr;

    for(vector<banken>::iterator it = BankVekt.begin(); it != BankVekt.end(); ++it){ // Itererar igenom konton en gång till
        if(it->KontoNr==KontoNr)
        {
            cout<<"Ditt saldo = "<<it->Saldo<<endl; // Och skriver sedan ut endast saldot på det kontonummer som skrivits in.
        }

    }
    system("PAUSE");
    system("cls");
    Meny(BankVekt);
return;
}
void WthCash(vector<banken>& BankVekt) // Ta ut pengar funktionen.
{
    system("cls");
    int KontoNr,Minskas;

    for(vector<banken>::iterator it = BankVekt.begin(); it != BankVekt.end(); ++it){ // Itererar igenom min vector för att kolla vilka konton som finns
    cout << endl << "NAMN: " << it->Namn << endl << "NUMMER: " << it->KontoNr << endl// och skriva ut dem och dess innehåll på skärmen
    << "SALDO: " << it->Saldo << endl << endl;}

    cout<<"Ange Kontonummer\n";
    cin>>KontoNr;
    cout<<"Hur mycket vill du ta ut?\n";
    cin>>Minskas;

    for(vector<banken>::iterator it = BankVekt.begin(); it != BankVekt.end(); ++it){ //Itererar igenom min vector efter konton.
    {it->Saldo-=Minskas;} // Minskar sedan Saldo med så mycket som användaren matade in. Dvs variablen 'Minskas'.
}
    system("cls");
    Meny(BankVekt);

}

void Avsluta(vector<banken>& BankVekt)
{
    ofstream SkrivTillFil;
    SkrivTillFil.open("konton.txt");

    for(vector<banken>::iterator it = BankVekt.begin(); it != BankVekt.end(); ++it){ // Itererar igenom min vector för att kolla vilka konton som finns
    cout << endl << "NAMN: " << it->Namn << endl << "NUMMER: " << it->KontoNr << endl// och skriva ut dem och dess innehåll på skärmen
    << "SALDO: " << it->Saldo << endl << endl;}

    if (SkrivTillFil)
{
    for(vector<banken>::iterator it = BankVekt.begin(); it != BankVekt.end(); ++it){
    SkrivTillFil << it->Namn << "#" << it->KontoNr << "#" << it->Saldo << endl;
}
}
else
{

   cout << "Kunde ej spara!" << endl;
}
SkrivTillFil.close();
exit(1);//Avsluta
}
}
int main()
{
    vector<banken> BankVekt; // Min vector som jag använder för att lagra mina konton i.
    string FilRad;

    ifstream MinFil;
    MinFil.open("konton.txt");

    while(MinFil) //while snurra som läser igenom min fil.
    {
        banken temp;
        getline(MinFil,FilRad);//Läser igenom min skapade fil och en rad i taget.
        string n;//namn
        int kn;//Kontonummer
        double s;//saldo

        n=FilRad.substr(0,FilRad.find("#"));//Läser in en rad(string) som börjar på noll och slutar på "#".
        FilRad.erase(0,FilRad.find("#")+1);//Tar sedan bort raden och ger den +1 för att även ta bort "#"

        kn = atoi(FilRad.substr(0,FilRad.find("#")).c_str()); // Använder en gammal C funktion som gör en string till en int för att kunna läsa in kontonummer som endast består av heltal.
        FilRad.erase(0,FilRad.find("#")+1);

        s = atof(FilRad.substr(0,FilRad.find("#")).c_str()); // Använder en gammal C funktion som gör en string till en double för att läsa in saldo som kan innehålla decimaltal.
        FilRad.erase(0,FilRad.find("#")+1);

        temp.Namn = n;
        temp.KontoNr = kn;
        temp.Saldo = s;
        BankVekt.push_back(temp);
    }
    MinFil.close();
    Meny(BankVekt);

    return 0;
}

If any of you guys can spot the error in my code and find out whats wrong it would be lovley. The comments is in swedish and i am sorry for that, But u dont have to mind the comments if you cant read swedish.
Thanks /Johannes

[Peter87]

I think the problem is with the loop in the main function. The stream will stay valid until you try make a read that fails. When you have read everything in the file the loop will make one iteration more but then getline fails and you get an empty string. Test the return of getline in the loop condition instead.
while(getline(MinFil,FilRad))

[It3rat0r]

Yes thank you it worked :)