Rozwiązanie ćwiczenia dotyczącego tworzenia struktur C++

Question

Witam, jestem w trakcie czytania i przerabiania ksiązki od Stephen'a Prata i utknąłem na tym zadaniu..

Zaprojektuj strukturę car zawierającą następujące informacje o samochodzie: markę jako tablicę znakową lub obiekt string, rok budowy jako liczbę całkowitą. Napisz program który spyta użytkownika o liczbę katalogowanych aut. Następnie za pomocą operatora new stwórz dynamiczną tablice struktur car, poproś użytkownika o podanie marki (może mieć więcej niż jedno słowo) oraz o rok. Należy zachować tu ostrożność, gdyż na zmianę odczytujemy łańcuchy znakowe i liczby. Na koniec wyświetl zawartości wszystkich struktur. Oto przykładowy przebieg sesji z programem:

 

Ile samochodów chcesz skatalogować? 2
Samochód #1:
Proszę podać markę: Hudson Hornet
Rok produkcji: 1952
Samochód #2:
Proszę podać markę: Kaiser
Rok produkcji: 1951
1952 Hudson Hornet
1951 Kaiser

Mój kod :

#include <iostream>
#include <string>
using namespace std;




int main(int argc, char** argv) {
	int i;
	
	struct car {
		string marka;
		int rok_budowy;
	};
	
	int ile_razy;
	
	cout<<"Ile razy chcesz wpisać ? \n";
	cout<<"Podaj liczbe : ";
	cin>>ile_razy;
	car * ps = new car[ile_razy];
	for(i = 0; i < ile_razy; i++){
		cout<<"Podaj marke samochodu : \n";
		cin>>ps->marka;
		cout<<"Podaj rok budowy : \n";
		cin>>ps->rok_budowy;
		
	}
for (i = 0; i< ile_razy; i++){
	cout<<"Samochod #"<<car;
}
	
	return 0;
}

Możecie mi pomóc ?

Answer 1

poproś użytkownika o podanie marki (może mieć więcej niż jedno słowo) 

Czyli innymi słowy użyj std::getline.

Wskaźnik ps wskazuje na tablicę, a nie pojedynczy element. Więc zamiast 

ps->marka

ma być

ps[i].marka

 

Comments

Zmienione, co dalej ? 

#include <iostream>
#include <string>
using namespace std;




int main(int argc, char** argv) {
	int i;
	
	struct car {
		string marka;
		int rok_budowy;
	};
	
	int ile_razy;
	
	cout<<"Ile razy chcesz wpisać ? \n";
	cout<<"Podaj liczbe : ";
	cin>>ile_razy;
	car * ps = new car[ile_razy];
	for(i = 0; i < ile_razy; i++){
		cout<<"Podaj marke samochodu : \n";
		getline(cin,ps[i].marka);
		cout<<"Podaj rok budowy : \n";
			cin>>ps[i].rok_budowy;
		
	}

	return 0;
}

 

---

Należy zachować tu ostrożność, gdyż na zmianę odczytujemy łańcuchy znakowe i liczby.

Po wczytaniu liczby całkowitej operatorem >> w strumieniu wejściowym zostaje znak '\n', którego należałoby się pozbyć przed wczytywaniem getlinem. Przez to, że teraz tego nie robisz "przeskakuje" tego getline'a i przechodzi do roku budowy.

---

Jak się go pozbyć ?

---

cin.ignore() przed getlinem.

---

Program działa, ale na samym końcu wywala błąd :/ 

#include <iostream>
#include <string>
using namespace std;




int main(int argc, char** argv) {
	int i;
	
	struct car {
		string marka;
		int rok_budowy;
	};
	
	int ile_razy;
	
	cout<<"Ile razy chcesz wpisac? \n";
	cout<<"Podaj liczbe : ";
	cin>>ile_razy;
	cout<<endl;
	car * ps = new car[ile_razy];
	
	
	for(i = 0; i < ile_razy; i++){
		cout<<"Podaj marke samochodu : ";
		cin.ignore() ;
		getline(cin, ps[i].marka);
		cout<<endl;
		
		cout<<"Podaj rok budowy : \n";
			cin>>ps[i].rok_budowy;
	}

for(int i=0;i<ile_razy; i++)
{
cout << ps[i].marka << " " << ps[i].rok_budowy << endl;
}
delete ps;


	return 0;
}

 

---

Źle zwalniasz pamięć. Alokowałeś tablicę, nie pojedynczy element.

---

Dziękuję za pomoc :)