złe wczytywanie danych

Question

Witam:)

Tworzę pewnego rodzaju symulator i potrzebuję pomocy. Zapis pliku wygląda ok, lecz wczytywanie cos nie działa pokazuje mi:
1
0
0
0
0
0
Dlaczego??

Dodam tylko, iż tablice to zmienne globalne.

void zapis()
{
    fstream save;
    save.open("save.sav", ios::out);

    for(int i=0; i<=5; i++)
    {
        save<<lvl_bud[i]<<endl;
    }

    for(int i=0; i<=3; i++)
    {
        save<<lvl_woj[i]<<endl;
    }

    for(int i=0; i<=3; i++)
    {
        save<<lvl_sur[i]<<endl;
    }

    for(int i=0; i<=15; i++)
    {
        save<<il_woj[i]<<endl;
    }

    for(int i=0; i<=3; i++)
    {
        save<<il_sur[i]<<endl;
    }

    exit(0);
}

void wczytaj()
{
    fstream save;
    save.open("save.sav",ios::in);

    if(save.good()==false)
    {
        cout<<"Nie mozesz wczytac";
        menu();
    }

    int nr_linii=1;
    string linia;

    while(getline(save,linia))
    {
        for(int i=nr_linii-1; i<=5; i++)
        {
            lvl_bud[i];
            nr_linii++;
        }



    }
    cout<<lvl_bud[0]<<endl<<lvl_bud[1]<<endl<<lvl_bud[3]<<endl<<lvl_bud[4]<<endl<<lvl_bud[5]<<endl;
    cout<<"Zakonczono wczytywanie. Nacisnij cokolwiek by kontynuowac."<<endl;
    getch();
    exit(0);
}

 

Answer 1

Do lvl_bud[i] nic nie przypisujesz, sprawdź lvl_bud[i]=linia; i ta podwójna pętla Ci nic nie daje w tym przypadku, 

for(int i=0; i<=5;i++)
{
getline(save,linia);
lvl_bud[i]=linia;
}

 

Comments

W którym miejscu dać ten fragment kodu??

---

Tam gdzie masz while(getline(save, linia)){for{...}}

---

To Ci już kolega DiZZie napisał w całości ten kod ;)

Answer 2

Zamiast tej pętli while z getline napisz

string buffer;
for(int i=0; i<=5; i++) //zakładając, że tablica ma 6 elementów
    {
        getline(save, buffer); //musisz pobrać dane jako string
        lvl_bud[i] = atoi(buffer.c_str()); //konwersacja ze stringa na int, dołącz bibliotekę               <cstdlib>
    }

 i analogicznie do wszystkich innych tablic. Zmienne nr_linii i linia nie są potrzebne. Na samym końcu zapomniałeś wypisać lvl_bud[2].

Comments

Nie mam pojęcia jak to zrobiłes ale u mnie masz szacun. Proszę mi tylko dlaczego mój sposób nie działał, wszystko robiłem tak naj Pan Mirosław na filmie.

---

No niezupełnie, jeśli chciałbyś to napisać sposobem, który jest na filmie Pana Mirosława to musiałoby wyglądać to mniej więcej tak

int nr_linii=1;
string linia;
 
    while(getline(save,linia))
    {
        if(nr_linii <= 6 )
        {
              lvl_bud[nr_linii-1] = atoi(linia.c_str());
        }
        else if( nr_linii <= 6 + ilosc elementów następnej tablicy)
        {
           //konwersacja na int i przypisanie do zmiennej
        }
        //else if ...
        nr_lini++;
    }

 

---

No ok, a jak zrobić kolejne tablice?? Bo mam jeszcze do zrobienia 4 tablice i nie wiem jak to zrobić.

---

Najprościej będzie napisać to tym pierwszym sposobem, który Ci podałem, jedyne co musisz zmieniać w tej pętli to warunek (zależny od ilości elementów w tablicy) oraz samej tablicy. Przykładowo dla tablicy

int tablica[10]

pętla będzie wyglądać następująco:

for(int i=0; i < 10; i++)
    {
        getline(save, buffer);
        tablica[i] = atoi(buffer.c_str());
    }

---

A można inaczej?? Bo mi zależy na czytelności i żebym wiedział która tablica za co odpowiada.

---

Mogę się mylić, ale wydaje mi się że to najlepszy sposób. Co do czytelności w samym kodzie to wystarczy odpowiednio nazwać tablice żeby się łatwo połapać lub można zostawić kilka komentarzy ;)

---

:( Skoro tak mówicie, to niech tak będzie. Może w przyszłości coś mi się uda zrobić. No nic, dzięki wszystkim za pomoc. Jeśli jednak znalazłby ktoś jakąś dobrą metodę to proszę podać.