Dynamiczna tablica wskaźników na strukturę, rożne działanie w zależności od ilości elementów.

Question

Witam,

Mam problem z kodem: 

#include <iostream>
#include <string>

using namespace std;

int main()
{
	struct uczen
	{
		string name;
		int age;
		float avg;
	};

	cout << "Podaj ilosc uczniow: ";
	int ile;
	cin >> ile;

	uczen *ps = new uczen[ile];

	for (int i = 0; i < ile; i++)
	{
		ps = ps + i;
		cin >> ps->name;
		//cin >> ps->age;
		//cin >> ps->avg;
	}
	cout << "\nWyswietlam baze danych: \n";

	ps = ps - ile + 1;

	for (int j = 0; j < ile; j++)
	{
		ps = ps + j;
		cout << ps->name << endl;
		//cout << ps->age << endl;
		//cout << ps->avg << endl << endl;
	}


	system("pause");
	return 0;
}

W zależności od wartości ile, mam rożne zachowanie programu,przy 1 i 2 wszystko jest w porządku,

przy 3 i więcej mam coś takiego, oraz w konsoli CMD losowe znaki i liczby. 

Czy ktoś potrafi zdiagnozować błąd? 

Answer 1

Gdybyś zamiast ps = ps + i; oraz ps = ps + j; miał ps = ps + 1; a zamiast ps = ps - ile + 1; miał ps = ps - ile; to by  to działało, (ps = ps + 1; powinno być na samym końcu ciała pętli).

Ale dlaczego nie napiszesz tego po ludzku, czyli

for (int i = 0; i < ile; i++)
    {
        cin >> ps[i].name;
        //cin >> ps[i].age;
        //cin >> ps[i].avg;
    }

bez żadnych dziwnych operacji na ps.

Comments

Ten sam kod pisze na kilka rożnych sposobów, aby się nauczyć, zobaczyć różnice, a te dziwne operacje, im dziwniejsze tym mydle ze więcej się uczę :D. Poprawiłem zgodnie ze wskazówkami i działa.

Możesz mi wytłumaczyć czemu mój sposób nie działał? z góry bardzo dziękuje :D

---

@monika90, jeszcze jedno, jak skasować tę strukturę, wpisując delete ps; lub delete[] ps; kompilator wyrzuca mi wyjątek podczas kompilacji

---

Żeby wskaźnik przeszedł do następnego elementu tablicy należy go zwiększyć o 1, a Ty dodawałeś i, a i przyjmuje coraz to większe wartości, tak więc wskaźnik wyszedł daleko poza zakres tablicy.

Po ostatniej pętli również musisz skorygować ps instrukcją ps = ps - ile; i użyj delete[] ps; bo alokowałeś tablicę.

Answer 2

Można też tak jeżeli chcesz poćwiczyć operacje na wskaźnikach:

    uczen *ps = new uczen[ile];

	for (uczen* i = ps; i != ps + ile; ++i)
	{
		cin >> i->name;
		//cin >> i->age;
		//cin >> i->avg;
	}
	cout << "\nWyswietlam baze danych: \n";

	for (uczen* i = ps; i != ps + ile; ++i)
	{
		cout << i->name << endl;
		//cout << i->age << endl;
		//cout << i->avg << endl << endl;
	}