Problem z dynamicznym alokowaniem tablicy dla struktury

Question

Witam, mam problem z dynamicznym alokowaniem tablicy dla stworzonej przeze mnie struktury. Czy ktoś mógłby pokazać mi gdzie jest błąd i dlaczego w taki sposób kod nie chce działać? Dodam, że gdy zamiast wskaźnika użyję "const int" wszystko działa w najlepszym porządku, a przy takim kodzie jak poniżej cały program zacina się po podaniu pierwszego imienia. Z góry dziękuję za pomoc.

#include <iostream>

using namespace std;

struct Kontakty
{
        string imie;
        string nazwisko;
        int wiek;
        string nr_tel;
};
int zakres;
Kontakty* tab = new Kontakty[zakres];

void Wprowadz_imie(int i)
{
    cout<<"Prosze podac imie: ";
    cin>>tab[i].imie;
}
void Wprowadz_nazwisko(int i)
{
    cout<<"Prosze podac nazwisko: ";
    cin>>tab[i].nazwisko;
}
void Wprowadz_wiek(int i)
{
    cout<<"Prosze podac wiek: ";
    cin>>tab[i].wiek;
}
void Wprowadz_telefon(int i)
{
    cout<<"Prosze podac numer telefonu: ";
    cin>>tab[i].nr_tel;
    cout<<endl;
}
void Wprowadz_dane()
{
    for(int i=0; i<zakres; i++)
    {
        Wprowadz_imie(i);
        Wprowadz_nazwisko(i);
        Wprowadz_wiek(i);
        Wprowadz_telefon(i);
    }
}
void Pokaz_dane()
{
    for(int i =0;i<zakres;i++)
    {
        cout<<"Imie: "<<tab[i].imie<<endl;
        cout<<"Nazwisko: "<<tab[i].nazwisko<<endl;
        cout<<"Wiek: "<<tab[i].wiek<<endl;
        cout<<"Numer telefonu: "<<tab[i].nr_tel<<endl;
        cout<<endl;
    }
}

int main()
{
    cout<<"Wprowadz liczbe dla tablicy: ";
    cin>>zakres;
    Wprowadz_dane();
    Pokaz_dane();
    delete [] tab;
    return 0;
}

 

Answer 1

int zakres;
Kontakty* tab = new Kontakty[zakres];

Masz tu zmienną `zakres` bez przypisanej wartości (to zmienna globalna, więc domyślnie 0), po czym od razu próbujesz alokować tablicę o tym rozmiarze. To, że w dalszym miejscu kodu robisz `cin>>zakres`, nie oznacza, że wszystkie miejsca "magicznie" zauważą że `zakres` się zmienił i zmieni się rozmiar tablicy.

Musisz najpierw pobrać `zakres` od użytkownika, a dopiero wtedy zaalokować tablicę z użyciem `new`.

(btw, polecam nie używać zmiennych globalnych :/ )

Comments

I wszystko stało się nagle jasne  Bardzo dziękuję.