Нужно удалить указатель на массив - на строке с delete вылетает Debug Error.

[ImmortalAlexSan]

Привет всем в очередной раз

bool DeletePointer(WCHAR* pPointer)
{
	if (pPointer != NULL) {
		delete [] pPointer;
		pPointer = NULL;
		return true;
	}
	else
		return false;
};

Пытаюсь вот таким макаром удалить указатель на массив. Смотрю в дебагере: pPointer принимает нужное значение и инфа в *pPointer хранится соответствующая. Но на строке с delete вылетает Debug Error. Может быть, я не правильно передаю указатель в функцию? Пробовал и указатель на указатель передавать, ничего не меняется.
А вот такое удаление работает нормально:

bool DeletePointer(WCHAR** pPointer)
{
	if (*pPointer != NULL) {
		delete pPointer;
		*pPointer = NULL;
		return true;
	}
	else
		return false;
};
Но что-то я не понимаю, что в данном случае удаляется? Созданный функцией указатель удаляется?

[Гром]

Как минимум в первом случае вы не сможете присвоить указателю значение NULL (или какое-то другое), т.к. переменную типа "указатель на WCHAR" вы передаете по значению, и этой переменной нельзя присвоить никакое новое значение (можно только локальной копии, которая будет уничтожена при выходе из функции - а это бессмысленно).
Во втором случае у вас тоже ошибка - вы передаете переменную "указатель на WCHAR" по указателю и присваиваете ей NULL корректно, однако для удаления массива необходимо вызвать delete[] для указателя на массив, т.е. нужна такая строчка:

delete[] (*pPointer);

Еще один вариант - т.к. указатель тоже является переменной, то эту переменную можно передавать не только по указателю, но и по ссылке. Фактически получится почти то же, что и в первом варианте:

bool DeletePointer((WCHAR*)& pPointer)   //Ссылка на переменную типа WCHAR*
{
	if (pPointer != NULL) {
		delete [] pPointer;
		pPointer = NULL;
		return true;
	}
	else
		return false;
};

Не уверен, что это гарантированно поможет, но так будет хотя бы меньше ошибок

[Rififi]

ImmortalAlexSan

Но на строке с delete вылетает Debug Error.

А вот такое удаление работает нормально:

Забавный все таки язык программирования C++. Вот есть два кода - один корректный, другой - говнокод. Но при этом первый вызывает ошибку, а второй отрабатывает нормально (((:

Может быть, я не правильно передаю указатель в функцию?

ты неправильно выделаешь память и/или работаешь с ней.

[profi]

ImmortalAlexSan, код правильный. Может ты не правильно передаешь. Вот быстро набросал, правда я не WCHAR юзал.

#include <iostream>
#include <string.h>

bool DeletePointer(char *&pPointer)
{
	if (pPointer != NULL) {
		delete [] pPointer;
		pPointer = NULL;
		return true;
	}
	else
		return false;
}

int main()
{
    char *buff = new char[255];

    strcpy(buff, "Hello");

    std::cout << buff << "\n";

    std::cout << DeletePointer(buff);

    return 0;
}

[ImmortalAlexSan]

Гром, bool DeletePointer((WCHAR*)& pPointer), компилятор не пропускает, подчеркивает.
delete[] (*pPointer); - с этим такая же канитель.
Если ошибка лежит глубже, то я с памятью работаю и выделяю её в одной лишь функции:

void SetNodeField(WCHAR* pSource, WCHAR** pTaker)
{
	WCHAR* PtrS = pSource;
	unsigned char i = 0;
	unsigned char j = 0;

	DeletePointer(pTaker);
	if (*pTaker != NULL) {
		delete [] *pTaker;
		*pTaker = NULL;
	}
	while (*PtrS != '\0') {
		j++;
		PtrS++;
	};
	PtrS = pSource;
	*pTaker = new WCHAR[j];
	for (i = 0; i <= j; i++) {
		(*pTaker)[i] = *PtrS;
		if (i != j)
			PtrS++;
	};
};

После этого я могу вывести данные, что лежат по нужному адресу нормально.
И кстати, после выпадения ошибки дебагера, жму "пропустить", и прога работает дальше корректно))) Опять всё нормально считывается из памяти.
profi, попробовал вызвать функцию не из функции, а из основного кода, - такая же хрень.

