Przy wpisywaniu danych do tabeli program wysypuje sie.

Question

#include <iostream>
#include <cstdlib>
#include <string>


using namespace std;


struct DataUrodzenia
{
	int rok;
	int miesiac;
	int dzien;	
};


struct Osoba 
{
	string imie;
	string nazwisko;
	string adres;
	DataUrodzenia *dataurodzenia;
};


int main()
{

int N;
Osoba *Dane[N];

for (int i=0; i<N; i++)
{
Dane[i] = (struct Osoba*) malloc ( N * sizeof(struct Osoba));
}

cout << "Podaj liczbe N" << endl;
cin >> N;	

for (int i=0; i<N; i++)
{
	cout << "Podaj imie" << endl;
	cin >> Dane[i]->imie;
	cout << "Podaj nazwisko" << endl;
	cin >> Dane[i]->nazwisko;
	cout << "Podaj adres" << endl;
	cin >> Dane[i]->adres; 
}

for (int i=0; i<N; i++)
{
	cout << Dane[i]->imie;
	cout << Dane[i]->nazwisko;
	cout << Dane[i]->adres;
}


return 0;
}

Witam, główny problem polega na Tym, iż przy wpisywaniu imienia do tabeli program sie konczy. Zakladam, że jest to jakis problem z przydzieleniem pamieci. Prosze o pomoc i pozdrawiam :) 

Comments

Dlaczego używasz malloca? Gdzie zwalniasz pamięć? Sama alokacja struktur to błąd. Może kiedyś pisałeś w C, ale C++ nie powinien tego przepuścić.

Answer 1

Problem jest z tym fragmentem:

int N;
Osoba *Dane[N]; // UB
 
for (int i=0; i<N; i++)
{
Dane[i] = (struct Osoba*) malloc ( N * sizeof(struct Osoba)); // później powoduje to UB
}
 
cout << "Podaj liczbe N" << endl;
cin >> N; 

Tworzysz statyczną tablicę wskaźników na `Osoba` z użyciem niezainicjowanej niestałej zmiennej `n` co powoduje niezidentyfikowane zachowanie. Ponadto korzystasz z dynamicznej alokacji z języka C do stworzenia obiektów nietrywialnej klasy, a następnie korzystasz z tych obiektów bez ich utworzenia. To wszystko powoduje, że twój program jest niezdefiniowany (nie wiadomo właściwie co się stanie).

Proponuję rozwiązanie korzystające z C++owego operatora new[]:

cout << "Podaj liczbe N" << endl;
int N; // lepiej użyć `std::size_t`
cin >> N; 
 
Osoba* Dane = new Osoba[N];
// dodatkowo trzeba będzie zmienić strzałki `->` na kropki `.`
// oraz usunąć tablicę operatorem delete[]

// lepiej użyć `std::vector<Osoba>` lub `std::unique_ptr<Osoba[]>`:
// std::vector<Osoba> Dane(N);
// auto Dane = std::make_unique<Osoba[]>(N);

 

Comments

Dziękuje bardzo, faktycznie problemem była funkcja malloc.

Pozdrawiam i życzę miłego wieczoru.