Динамический массив как закрытый член класса

[Zhigool']

Здравствуйте Господа и Дамы!
У меня задачка: необходимо создать программу, которая создаёт массив объектов. У каждого объекта есть свой массив произвольного размера и содержания. Доступ к этому массиву осуществляется через открытые члены-функции.
Вот что я наваял:

#include "stdafx.h"
#include <iostream>
#include <conio.h>
using namespace std;

class Mass
{
	float *mas;
	unsigned short k;
public:
	float *mass(unsigned short i)
	{
		float *mas = new float [i];
		for(unsigned short j=0;j<i;j++)
		{
			mas[j]=j;
		}
		k=i;
	return mas;
	}
	void GetMass()
	{
		for(unsigned short i=0; i<k;i++)
		{
			cout<<*mas<<"  ";
			*mas++;
		}
		cout<<"\n"<<endl;
		delete [] mas;
	}
};
void _tmain()
{
	Mass s;
	unsigned short i=10;
	s.mass(i);
	s.GetMass();
	_getch();
}

Скомпилировал, запустил и выдалась ошибка - "Необработанное исключение в "0x00fe16f3" в "Динамический массив.exe": 0xC0000005: Нарушение прав доступа при чтении "0xcccccccc"."
Поправьте если я не прав. Функцией "float *mass()" я создаю массив и передаю указатель на него в закрытый "float *mas". Но когда "float *mass()" заканчивает своё выполнение массив уничтожается, а указатель сохранен в "float *mas", который указывает на первый элемент уже несуществующего массива. Так?
Но тогда ка же решить эту задачу?

[pproger]

	void GetMass()
	{
		for(unsigned short i=0; i<k;i++)
		{
			cout<<*mas<<"  ";
			*mas++;
		}
		cout<<"\n"<<endl;
		delete [] mas;
	}

потерял указатель

[Zhigool']

ммм...

delete [] *mas;

так? но visual studio ругается...

[ACE Valery]

Да тут помимо потери указателя...
В следующем куске кода вы почему-то переобъявляете свой массив локально.

float *mass(unsigned short i)
	{
		float *mas = new float [i];
		for(unsigned short j=0;j<i;j++)
		{
			mas[j]=j;
		}
		k=i;
	return mas;
	}

Поэтому записываются данные в локальный mas, а свойство вашего класса остается без изменений. Исправьте строчку

float *mas = new float [i];

на

mas = new float [i];

Во-вторых, что это за ужас?

void GetMass()
	{
		for(unsigned short i=0; i<k;i++)
		{
			cout<<*mas<<"  ";
			*mas++;
		}
		cout<<"\n"<<endl;
		delete [] mas;
	}

Вы хотите вывести массив на экран? Вот

void GetMass()
	{
		for(unsigned short i=0; i<k;i++)
		{
			cout<<mas[i]<<"  ";
			
		}
		cout<<"\n"<<endl;
		delete [] mas;
	}

Что за ухищрения с передвиганием указателя? Вы же потом удаляете массив как? По указателю, который уже давно не показывает на начало массива.

[Zhigool']

2 ACE Valery
Я только начал программировать. Спасибо Вам за помощь! Заработало.
Интересно, что вот так:

	
void GetMass()
   {
	for(unsigned short i=0; i<k;i++)
        {
	      cout<<*mas<<"  ";
	      *mas++;
	}
	cout<<"\n"<<endl;
	delete [] mas;
}

Работает! )

[ACE Valery]

Что, правда работает? И не вываливается после вывода массива? Странно-странно...

[Пепел Феникса]

может и не вываливается, но утечки должны быть.

[Zhigool']

В развитии темы:

#include "stdafx.h"
#include <iostream>
#include <conio.h>
#include <time.h>
using namespace std;

class Mass
{
	float **mas;
	unsigned short m,n;
public:
	float **mass(unsigned short a,unsigned short c)
	{
		float **mas = new float*[a];
		for(unsigned short j=0;j<a;j++)
		{
			mas[j]=new float[c];
		}
		for(unsigned short i=0;i<a;i++)
		{
			for(unsigned short j=0;j<c;j++)
			{
				mas[i][j]=0;
			}
		}
		m=a;n=c;
		return mas;
	}
	void SetMass()
	{
		srand(clock());
		for(unsigned short i=0;i<m;i++)
		{
			for(unsigned short j=0;j<n;j++)
			{
				float g=(rand()%10000);
				mas[i][j]=(g/10000)*2;
			}
		}
	}
	void GetMass()
	{
		for(unsigned short i=0;i<m;i++)
		{
			for(unsigned short j=0;j<n;j++)
			{
			cout<<mas[i][j]<<" ";
			}
		}
		cout<<"\n"<<endl;
	}
	void DelMass()
	{
		delete[]mas;
	}
};
void _tmain()
{
	clock_t time;
	time = clock();
	Mass s[2];
	unsigned short a=10,c=10;
	for(unsigned short p=0; p<2;p++)
	{
		s[p].mass(a,c);
	}
	for(unsigned short p=0; p<2;p++)
	{
		s[p].SetMass();
		s[p].GetMass();
	}
	
	for(unsigned short p=0; p<2;p++)
	{
		s[p].DelMass();
	}
	time=clock()-time;
	cout<<(double)time/CLOCKS_PER_SEC;
	_getch();
}

Не работает и выдает ошибку:"Необработанное исключение в "0x01321121" в "Динамический массив.exe": 0xC0000005: Нарушение прав доступа при записи "0x42bc2583"." Что не так?

[ACE Valery]

Мдяяя... Перечитайте сообщение №4. Вы снова в функции
float **mass(unsigned short a,unsigned short c)
объявляете локальную переменную mas, в которую и записываются данные.

[Zhigool']

Ещё проблемка возникла. При создании большого количества объектов, в которых содержится двумерный массив с большим количеством элементов, происходит переполнение стека. Как можно избавиться от ограничений. Планирую использовать все 8ГБ оперативы.

#include "stdafx.h"
#include <iostream>
#include <conio.h>
#include <time.h>
using namespace std;
#define KOLVONS 1000 //количество объектов класса
class Mass
{
	float **mas;
	int m,n;
public:
	float **mass(int a,int c) //создание массива
	{
		mas=new float*[a];
		for(int j=0;j<a;j++)
		{
			mas[j]=new float[c];
		}
		for(int i=0;i<a;i++)
		{
			for(int j=0;j<c;j++)
			{
				mas[i][j]=0;
			}
		}
		m=a;n=c;
		srand(clock());
		return mas;
	} 
	void SetMass() //Установка элементов динамического массива в случайное значение
	{
		srand(clock());
		for(int i=0;i<m;i++)
		{
			for(int j=0;j<n;j++)
			{
				double g=(rand()%10000);
				mas[i][j]=(g/10000)*2;
			}
		}
	}
	void GetMass() //Вывод результатов работы проги для её контроля
	{
		for(int i=0;i<m;i++)
		{
			for(int j=0;j<n;j++)
			{
			cout<<mas[i][j]<<" ";
			}
		}
		cout<<"\n"<<endl;
	}
	void DelMass() //удаление динамического массива
	{
		for(int i=0;i<n;i++)
		{
			delete[]mas[i];
		}
		delete[]mas;
	}
};
void _tmain()
{
	clock_t time;
	time = clock();
	Mass s[KOLVONS];
	int a=100,c=100;
	for(int p=0; p<KOLVONS;p++)
	{
		s[p].mass(a,c);
	}
	for(int p=0; p<KOLVONS;p++)
	{
		s[p].SetMass();
	}
	for(int p=0; p<KOLVONS;p++)
	{
		s[p].DelMass();
	}
	time=clock()-time;
	cout<<(double)time/CLOCKS_PER_SEC;
	_getch();
}