[profi]

ImmortalAlexSan ну теперь ясно . Ты же не правильно указатель удаляешь. Ты в функцию передавал массив указателей.

Сигнатура функции:

bool DeletePointer(WCHAR* pPointer)

Передаешь:

WCHAR** pTaker

Ниче не смущает?

bool DeletePointer(WCHAR** pPointer)
{
	if (*pPointer != NULL) {
		delete pPointer;
		*pPointer = NULL;
		return true;
	}
	else
		return false;
}

Этот код работал правильно, так как ты удалял указатель, который является первым элементом массива указателей pPointer. Здесь нужно циклом под массив указателей выделить память, а потом циклом её освободить.
Вот исправил, как то так:

#include <iostream>
#include <string.h>

bool DeletePointer(char **&pPointer)
{
    for(int i = 0; i < 3; i++)
    {
        if(pPointer[i] != NULL)
        {
            delete []pPointer[i];
            pPointer[i] = NULL;
        }
    }

	if (pPointer != NULL) {
		delete []pPointer;
		pPointer = NULL;
		return true;
	}
	else
		return false;
}

int main()
{
    char **buff;
    buff = new char*[3];
    for(int i = 0; i < 3; i++)
        buff[i] = new char[255];

    std::cout << DeletePointer(buff);

    return 0;
}

[ImmortalAlexSan]

profi, у меня не двумерный массив... У меня просто

char *buff;

P.S. Создал новый проект, написал там:

bool DeletePointer(char *pPointer)
{
	if (pPointer != NULL) {
		delete []pPointer;
		pPointer = NULL;
		return true;
	}
	else
		return false;
}

int _tmain(int argc, _TCHAR* argv[])
{
    char *buff;
    buff = new char[3];
    std::cout << DeletePointer(buff);
    std::cin.get();
    return 0;
}

Всё работает как надо... Но buff не обнуляется... Щас указатель на buff передам, посмотрим. Передал указатель. Вс работает как надо) Но это не совсем та структура, что в другом проекте... Щас поробую реализовать.

[profi]

void SetNodeField(WCHAR* pSource, WCHAR** pTaker)
{
	WCHAR* PtrS = pSource;
	unsigned char i = 0;
	unsigned char j = 0;

	DeletePointer(pTaker);

Я указал примерный путь как исправить ошибку по листингу, который ты привел. Дерзай. Я же не видел твой код полностью. Мой пример в посте #4 полностью правильный.

[ImmortalAlexSan]

profi, спасибо огромное вам! Я кажется понял в чем ошибка. С удалением указателя в функции придется работать как с двумерным динамич. массивом, сейчас проверю довод... Сначала проверил так:

struct Second
{
	WCHAR* text;
};

struct First
{
	Second* second;
};

First* GetPtr()
{
	First* first = new First();
	first->second = new Second();
	first->second->text = new WCHAR[10];
	return first;
};

bool DeletePointer(WCHAR** pPointer)
{
	if (*pPointer != NULL) {
		delete [] *pPointer;
		*pPointer = NULL;
		return true;
	}
	else
		return false;
}

int _tmain(int argc, _TCHAR* argv[])
{
    First* first;
    first = GetPtr();
	std::cout << DeletePointer(&first->second->text);
	std::cin.get();
    return 0;
}

Всё работает как по маслу - без ошибок по крайней мере. Правильно я сделал? Всю память, что выделял под тип WCHAR освободил? Как проверить утечку ребят?

[profi]

Огогого сколько указателей . Кстати сделай лучше так:

bool DeletePointer(WCHAR*& pPointer)
{
	if (pPointer != NULL) {
		delete [] pPointer;
		pPointer = NULL;
		return true;
	}
	else
		return false;
}
........
std::cout << DeletePointer(first->second->text);

P.S. Просто так понятней будет.