0

Согласно документации HANDLE - это просто указатель типа void*. Я решил сделать обертку-класс для этого указателя, чтобы контролировать его цикл жизни. Выглядит примерно так:

class AnyHandler {

private:

    HANDLE handle;

public:

    AnyHandler();
    AnyHandler(const AnyHandler&);
    AnyHandler(AnyHandler&&);
    ~AnyHandler();

    AnyHandler& operator=(const AnyHandler&);
    AnyHandler& operator=(AnyHandler&&);

    AnyHandler(const HANDLE);
    AnyHandler& operator=(const HANDLE);

    const HANDLE get() const;
    void set(const HANDLE);

};

Вопрос таков: как реализовывать конструкторы копирования и перемещения для HANDLE? Ведь если у нас будет конструктор копирования + конструктор перемещения, то копирование должно быть глубоким. То есть с выделением новой динамической памяти и копирование в него объекта, на который указывает исходный HANDLE. Но мы не знаем ни тип, ни на что указывает void*. В таком случае напрашивается вывод, что перемещение для данной обертки не имеет смысла и можно оставить просто поверхностное копирование указателя в конструкторе копирования? Но ведь в таком случае у нас может быть два объекта, указывающих на один и тот же адрес памяти. И будет плохо, если один из них удалится, освободив память, на которую указывает HANDLE. В таком случае второй указатель станет висячим и вызовет проблемы при попытке разыменовать его или освободить. Как поступить в таком случае?

7
  • 1
    Мне кажется, что тут нужно запретить копирование/перемещение вообще, или что-то типа shared_ptr — со счетчиком владения.
    – Harry
    Commented 31 окт. 2021 в 9:36
  • @Harry я тоже подумал про shared_ptr, но я вот не знаю, можно ли его самого будет сюда впихнуть. Не будет ли конфликтов с WinAPI. Или же придется делать свою собственную реализацию? Все таки это void* и неизвестно на что он указывает и как преобразуется в функциях WinAPI
    – ComeInRage
    Commented 31 окт. 2021 в 9:38
  • Ну, можно свой счетчик набросать, это несложно :) Но мне все же кажется, что это как поток в С++ — некопируемая вещь...
    – Harry
    Commented 31 окт. 2021 в 9:43
  • @Harry в WinAPI HANDLE передается в функции по значению. Если удалить конструкторы копирования и перемещения, разве это не помешает ему передаваться в функции?
    – ComeInRage
    Commented 31 окт. 2021 в 9:49
  • 1
    Конструктора копирования быть не должно. Для хранения именно копируемых дескрипторов имеет смыл делать явный метод Clone(). А перемещение - реализуется тривиально. Commented 31 окт. 2021 в 10:02

1 ответ 1

1

Все такие обертки пишутся одинаково. Я бы делал ее не универсальной, а под конкретный тип содержимого.

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

class Foo
{
    // Если больше одного поля, сложите их во вложенную структуру, для удобства
    // написания перемещающего конструктора и оператора присваивания.
    HANDLE handle = nullptr; 

  public:
    Foo() {}

    // Конструктор, который заполняет чем-то `handle`.
    Foo(...) {...}

    Foo(Foo &&other) noexcept : handle(std::exchange(other.handle, nullptr)) {}

    // Заметьте, нет `&&`.
    // Прием назвается copy&swap idiom.
    Foo &operator=(Foo other) noexcept
    {
        std::swap(handle, other.handle);
        return *this;
    }

    ~Foo()
    {
        if (handle)
            // уничтожить handle
    }
};

Если хотите, добавьте конструктор копирования. При этом копирующий operator= не нужен, потому что этот может работать и как перемещающий (если есть перемещающий конструктор), и как копирующий (если есть копирующий конструктор).

Еще, в таких обертках хорошо смотрится explicit operator bool() const {return bool(handle);}, чтобы проверять их на пустоту.

1
  • Спасибо, учту подобный подход. Однако в данном случае мне действительно подошел лучше std::shared_ptr. Он имеет все, что мне нужно, кроме одного метода, который я добавил, унаследовавшись от std::shared_ptr<void>, поскольку HANDLE - это указатель типа void
    – ComeInRage
    Commented 31 окт. 2021 в 12:14

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.