ошибка в работе с динамической памятью

[thrashkid]

вот часть кода, класс используется для создания динамического массива
для подсчета среднего бала всех студентов. в двух строках где идет выделение памяти логическая ошибка, выручайте.... не могу понять в чем косяк...препод говорит что чтоб понять, нужно рисовать память и как куда что идет.... нужно сдать лабу срочно

class Group
{
public:
	Group();
	void printaveret();
	void count();
	void add( Marks  );
	void print();
	
private:
	int n;
	double averet;
	int k;
	Marks * array;   // Marks это еще один класс, к которому вроде идет обращение
};


Group::Group()
{
	k = 0;
	averet = 0;
	n = 50;
	array = new Marks[n];
}

void Group::add( Marks  newElement )
{
	array[k].set(newElement.firstMark,newElement.secondMark,newElement.thirdMark,newElement.firstPtr,newElement.lastPtr,
		newElement.year,newElement.course,newElement.firstDiscipline,newElement.secondDiscipline,newElement.thirdDiscipline);
	k++;
}

void Group::count()
{
	double sum = 0;
	for ( int i = 0; i < k; i++ )
	{
		Marks *m = new Marks;     тут ошибка вроде 
		*m=array[i];     и тут........прога работает, но препод говорит, что память используется не так как нужно....я вообще не понимаю в чем тут косяк
		double s;
		run(m);
		
		s=m->averageMark;
		array[i].averageMark=m->averageMark;
		sum += s;
	}

[MaVaL]

Что за run(m); ?

[pproger]

Marks *m = new Marks;
*m=array[i];

ну а что непонятного то. почитай про динамическую память

[MaVaL]

(1) Marks *m = new Marks;
(2) *m=array[i];
(3) double s;
(4)run(m);
(5)s=m->averageMark;
(6)array[i].averageMark=m->averageMark;
(7)sum += s;

Так по порядку:
(1) - Обьявил указатель и создал объект класса Marks указатель m - указывает, на вновь созданный объект.
(2) - Новый объект равен объекту из массива.
(4) - неведомая функция, которая возможно меняет член данных averageMark объекта находящегося по указаелю m.

Ошибка - после выхода из тела цикла созданы k штук объектов класса Marks. За ними помять не очищена. И указатели на них утеряны.

P.S. я бы написал:
for ( int i = 0; i < k; i++ )
{
Marks m;
run(&m);
double s=m.averageMark;
array[i].averageMark=m.averageMark;
sum += s;
}
На каждой итерации объект создаётся и уничтожается по выходу.

[thrashkid]

run описывается в классе Marks

void run( Marks* marks )
{
marks->averageMark = ( (double) marks->firstMark + (double) marks->secondMark + (double) marks->thirdMark ) / 3;
}

[pproger]

marks->averageMark = ( (double) marks->firstMark + (double) marks->secondMark + (double) marks->thirdMark ) / 3;

так никто не делает. хочешь получить double без каста, так и дели на double, либо кастуй по-человечески

double val = static_cast<double>(first_int + second_int + third_int) / 3;

double val = (first_int + second_int + third_int) / 3.0;

выбирай

[MaVaL]

run описывается в классе Marks

Тогда его нужно делать методом класса. Ещё лучше определять значение averageMark в констукторе класса Marks
В таком случае у тебя получится цикл таким:

for ( int i = 0; i < k; i++ )
 sum += array[i].averageMark;

И ни какой долбёжки не будет)

[thrashkid]

в твоем 1ом варианте Maval выдается направильный ответ, вместо среднего бала выдает -858993000,
поменял
for ( int i = 0; i < k; i++ )
sum += array[i].averageMark; ответ все равно -858993000

[MaVaL]

for ( int i = 0; i < k; i++ )
{
Marks m=array[i];
run(&m);
double s=m.averageMark;
array[i].averageMark=m.averageMark;
sum += s;
}

Да ошибся прошу прощения. Строчку подчеркнул.

Второй случай не работает. Т.к. ты не изменил констуктор класса. Соответственно у тебя выводилось не инициализированное значение.

Ещё лучше определять значение averageMark в констукторе класса Marks

Если этого не даелать то чможно сделать так:

for ( int i = 0; i < k; i++ ){
 run(&array[i]) - здесь произоёдет определение значения члена данного объекта
 sum += array[i].averageMark;
}