конструктор копирования

[revaldo666]

Всем привет.
есть класс CMatrix
с полями
int ** matrix;
int size_x;
int size_y;
подскажите как сделать для него конструктор копирования и оператор =, а то получается какой то бред.
заранее спасибо.

С конструктором копирования разобрался, немного натупил с выделением памяти.
а вот как перегрузить оператор =?

[artush1984]

CMatrix& CMatrix::operator=(const CMatrix& rhs)
{
     size_x = rhs.size_x;
     size_y = rhs.size_y;
     matirx = rhs.matrix;//или что ещё ты там делаешь ( я о выделении памяти)
     return *this;
}

как то так по моему

[fenryr]

так нельзя делать
при уничтожении 2го объекта будет попытка освободить уже освобожденную память.

CMatrix& CMatrix::operator=(const CMatrix& rhs)
{
     if (this != & rhs) {
    size_x = rhs.size_x;
     size_y = rhs.size_y;
     matrix = new int [size_x ] ;
      for (int i = 0 ; i < size_x ; i++)
           matrix = new int [size_y ] ;
   for (int i = 0 ; i < size_x ; i++)
for (int j = 0 ;j < size_y ;j++)
       matrix [i][j] = rhs.matrix[i][j];
}
     return *this;
}

[pproger]

2fenryr
забыл проверить и уничтожить уже существующий массив в памяти

[artush1984]

Но я же в комментарии написал
А по скольку в вопросе было сказано что Copy Constuctor работает правильно я и написал так !!!
А без всего этого конечно же вы правы

for (int i = 0 ; i < size_x ; i++)
matrix = new int [size_y ] ;

По моему

matirx[i] = new int[sitze_y];

[revaldo666]

спасибо...чем то похоже на конструктор копирования))
Практически один в один.

[pproger]

2revaldo666
да, похожи. в операторе присваивания нужно проверить, не присваиваешь ли самому себе и освободить выделенную ранее память, когда в копирующем конструкторе просто скопировать данные (в случае с указателем - выделить и скопировать)

[revaldo666]

наконецто разобрался, никогда не любил динамические двухмерные массивы...
Теперь полюбил)
всем огромное спасибо

[revaldo666]

И ещё один вопрос.
Нужно перегрузить оператор инкримента, который будет добавлять строку в массив и инициализировать её нулями.
Я сделал так.
но вот чёт оно неработает...
можете сказать где я натупил?

CMatrix& CMatrix::operator++()
{
	
	CMatrix tmp;
	tmp.matrix=new int *[size_x];		//Выделяем память под временный массив
		for(int i=0;i<size_y;i++)
		{
			tmp.matrix[i]=new int [size_y];	
		}
		
		for(int i=0;i<size_y;i++)		//копируем существующий массив во временный
		{
			for (int j=0;j<size_x;j++)
			{
				tmp.matrix[i][j]=matrix[i][j];
			}
		}
		size_y+=1;		//добовляем строку
		matrix=new int *[size_x];
		for(int i=0;i<size_y;i++)
		{
			matrix[i]=new int [size_y];
		}
		for(int i=0;i<size_x;i++)	//заполняем добавленную строку нулями
		{
			matrix[size_y][i]=0;
		}
		for(int i=0;i<size_y-1;i++)	//копируем временный массив в наш изменённый
		{
			for (int j=0;j<size_x-1;j++)
			{
				matrix[i][j]=tmp.matrix[i][j];
			}
		}
		return *this;
	
	

}

[An1ka]

Сообщение от revaldo666

И ещё один вопрос.
Нужно перегрузить оператор инкримента, который будет добавлять строку в массив и инициализировать её нулями.

 CMatrix& CMatrix::operator++()
{
	int i, j;
	//Выделяем память под временный массив
	int **tmat = new int *[size_x];
	for( i=0; i<size_x; i++) tmat[i] = new int [size_y];
	//копируем существующий массив во временный
	for( i=0; i<size_x; i++)
		for( j=0; j<size_y; j++) tmat[i][j] = matrix[i][j];
	//Освобождаем память массива
	for( i=0; i<size_x; i++) delete[] matrix[i];
	delete[] matrix;

	size_y++;
	//Создаем новый массив
	matrix = new int *[size_x];
	for( i=0; i<size_x; i++) matrix[i] = new int [size_y];
	//копируем временный массив в наш изменённый
	for( i=0; i<size_x; i++)
		for( j=0; j<size_y; j++)
			if( j==size_y-1) matrix[i][j] =0; //заполняем добавленную строку нулями
			else matrix[i][j] = tmat[i][j];
	//Освобождаем память временного массива
	for( i=0; i<size_x; i++) delete[] tmat[i];
	delete[] tmat;

  return *this;
}