При попытке очистки классов, деструктор падает

@Segaman · Регистрация: 03.09.2014

Author24 — интернет-сервис помощи студентам

вообщем при создании окна (WM_CREATE). я создаю класс и присваиваю его поинтер в USERDATA.
внутри класса я объявляю массив окон со своими классами в USERDATA.
этот самый массив пытаюсь удалить при WM_CLOSE и WM_DESTROY, но удаление(delete), когда хочет, тогда и срабатывает.
а если не сработает, то деструктор просто перестаёт обработку оставшегося кода и всё.
вот класс:

C++

class ExListview
{
public:
    ExListview(HWND _hwnd);
    ~ExListview();
    HWND        item[itemssmax];
}

вот WM_CREATE:

C++

if (message==WM_CREATE)
    {
        ExListview*b = new ExListview(hwnd);
        SetWindowLongPtr(hwnd, GWLP_USERDATA, (ULONG)b);
        return 0;
    }

вот деструктор:

C++

ExListview::~ExListview()
{
    for(int i=0;i<itemssmax;i++)
    {
        ExList*a = GetExListviewItem(hwnd,i);
        if ((int)a)delete a;
    }
}

функция GetExListviewItem возвращает класс (ExList) присвоенный каждому окну item из класса ExListview.
деструктора у ExList нет.

так вот я пробовал проверять чему равно ExList*a.
пробовал создавать деструктор ExList с выводом текста.
пробовал ставить сообщения перед delete и после delete.
вышло что-то вроде

C++

    MessageBox(0,"start deleting","~ExListview",MB_ICONINFORMATION);
    for(int i=0;i<itemssmax;i++)
    {
        //MessageBox(0,"start item","~ExListview",MB_ICONINFORMATION);
        ExList*a = GetExListviewItem(hwnd,i);
        ShowInt((int)a,"before delete item");
        if ((int)a)delete a;
        ShowInt((int)a,"after delete item");
        //MessageBox(0,"finished item","~ExListview",MB_ICONINFORMATION);
    }
    MessageBox(0,"delete items","~ExListview",MB_ICONINFORMATION);

C++

ExList::~ExList()
{
    MessageBox(0,"delete exlist","~ExList",MB_ICONINFORMATION);
}

так вот, когда всё падает показывается только сообщение перед delete ("before delete item").
т.е. сообщения в ExList деструкторе нет и сообщения после delete тоже нет.
в чем я не прав?

Убежденный · 19.09.2014, 22:23

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

этот самый массив пытаюсь удалить при WM_CLOSE и WM_DESTROY

Выберите что-то одно, иначе получится двойное удаление.

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

SetWindowLongPtr(hwnd, GWLP_USERDATA, (ULONG)b);

Опасно !
Размер ULONG - 4 байта, размер указателя может быть 8 байт (на x64).

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

if ((int)a)delete a;

Незачем проверять на NULL.
По стандарту, delete на нулевом указателе - разрешенная операция.
Про усечение ptr в int уже написал выше.

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

ShowInt((int)a,"before delete item");

Снова усечение. Лучше какой-нибудь printf со спецификатором %p.

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

в чем я не прав?

Хотелось бы видеть деструктор ExListview и функцию GetExListviewItem,
может ошибка там.

@Segaman · 22.09.2014, 15:48 **[ТС]**

Убежденный,

C++

/*=====================================================================================================*/
//получить класс элемента списка
ExList*GetExListviewItem(HWND hwnd, int slot)
{
    ExListview*a = (ExListview*)GetWindowLongPtr(hwnd, GWLP_USERDATA);
    return GetExList(a->item[slot]);
}
/*=====================================================================================================*/

C++

ExListview::~ExListview()
{
    //MessageBox(0,"start deleting","~ExListview",MB_ICONINFORMATION);
    for(int i=0;i<itemssmax;i++)
    {
        //MessageBox(0,"start item","~ExListview",MB_ICONINFORMATION);
        ExList*a = GetExListviewItem(hwnd,i);
        //ShowInt((int)a,"before delete item");
        if ((int)a)delete a;
        //ShowInt((int)a,"after delete item");
        //MessageBox(0,"finished item","~ExListview",MB_ICONINFORMATION);
    }
}

компилирую я только 32битную версию. в 64 надобности нет.

Добавлено через 16 секунд
Убежденный,

C++

