Problem z dynamiczną tablicą obiektów C++

Question

Tworze dynamiczną tablicę obiektów, następnie w pętli for próbuję wypełnić tę tablicę obiektami z odpowiednimi wartościami. Używam do tego konstruktora, ale coś mi się sypie.

class Student{
    private:
        string name;
        float tab[5];
    public:
        string getName(){
            return this->name;
        }
        float getAvrg(){
            float sum=0;
            for(int i=0;i<5;i++)
                sum+=this->tab[i];
            return sum/5;
        }
        Student(string n="student",float a=3,float b=3,float c=3,float d=3,float e=3){
            this->name=n;
            this->tab[0]=a;
            this->tab[1]=b;
            this->tab[2]=c;
            this->tab[3]=d;
            this->tab[4]=e;
        }
};

Student *students = new Student[numb];
    for(int i=0;i<numb;i++){
        cin>>name>>a>>b>>c>>d>>e;
        students[i] = new Student(name,a,b,c,d,e);
    }

Pozdrawiam!

Answer 1

Owszem sypie się bo źle przypisujesz wartości do tablicy, stworzyłeś dynamicznie tablice i skoro ona jest to po co chcesz na nowo alokować pamięć dla jej elementów ? 

Student *students = new Student[numb];
for (int i = 0; i<numb; i++)
{
	cin >> name >> a >> b >> c >> d >> e;
	students[i] = Student(name, a, b, c, d, e);
}

Tak to powinno być jeżeli chcesz to robić przez konstruktor :D

Comments

Czasem rzeczy najbardziej oczywiste są najtrudniejsze do zauważenia :) Dzięki wielkie!

Answer 2

Okej, chyba już sam rozwiązałem swój problem :D

Dla potomnych, stworzyłem tablicę wskaźników do obiektów i wszystko śmiga :)

Student **students = new Student*[numb];
    for(int i=0;i<numb;i++){
        cin>>name>>a>>b>>c>>d>>e;
        students[i] = new Student(name,a,b,c,d,e);
    }

Ale jeżeli ktoś ma inny pomysł jak sobie z tym poradzić to chętnie zobaczę :)

Pozdrawiam!

 

Comments

w tym wypadku tworzysz tablice wskaźników, a następnie wypełniasz tą tablice nowymi obiektami, niby ok ale to dodatkowy problem przy zwalnianiu pamięci chyba nie ma potrzeby utrudniania sobie tak bardzo, w ten sposób tworzy się dynamiecznie tablice dwuwymiarowe :)