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

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

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

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

C++
1
2
3
4
5
6
7
class ExListview
{
public:
    ExListview(HWND _hwnd);
    ~ExListview();
    HWND        item[itemssmax];
}

вот WM_CREATE:

C++
1
2
3
4
5
6
if (message==WM_CREATE)
    {
        ExListview*b = new ExListview(hwnd);
        SetWindowLongPtr(hwnd, GWLP_USERDATA, (ULONG)b);
        return 0;
    }

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

C++
1
2
3
4
5
6
7
8
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++
1
2
3
4
5
6
7
8
9
10
11
    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++
1
2
3
4
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++
1
2
3
4
5
6
7
8
/*=====================================================================================================*/
//получить класс элемента списка
ExList*GetExListviewItem(HWND hwnd, int slot)
{
    ExListview*a = (ExListview*)GetWindowLongPtr(hwnd, GWLP_USERDATA);
    return GetExList(a->item[slot]);
}
/*=====================================================================================================*/

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
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++
1
2
3
4
5
6
7
8
/*=====================================================================================================*/
//получить класс элемента списка
ExList*GetExListviewItem(HWND hwnd, int slot)
{
    ExListview*a = (ExListview*)GetWindowLongPtr(hwnd, GWLP_USERDATA);
    return GetExList(a->item[slot]);
}
/*=====================================================================================================*/

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
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) в каждой функции, где оно используется.
"Отбрейк-пойнти" каждую функцию, нет ли где "нулевых" классов / переменных, к которым потом идёт обращение.

Новые блоги и статьи Все статьи Все блоги /
ПЛИС zxcha1ka_ 27.01.2026 AHDL Разработать программы для синтеза следующих устройств: 1. Параллельного регистра 4-х разрядного с синхронной загрузкой и асинхронным сбросом (обнулением); Пoмoгитe пoжaлyйстa	Загрузка PNG-файла с альфа-каналом с помощью библиотеки SDL3_image на Android 8Observer8 27.01.2026 Содержание блога SDL3_image - это библиотека для загрузки и работы с изображениями. Эта пошаговая инструкция покажет, как загрузить и вывести на экран смартфона картинку с альфа-каналом, то есть с. . .	влияние грибов на сукцессию anaschu 26.01.2026 Бифуркационные изменения массы гриба происходят тогда, когда мы уменьшаем массу компоста в 10 раз, а скорость прироста биомассы уменьшаем в три раза. Скорость прироста биомассы может уменьшаться за. . .	Воспроизведение звукового файла с помощью SDL3_mixer при касании экрана Android 8Observer8 26.01.2026 Содержание блога SDL3_mixer - это библиотека я для воспроизведения аудио. В отличие от инструкции по добавлению текста код по проигрыванию звука уже содержится в шаблоне примера. Нужно только. . .
Установка Android SDK, NDK, JDK, CMake и т.д. 8Observer8 25.01.2026 Содержание блога Перейдите по ссылке: https:/ / developer. android. com/ studio и в самом низу страницы кликните по архиву "commandlinetools-win-xxxxxx_latest. zip" Извлеките архив и вы увидите. . .	Вывод текста со шрифтом TTF на Android с помощью библиотеки SDL3_ttf 8Observer8 25.01.2026 Содержание блога Если у вас не установлены Android SDK, NDK, JDK, и т. д. то сделайте это по следующей инструкции: Установка Android SDK, NDK, JDK, CMake и т. д. Сборка примера Скачайте. . .	Использование SDL3-callbacks вместо функции main() на Android, Desktop и WebAssembly 8Observer8 24.01.2026 Содержание блога Если вы откроете примеры для начинающих на официальном репозитории SDL3 в папке: examples, то вы увидите, что все примеры используют следующие четыре обязательные функции, а. . .	моя боль iceja 24.01.2026 Выложила интерполяцию кубическими сплайнами www. iceja. net REST сервисы временно не работают, только через Web. Написала за 56 рабочих часов этот сайт с нуля. При помощи perplexity. ai PRO , при. . .

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

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