удаление динамических массивов

[Dimarik]

Вроде все делаю правильно но во время работы прога выдает ошибку. Очевидно неправильно удаляю массив (удаляю не выделенную память)

for (int i = 0; i < yy; i++)
	{
		uu[i] = new unsigned char[16];//блоки 
		for(int j = 0; j < 16; j++)
		{
			uu[i][j] = iCount;
			iCount++;
		}
	}
for(int j = 0; j < yy; j++)
		delete[] uu[j];
	delete [] uu;

From Stilet: Еще раз код не выделишь тегом - недельную командировку получишь.

[Mandrivnyk]

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

[Dimarik]

unsigned char ** unsCharArray = new unsigned char*[yy];//массив блоков

а вот такое как правильно удалить?

	int *** dynamic = new int**[5];
	for (int i = 0; i < 5; i++)
	{
			dynamic[i] = new int*[4];
			for (int j = 0; j < 4; j++)
			{
				dynamic[i][j] = new int[7];
				for(int k = 0; k < 7; k++)
					dynamic[i][j][k] = k;
			}
	}

и правильно ли я его создаю?

[Mandrivnyk]

    int yy;
    int iCount = 48;    // десятичный ASCII код символа "0"
    cin >> yy;

    unsigned char **unsignedCharArray = new unsigned char *[yy];
    
    for (int i = 0; i < yy; i++)

        {
        unsignedCharArray[i] = new unsigned char[6];//блоки
        for(int j = 0; j < 6; j++)
            {
            unsignedCharArray[i][j] = iCount;
            iCount++;
            }
        }

    for (int i = 0; i < yy; i++) delete[]unsignedCharArray[i];
    delete[]unsignedCharArray;

Это по поводу двумерного массива. Вроде бы, я нигде не напортачил?

[Dimarik]

А как с трёхмерным быть?

[Mandrivnyk]

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

А как с трёхмерным быть?

Уничтожение трёхмерного массива требует дополнительного цикла. Принцип тот же.

for (i = 0; i < 4; i++)
{
    for (j = 0; j < 7; j++) delete[]dynamic[i][j];  
}
for (i = 0; i < 5; i++) delete[]dynamic[i];
delete[] dynamic;

Надеюсь, ничего не перепутал...

Естественно, подразумевается, что 4, 7 и 5 (размерности массива) -- именно некие переменные, иначе нет смысла связываться с динамическими массивами -)

ЗЫ Всех с Пятницей! -))

[onewho]

const int n=10;
const int m=10;
const int z=10;

int i,j;

int*** matr;

matr = new int**[n];
for (i=0; i<n; i++)
matr[i] = new int*[m];

for (i=0; i<n; i++)
for (j=0; j<m; j++)
matr[i][j] = new int[z];


for (i=0; i<n; i++)
for (j=0; j<m; j++)
delete matr[i][j];

for (i=0; i<n; i++)
delete[] matr[i];

delete[] matr;

3-х мерный динамо массив

upd: поправил

[Dimarik]

Спасибо!!!!!!!!!!!!!!!!!!!!!!!!!!!! !!!!!!!!!

for (i = 0; i < 5; i++) delete[]dynamic[i];
delete[] dynamic;
зачем тут цикл? и почему нельзя написать просто последнюю строчку?

Посмотрел в отладчике. Кажется, я не до конца очистил память:

//очищаем память
	for(int j = 0; j < 4; j++)
	{
		for (int k = 0; k < 4; k++)
			delete[] unsCharArray[j][k];
		delete[] unsCharArray[j];
	}
	delete [] unsCharArray;

массив создавался размерностью unsCharArray[I_][J_][K_]в счётчике использовал названия переменных похожие на названия переменных, определяющих размерность массива чтобы понятнее было что к чему. Правильно ли я удаляю? Потому что кажись не до конца тут удалено

сТРАННО. Сейчас ошибка: обращение по невыделенной памяти access violantation

for(int j = 0; j < 4; j++)
	{
		for (int k = 0; k < 4; k++)
			delete[] unsCharArray[j][k];//вот тут ошибка при j =2; k = 0
		delete[] unsCharArray[j];
	}
	delete [] unsCharArray;

Но что самое интересное, вот такой кусок кода перед удалением работает нормально!!!

for (int i = 0; i < _iBlocks; i++)
	{
		for(int j = 0; j < 4; j++)
		{
			for (int k = 0; k < 4; k++)
				fObject<<int(unsCharArray[i][j][k])<<'\t';
		}
		fObject<<endl;
	}

[Mandrivnyk]

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

for (i = 0; i < 5; i++) delete[]dynamic[i];
delete[] dynamic;
зачем тут цикл? и почему нельзя написать просто последнюю строчку?

Как ты уже сам понял, организация трехмерного динамического массива производится в три этапа.
Сначала создается одномерный массив указателей (i), потом каждому элементу этого массива присваивается адрес двумерного массива, который (массив) также является массивом указателей (j), каждый из которых указывает уже на одномерный массив указателей (k).
Таким образом, первый из созданных массивов (i) является массивом указателей на указатели на указатели.
Немного запутано -- пятница, все-таки -) -- но, думаю, идея понятна.

При удалении надо освободить память в обратном порядке.
Вначале освобождается память, выделенная под массив (k) -- двойным циклом по индексам (j) и (i).
Затем -- память, выделенная под массив (j) -- циклом по индексу (i).
И только в самом конце -- память, выделенная под массив (i). Цикл не нужен.

Чтобы убедиться в этом -- просто посчитай, для скольких элементов ты вызвал оператор new() и сравни с числом вызовов оператора delete().
Насколько я понимаю, количество вызовов должно быть равно.

Именно поэтому ты и получал ошибку Access Violation

Я ответил на твой вопрос?

[Dimarik]

Я сделал всё как ты мне сказал, но он кажись не удалился.

int _tmain(int argc, _TCHAR* argv[])
{ 
	
	int iCount = 0;
	int *** dynamic = new int**[5];
	for (int i = 0; i < 5; i++)
	{
			dynamic[i] = new int*[6];
			for (int j = 0; j < 6; j++)
			{
				dynamic[i][j] = new int[7];
				for(int k = 0; k < 7; k++)
				{
					dynamic[i][j][k] = iCount;
					iCount++;
				}
			}
	}
	
	fstream file;
	file.open("C:\\file.txt", ios::out);
	for (int i = 0; i <  5; i++)
	{
		for (int j = 0; j < 6; j++)
		{
			for (int k = 0; k < 7; k++)
			{
				file<<"dynamic["<<i<<"]["<<j<<"]["<<k<<"] = "<<dynamic[i][j][k]<<"\t";
			}
			file<<endl;
		}
		file<<endl;
	}
	for (int i = 0; i < 5; i++)
		for(int j = 0; j < 6; j++)
			delete[] dynamic[i][j];
	for (int i = 0; i < 5; i++)
		delete[] dynamic[i];
	delete[] dynamic;
	return 0;
}