Очень странно вести себя стал
Допустим есть...
char **test = new char*[MAX_TEST];
for(unsigned long i = 0; i <= MAX_TEST; i++)
test[i] = new char[MAX_LONG_TEST+1];

Чтобы освободить память xочу испоьзовать...
for(unsigned long i = 0; i <= MAX_TEST; i++)
delete[] test[i];
delete[] test;
Но программа при выполнении освобождения памяти почему-то сообщает о фатальной ошибке. ЗЫ нигде больше памятью test не управлял, поэтому грешить на удаление несуществующих членов не приходиться.
Да к тому же вот это работает:
delete[] test[0];
delete[] test[MAX_TEST];
delete[] test;

Это сообщение отредактировал(а) En_t_end - 27.6.2005, 11:57

Че-то я не втыкаю, почему ты удаляешь test[i] оператором delete[]. Ведь создаешь ты (char*) так:

Код

char *_string = new char[30]; // а удаляешь delete _string;

ведь __string - не массив указателей, чтобы удалять ее delete[].

Было бы странно, если бы фатальной ошибки не было...

Код

char **test = new char*[MAX_TEST]; for(unsigned long i = 0; i <= MAX_TEST; i++) test[i] = new char[MAX_LONG_TEST+1];

Вы выделяете MAX_TEST кусков памяти, а потом записываете в них MAX_TEST+1 указателей. Элемент с индексом MAX_TEST (test[MAX_TEST]) - незаконный. Программа валится на освобождении памяти, т.к. сама по себе запись в чужую память редко сразу приводит к фатальным последствиям.

commom1
Формально, любой массив можно (нужно) удалять оператором delete[]. При этом компилятор вызывает деструктор для каждого элемента массива. Но в данном случае это, действительно, не обязательно, хотя вполне законно написано. А для массива указателей вызывать delete[] как раз бесполезно: delete для каждого элемента вызван не будет - это ответственность программиста - любишь вызывать new, люби и саночки возить... Или пользуйся smart-указателями.

Это сообщение отредактировал(а) Earnest - 27.6.2005, 12:59

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

Вот код:

Код

#include <iostream> using std::cout; using std::endl; #define MAX_TEST 10 #define MAX_LONG_TEST 11 int main() {     char **test = new char*[MAX_TEST];     for(unsigned long i = 0; i <= MAX_TEST; i++)         test[i] = new char[MAX_LONG_TEST+1];     for(unsigned long i = 0; i <= MAX_TEST; i++)         delete[] test[i];     delete[] test;     system("pause");     return 0; }

Да вы что, господа, какой рабочий! Валится или не валится - это уж как повезет. Причем везет - это когда валится сразу.
Но выделить MAX_TEST элементов, а заполнять MAX_TEST+1 - это, по-вашему, нормально?

Извините не заметил, надо просто в циклах убрать снак равенства и всё будет в ажуре.

Earnest
Тогда покажи пример с char**, где бы память выделялась и освобождалась правильно.

Pablo уже написал - замени в циклах "<=" на "<" и все будет правильно.
Цикл у тебя сколько раз выполняется? Правильно, MAX_TEST+1 раз.
Последний элемент, который ты инициализируешь в цикле - это test[MAX_TEST]. Неважно, что ты туда пишешь - не имешь права, т.к. этого элемента нет.

Это сообщение отредактировал(а) Earnest - 27.6.2005, 15:27

Earnest согласен. Жульничают

Насчёт:

Цитата(Earnest @ 27.6.2005, 13:51)

Формально, любой массив можно (нужно) удалять оператором delete[].

Цитата(Earnest @ 27.6.2005, 13:51)

А для массива указателей вызывать delete[] как раз бесполезно

По-хорошему всегда нужно удалять массив оператором delete[]. Даже в приведённом примере никто не гарантирует, что через какое-то время программу достанут и добавят какую-нибудь новую фичу, использующую переопределённые операторы new, new[], delete, delete[] в глобальной области. Насколько помню, могут быть немаленькие казусы и, главное, вы сначала порвёте все волосы у себя на голове и только потом поймёте, что вы сделали не так.

Earnest
delete[] test[MAX_TEST]; почему тогда это работает ?

Цитата(En_t_end @ 27.6.2005, 16:26)

delete[] test[MAX_TEST]; почему тогда это работает ?

Насколько понимаю, удаляется массив указателей - и удаляется он верного размера. А вот если попытаться удалить элемент MAX+1 в нём - то будут проблемы, т.к. этого элемента не существует.

Цитата(En_t_end @ 27.6.2005, 11:55)

Очень странно вести себя стал Допустим есть... char **test = new char*[MAX_TEST]; for(unsigned long i = 0; i <= MAX_TEST; i++) test[i] = new char[MAX_LONG_TEST+1]; Чтобы освободить память xочу испоьзовать... for(unsigned long i = 0; i <= MAX_TEST; i++) delete[] test[i]; delete[] test; Но программа при выполнении освобождения памяти почему-то сообщает о фатальной ошибке. ЗЫ нигде больше памятью test не управлял, поэтому грешить на удаление несуществующих членов не приходиться. Да к тому же вот это работает: delete[] test[0]; delete[] test[MAX_TEST]; delete[] test;

Попробуй так:

for(unsigned long i = 0; i <= MAX_TEST; i++)
delete[] test[i];
delete[] (char*)test;

Короче может быть чтото с несоответствием типов

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

Почему сразу не валится, это еще и от компилятора зависит. Например, VC в отладочном режиме специально помечает память, выделенную new - а потом при освобождении проверяет, освобождаешь ли ты то, что выделил. Так что с этой точки зрения все, как кажется, нормально. Выделил указатель - освободил указатель. То, где хранится этот указатель, в данном случае не срабатывает - ты просто больше не трогаешь эту память.
Вполне вероятно, что память сразу за массивом test выделяется для какого-то из массивов test[i]. Я думаю, если ты пройдешься по своей программе отладчиком и последишь за значениями переменных, увидишь много интересного. Скорее всего, содержимое ячейки test[MAX_TEST] изменяется во время одного из освобождений. А потом ты вызываешь для нее delete...

Это сообщение отредактировал(а) Earnest - 27.6.2005, 15:46

B2_Russia
Попробывал(предварительно в циклах убрал <= на <(это верно, просто я ступил )) Все равно - ошибка . Может это у моей системы крыша едет ?

А чему равна константа MAX_TEST если не секрет?