Co mam źle w kodzie?

Question

Czy mógłby mi ktoś powiedzieć co jest nie tak w kodzie? Wywala mi błąd gdy chce zapisać listę do pliku w case 2 w switchu:)
 

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

using namespace std;

 

class samochod
{

public:
    string kolor, marka,nw,nr;
    int wiek,ps;

    samochod(string kol="niebieski", string mark="Honda", string nww="Kowalski", string nrr="Dw650", int wie=10, int pss=2000) :kolor(kol), marka(mark), nw(nww), nr(nrr), wiek(wie), ps(pss)
    {}
    void wczytaj()
    {
        string linia;
        ifstream plik;
        plik.open("mytmp.txt");
        if (plik.good() == false)    cout << "nie udało sie otworzyc pliku" << endl;
        else
        while (getline(plik, linia))
        {
            cout << linia << endl;
        }

        }
    
    void ZapiszDoPliku(vector<samochod>&lista, string fileName)
    {

        ofstream plik;
        plik.open(fileName, ios::out | ios::app);
        if (plik.good() == false) cout << "nie udało sie otworzyc pliku" << endl;
        else for (int i = 0; i<lista.size(); i++)
        {
            string tmp;
            tmp = lista[i].kolor + " ; " + lista[i].marka + " ; " + lista[i].nw + " ; " + lista[i].nr + " ; " + to_string(lista[i].wiek) + " ; " + to_string(lista[i].ps);
            plik << tmp << "\n";
        }

    }
};

int main()
{
    samochod tmp;
    vector<samochod> lista;
    int wybor;
    do
    {
        system("cls");
        cout << "Kolor ; Marka ; Nazwa wlasciciela ; Nr rejestracyjny ; Wiek ; Poj. silnika " << endl;
        cout << "---------------------------------------------------------------------------" << endl;
        tmp.wczytaj();
        cout << "---------------------------------------------------------------------------" << endl << endl;
        cout << "1.Dodaj samochod do listy: " << endl;
        cout << "2.Wyjdz: " << endl;
        cout << "Co chcesz zrobic?: " << endl;
        cin >> wybor;
        switch (wybor)
        {
        case 1:
            int ile;
            cout << "Ile samochodow chcesz dodac do listy?: " << endl;
            cin >> ile;
            for (int i = 0; i < ile; i++)
            {
                cout << "Podaj kolor samochodu: \n";
                cin >> tmp.kolor;
                cout << "Podaj marke samochodu: \n";
                cin >> tmp.marka;
                cout << "Podaj nazwe wlasciciela samochodu: \n";
                cin >> tmp.nw;
                cout << "Podaj nr rejestracyjny samochodu: \n";
                cin >> tmp.nr;
                cout << "Podaj wiek samochodu: \n";
                cin >> tmp.wiek;
                cout << "Podaj pojemnosc silnika samochodu: \n";
                cin >> tmp.ps;
                lista.push_back(tmp);
            }
            tmp.ZapiszDoPliku(lista, "mytmp.txt");
            break;
        case 2:
            exit(0);
            break;

        default:
            cout << "POMYLKA";
            Sleep(2000);
            break;
        }
    } while (wybor !=2);

    
    
    system("PAUSE");

    return 0;

}

Answer 1

Jaki błąd Ci "wywala"? Wiesz, to bardzo cenna wiedza. ;)

A poza tym w 'case 2 w switchu' masz funkcję exit(0). ;)

Tak w ogóle to co to za dziwna moda na używanie extit() w main()? Przecież jak się rzuci zwykłego returna, efekt będzie tożsamy, a nie będzie trzeba wołać dodatkowej funkcji z osobnego nagłówka...

Comments

wybacz chodzi o case 1, juz sobie z tym poradziłem, dzięki za odpowiedź :)

---

Cieszy mnie to. :) Powodzenia! :)