Следует ли утилизировать HANDLE, возвращенный CreateEvent?

Question

Следует ли утилизировать HANDLE, возвращенный CreateEvent?

0

Я проверял чей-то код, и я увидел это:

template<typename Data>
class ConcurrentQueue {

    private:
        HANDLE dataPushEvent;
        // More Private Members...

    public:
        ConcurrentQueue() {
            dataPushEvent = CreateEvent(NULL, TRUE, FALSE, NULL);
        }

    // Public Methods...

};

Как вы можете видеть, в этом классе нет деструктора, и dataPushEvent явно не освобождается нигде в этом классе. Поскольку это частный член, он не может быть использован снаружи, поэтому я думал, что это, вероятно, приведет к утечке памяти.

Если этот дескриптор не должен быть удален.

Я новичок в программировании C++ и Windows. Насколько мне известно, HANDLE - это void *, и, как и все указатели, ее ссылка должна быть выпущена, когда мы закончили ее использование.

Я прав?

Matias Cicero 05 янв. 2015, в 18:17

Источник

0

CloseHandle() должен вызываться для dataPushEvent если вы хотите освободить его до завершения процесса.
Alex K. 05 янв. 2015, в 17:04
2

Это HANDLE , верно?
Roger Lipscombe 05 янв. 2015, в 17:07
0

@AlexK. А ну понятно! Должен ли я попытаться вручную delete указатель, если CloseHandle() не работает?
Matias Cicero 05 янв. 2015, в 17:09
0

HANDLE имеет тип void* , но на самом деле не является указателем. Это не указывает ни на что и не может быть разыменовано. Вы не можете освободить дескриптор как указатель (не пытайтесь delete его).
ElderBug 05 янв. 2015, в 17:09
2

@MatiCicero Нет, но вы должны подтвердить возвращаемое значение из CloseHandle() - если это не удастся, возможно, у вас ошибка и / или повреждение памяти, и вы хотите, чтобы программа вызывала аварийный сбой, чтобы вы знали, что есть проблема.
cdhowie 05 янв. 2015, в 17:11

Показать ещё 3 комментария

Теги:

c++

windows

winapi

handle

1 ответ

Ещё вопросы

CloseHandle() должен вызываться для dataPushEvent если вы хотите освободить его до завершения процесса.
@AlexK. А ну понятно! Должен ли я попытаться вручную delete указатель, если CloseHandle() не работает?
HANDLE имеет тип void* , но на самом деле не является указателем. Это не указывает ни на что и не может быть разыменовано. Вы не можете освободить дескриптор как указатель (не пытайтесь delete его).
@MatiCicero Нет, но вы должны подтвердить возвращаемое значение из CloseHandle() - если это не удастся, возможно, у вас ошибка и / или повреждение памяти, и вы хотите, чтобы программа вызывала аварийный сбой, чтобы вы знали, что есть проблема.

cdhowie · Accepted Answer · 2015-01-05T16-03-00.000Z

Если нет деструктора, то да, ручка просочилась. Для уничтожения события должен быть деструктор, вызывающий CloseHandle().

В качестве побочного примечания класс должен также удалить (С++ 11) или сделать недоступным (С++ 03) конструктор копирования и операторы присваивания копий, так как значения по умолчанию для компилятора не будут иметь смысла - они, скорее всего, скопируют в очереди, но все же ссылаются на один и тот же дескриптор события, заставляя несколько кажущихся независимыми очередей разделить событие без уважительной причины. И когда вы реализуете деструктор, реализация копии по умолчанию будет еще хуже, потому что, как только одна копия будет уничтожена, она уничтожит дескриптор события, который все еще используется всеми другими копиями!

В С++ 11 вы можете реализовать move-constructor и операторы присваивания move, но вам придется настроить деструктор на учет, чтобы тот факт, что дескриптор мог быть перемещен на другой объект, чтобы он не вызовите CloseHandle().

Спасибо за совет C ++ 11 и C ++ 03! У меня нет доступа к C ++ 11, поэтому я думаю, что я должен сделать их конфиденциальными!
@MatiCicero Да, объявите их как частные и не реализуйте их . (Если вы используете их случайно в контексте, в котором они видны, вы хотите получить ошибку во время соединения, чтобы вы знали, что вы случайно использовали их.)
Также стоит отметить: если конструктор содержит код после вызова CreateEvent который может генерировать исключения, вы все равно пропускаете HANDLE , так как деструктор не запустится. Вероятно, неплохо иметь оболочку RAII исключительно для объекта Event.
@ IInspectable Да, очень хорошая мысль. Если этот объект сделан не подлежащим копированию (а также перемещаемым в C ++ 11), то ConcurrentQueue автоматически станет также не подлежащим копированию и / или перемещаемым, и пользовательский деструктор не понадобится!