/*=====================================================================================================*/
//получить класс элемента списка
ExList*GetExListviewItem(HWND hwnd, int slot)
{
    ExListview*a = (ExListview*)GetWindowLongPtr(hwnd, GWLP_USERDATA);
    return GetExList(a->item[slot]);
}
/*=====================================================================================================*/

C++

ExListview::~ExListview()
{
    //MessageBox(0,"start deleting","~ExListview",MB_ICONINFORMATION);
    for(int i=0;i<itemssmax;i++)
    {
        //MessageBox(0,"start item","~ExListview",MB_ICONINFORMATION);
        ExList*a = GetExListviewItem(hwnd,i);
        //ShowInt((int)a,"before delete item");
        if ((int)a)delete a;
        //ShowInt((int)a,"after delete item");
        //MessageBox(0,"finished item","~ExListview",MB_ICONINFORMATION);
    }
}

компилирую я только 32битную версию. в 64 надобности нет.

@Segaman · 26.09.2014, 16:29 **[ТС]**

Сообщение от Убежденный

Опасно !
Размер ULONG - 4 байта, размер указателя может быть 8 байт (на x64).

так и есть. на Windows 8 программа падает. Куда тогда можно писать указатель?
сосдавать массив ссылок с последующей идентификацией как-то не хочется.

@volvo · 26.09.2014, 17:30

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

Куда тогда можно писать указатель?

Туда же, но не надо приводить его к ULONG, приводи к LONG_PTR

@Segaman · 26.09.2014, 17:54 **[ТС]**

заменил ULONG на LONG_PTR, но на Windows 8 все равно падение при попытке записи в USERDATA.
даже если компилить в 64 бита

@panotnap · 30.09.2014, 20:19

Сделай проверку существования окна (IsWindow) в каждой функции, где оно используется.
"Отбрейк-пойнти" каждую функцию, нет ли где "нулевых" классов / переменных, к которым потом идёт обращение.

Убежденный Ушел с форума 16473 / 7436 / 1187 Регистрация: 02.05.2013 Сообщений: 11,617 Записей в блоге: 1
	19.09.2014, 22:23	2
	Сообщение от Segaman этот самый массив пытаюсь удалить при WM_CLOSE и WM_DESTROY Выберите что-то одно, иначе получится двойное удаление. Сообщение от Segaman SetWindowLongPtr(hwnd, GWLP_USERDATA, (ULONG)b); Опасно ! Размер ULONG - 4 байта, размер указателя может быть 8 байт (на x64). Сообщение от Segaman if ((int)a)delete a; Незачем проверять на NULL. По стандарту, delete на нулевом указателе - разрешенная операция. Про усечение ptr в int уже написал выше. Сообщение от Segaman ShowInt((int)a,"before delete item"); Снова усечение. Лучше какой-нибудь printf со спецификатором %p. Сообщение от Segaman в чем я не прав? Хотелось бы видеть деструктор ExListview и функцию GetExListviewItem, может ошибка там. 0

@Segaman Я в этом не шарю 4 / 4 / 3 Регистрация: 03.09.2014 Сообщений: 191
	26.09.2014, 16:29 [ТС]	4
	Сообщение от Убежденный Опасно ! Размер ULONG - 4 байта, размер указателя может быть 8 байт (на x64). так и есть. на Windows 8 программа падает. Куда тогда можно писать указатель? сосдавать массив ссылок с последующей идентификацией как-то не хочется. 0

@volvo Супер-модератор 32830 / 21168 / 8147 Регистрация: 22.10.2011 Сообщений: 36,428 Записей в блоге: 8
	26.09.2014, 17:30	5
	Сообщение от Segaman Куда тогда можно писать указатель? Туда же, но не надо приводить его к ULONG, приводи к LONG_PTR 0

@Segaman Я в этом не шарю 4 / 4 / 3 Регистрация: 03.09.2014 Сообщений: 191
	26.09.2014, 17:54 [ТС]	6
	заменил ULONG на LONG_PTR, но на Windows 8 все равно падение при попытке записи в USERDATA. даже если компилить в 64 бита 0

@panotnap 31 / 31 / 5 Регистрация: 24.09.2014 Сообщений: 243
	30.09.2014, 20:19	7
	Сделай проверку существования окна (IsWindow) в каждой функции, где оно используется. "Отбрейк-пойнти" каждую функцию, нет ли где "нулевых" классов / переменных, к которым потом идёт обращение. 0

Опции темы