Dynamiczne pole w klasie

[Uzikan 0]

Witam,

musze na jutro skończyć program, a jako że jest już mało czas pozwoliłem sobie poprosić was o pomoc.

Poniżej zamieszczam kod programu, wyrzuciłem z niego zbędne wiersze aby nie komplikować. Generalnie problem wygląda tak: mam klase Student, dynamiczą tablice TAB obiektów klasy Student. W klasie jest zdefiniowana dynamiczna tablica na Przedmioty. Dla każdego studenta użytkownik będzie podawał dowolną ilość Przedmiotów. Do rzeczy: problem mam z alokacją pamięci na Przedmioty oraz przypisania do tego pola wartości. Chodzi o pogrubioną linijkę w kodzie. Nie chce mi się przez nią kompilować. Aha i jeszcze jedno, w konstruktorze klasy jest niby alokacja pamięci ale nie wiem czy to jest tak jak być powinno.

int N;
int K;

class Student {
public:
string *Przedmioty;	
Student(){		//konstruktor
	Przedmioty=new string[N];
}
};

int _tmain(int argc, _TCHAR* argv[])
{
cerr<<"Ilu studentow chcesz dodac? ";
cin>>K;

Student *TAB=new Student[K];	//dynamiczna tablica obiektow klasy Student

for(int i=0;i<K;i++){	//wczytanie przedmiotow dla kazdego studenta

	for(int k=0;k<N;k++){
		cerr<<"\t przedmiot "<<k+1<<": ";
		[b]cin>>TAB[i].Przedmioty[k];[/b]
	}
	cout<<endl;
} 

delete[]TAB;
system("PAUSE");
return 0;
}

[strom 0]

na poczatku pierwszej petli powinno byc jeszcze TAB = new Student()

 

to tak na pierwszy rzut oka

[Gość tobix10]

A ile wynosi N ?

W destruktorze musi być zwolnienie tablicy przedmiotów.

 

Wszystko wydaje się być dobrze, może masz gdzie indziej błąd albo przez to, że N nie ma przypisanej żadnej wartości.

 

//edit

Teraz problemy innej natury:

1. cerr służy do wypisywania do strumienia błędów, nie powinieneś go używać w takich sytuacjach. Od tego jest cout.

2. Przedmioty to składowa Studenta i nie powinna być widoczna na zewnątrz , a szczególnie nie powinieneś tak ich wczytywać. Klasy są po to, żeby były 'samowystarczalne'. Lepszym rozwiązaniem byłaby funkcja wczytaj lub przeładowanie operatora <<.

3. N jako wielkość tablicy Przedmioty w sumie też powinna należeć do klasy. To w klasie ma być wiadomo ile jest przedmiotów. Ponadto mogłaby być argumentem konstruktora z jakąś domyślną wartością np. Student(int _N = 5) : N(_N) { Przedmioty = new string[N]; }

[Uzikan 0]

1. cerr służy do wypisywania do strumienia błędów, nie powinieneś go używać w takich sytuacjach. Od tego jest cout.

Powiem tak, to jest nawyk który wpoił mi nie dawno mój wykładowca (dr). Chodzi o rozróżnienie tego co chcemy wydrukować a co jest potrzebne aby program się wykonał. Dr mówił że do zaproszeń do wczytania danych powinno się używać właśnie cerr. Jakby używać "cout" to jak będziesz chciał przekierować strumień na drukarke to wydrukuje ci te wszystkie zaproszenia typu "podaj x:" itd. Jak użyjesz cerr to się nie wydrukuje

[Gość tobix10]

Nie każdy program jest pisany, żeby działał z drukarką, skanerem czy jakimkolwiek innym sprzętem.

Jeżeli chciałbym dodać możliwość wydruku to bym zrobił sam taką funkcję, która ładnie sformatowałaby mi wszystko.

[209458 867]

No ale ta drukarka to taki przykład. Pod linuksem masa narzędzi działa na strumieniach - można przekierować wyjście z jednego programu pod wejście drugiego, zapisać tymczaswe wyniki do pliku, albo po prostu wyświetlić na monitorze. Program jest uniwersalny, a do tego nie trzeba osobno oprogramowywać poszczególnych przypadków.