Struktury w c++

Question

Czesc, dostrzega ktos blad w tym kodzie? Zauwazylem ze caly program sypie sie konkretnie przez jedna linijke

a->klatki[i].pietra[j].mieszkania[k].numer = B;

A struktury wygladaja tak:

struct SMieszkanie
{
	int numer;
	float powierzchnia;
	float wysokosc;
	int liczba_pokoi;
};

struct SPietro
{	
	int nr_pietra;
	int ile_mieszkan;		
	SMieszkanie *mieszkania = new SMieszkanie[ile_mieszkan];
	
};

struct SKlatka
{
	int nr_klatki;
	int ile_pieter;
	SPietro *pietra = new SPietro[ile_pieter];	
};
struct SBlok
{
	string ulica;
	int numer;
	int ile_klatek;
	SKlatka *klatki = new SKlatka[ile_klatek];
};

Nie wywala zadnego errora ale kiedy dodaje ta linijke program zatrzymuje sie w pewnym momencie, i konczy dzialanie.

Comments

Pokazy csly kod. Ja dostrzegam problem myslowy: nie wiem czemu klatka i pietro to agregacja. I czemu klatka nie jest blokiem. Majac blok, ktory ma 2 klarki, to kazdy z nich ma inny numer, znajduja sie w tym samym budynku, ale maja inne adresy

---

To nie ja to wymyslalem, jest to z gory narzucone przez wykladowce, ja mam to z pliku sciagnac, i chodzi mi tylko o to czy ta linijka jest prawidlowo zapisana biorac pod uwage te struktury

---

Ale w ktorym momencie przestaje, tutaj problemem jest chyba odwolanie sie do pointera, ktory ma w sobie pointer, ten z kolei ma w sobie koleny itd i chyba trzeba by przeciazyc operator []. Ostatnio mialem chyba cos podobneo, ale nie jestem pewien

---

Wydaje mi sie ze wlasnie to tu jest problemem, jak to rozwiazac? Jak to inaczej zapisac zeby nie bylo takiego problemu?

Answer 1

struct SPietro
{   
    int nr_pietra;
    int ile_mieszkan;       
    SMieszkanie *mieszkania = new SMieszkanie[ile_mieszkan];
};

Jak myślisz, jak duża tablica została zaalokowana w linii 5.? :)

Comments

   ///

---

Widzisz, dopiero w linijce 11. wpisujesz ile chciałbyś mieć mieszkań. A to już za późno, bo to j-te piętro z i-tej klatki już dawno zostało stworzone, (więc i tablica została zaalokowana).

---

Da sie to jakos zmodyfikowac, zeby sama forma kodu zostala mniej wiecej taka sama?

---

Nie wiem, bo nie wysłałeś mi całego kodu. Jeżeli inicjalizacja jest robiona źle, to cały program ma niezdefiniowane zachowanie i może robić co będzie chciał. Jeżeli jest robiona poprawnie, to bazując na tym kawałeczku który mi wysłałeś, możesz zdealokować te tablice o nie wiadomo jakich rozmiarach i zaalokować poprawne.

int juzZnanaLiczbaMieszkan;
std::cin >> juzZnanaLiczbaMieszkan;

// Usuwam nie wiadomo jak dużą tablicę
delete[] pietro.mieszkania;

// Alokuję poprawną, bo właśnie się
// dowiedziałem jak duża powinna być
pietro.mieszkania = new Mieszkanie[juzZnanaLiczbaMieszkan];

 

---

Dobra to juz jakos sobie z tym poradze przynajmniej widze gdzie lezy problem dzieki wielkie!